Working in diff filesystem doesn't allow copy operation

[volo78]

I can remove/copy patches from diff filesystem but I cannot copy to it and I cannot save changes to patches in diff filesystem. It was possible in older versions of mc. I searched for a corresponding ticket but it seems no one reported it before.

[volo78]

Additional note. When I remove a patch from diff filesystem there still remains an artifact like this.

Index: ChangeLog?
===================================================================

I doesn't hurt too much but clutters the whole patch.

[andrew_b]

These regressions were introduced in #90 (changeset:b29e81556f67087955a9eceec45347d002c781f1).

[kdave]

Replying to volo78:

Additional note. When I remove a patch from diff filesystem there still remains an artifact like this.

Index: ChangeLog?
===================================================================

I doesn't hurt too much but clutters the whole patch.

This is in the 'comment' part of a patch and may be useful in some cases. Use the 'filterdiff' utility to remove all artifacts.

[kdave]

Replying to andrew_b:

These regressions were introduced in #90 (changeset:b29e81556f67087955a9eceec45347d002c781f1).

'copyin' operation is not supported and was broken in previous version. See my comment/todo:

sub copyin($$$) {
    # TODO: seems to be tricky. what to do?
    # copyin of file which is already there may:
    #  * delete the original and copy only the new
    #  * just append the new hunks to the same file
    #    problems: may not be a valid diff, unmerged hunks
    #  * try to merge the two together
    #    ... but we do not want write patchutils again, right?
    error "Copying files into diff not supported";
    return;

The only safe case is if any of the files the diff changes does not exist in the original diff. Any other case must be verified to produce a valid diff.

[volo78]

Replying to kdave:

Replying to andrew_b:

These regressions were introduced in #90 (changeset:b29e81556f67087955a9eceec45347d002c781f1).

'copyin' operation is not supported and was broken in previous version. See my comment/todo:

sub copyin($$$) {
    # TODO: seems to be tricky. what to do?
    # copyin of file which is already there may:
    #  * delete the original and copy only the new
    #  * just append the new hunks to the same file
    #    problems: may not be a valid diff, unmerged hunks
    #  * try to merge the two together
    #    ... but we do not want write patchutils again, right?
    error "Copying files into diff not supported";
    return;

The only safe case is if any of the files the diff changes does not exist in the original diff. Any other case must be verified to produce a valid diff.

Does this mean that this bug will not be fixed? I understand the difficulty but there is one important part (at least for me) that you didn't address in your reply. If I change a patch in patchfs I cannot save the change. Strictly speaking saving a patch doesn't differ too much from copying into patch but that's the functionality that I miss.

Another note. Why mc should be smart? Let's allow a user to be smart. Even if the previous functionality was broken (I mean the functionality of 4.6) no one complained as far as can see. It used to be as in your comment "* delete the original and copy only the new" and even if sometimes it could produce invalid diff then only user was responsible for this.