8000 Copy and move do not honor passive pane directory as targets · Issue #124 · sunrise-commander/sunrise-commander · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Copy and move do not honor passive pane directory as targets #124

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
scarpazza opened this issue Feb 8, 2023 · 1 comment
Open

Copy and move do not honor passive pane directory as targets #124

scarpazza opened this issue Feb 8, 2023 · 1 comment

Comments

@scarpazza
Copy link
scarpazza commented Feb 8, 2023 •
edited
Loading

Functions sunrise-do-copy and sunrise-do-rename, contrary to their expected behavior, offer to copy and to move files always to the home directory rather than the directory that the passive pane is showing.

I originally believed this was due to interference with package helm, but that's not the case. The bug appears whether or not helm is active.

This is under Emacs 27.2, compiled from SRPMS, running on Linux x86_64.
@scarpazza scarpazza changed the title Copy and move do not honor passive pane directory as targets when helm is active Copy and move do not honor passive pane directory as targets Feb 8, 2023
@enzuru
Copy link
Contributor
enzuru commented Feb 8, 2023 •
edited
Loading

I have an old PR that mostly fixes that behavior, but it's still buggy and hacky: #119

Do we have confirmation that the current Sunrise Commander is fine on old versions of Emacs? Do we know which was the last working Emacs version?

TBH, I don't think that I know the codebase well enough to get this PR across the finish line. When I began working in it, it appeared that the codebase was halfway through some kind of refactor (like left and right being replaced inconsistently with a and b, and the codebase referencing functions that no longer existed like sunrise-setup-windows). There are also a bunch of Sunrise Commander features that I don't use (or even know of!), so I'm not always sure what I'm breaking.

I think we'll need the support of someone like @lassik to get this PR across the finish line, or develop a new PR.

PS: Always nice to see a fellow New Yorker on GitHub

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@enzuru @scarpazza
0