Skip to content
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.

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

fix/paste from wsl #1994

Merged
merged 2 commits into from
Feb 5, 2025
Merged

fix/paste from wsl #1994

merged 2 commits into from
Feb 5, 2025

Conversation

psicho2000
Copy link
Contributor

  1. In Windows, lines are separated by CRLF. We need to remove the CR (carriage return) in order for IFS to work in all cases (i.e., unixoid and bsd like OS on one hand and Windows on the other hand).
  2. When calling the script, we never get to know why something breaks. Therefore, we are adding error messages. This is of value if we call the script via fzplug which prints the error (or, wenn quitting from nnn).
  3. I am honestly unsure what value this script provides. I see that it successfully writes to the selection file (reproducible), but nnn never seems to read from that file, it only writes into it. @jarun could you provide some insight about that?

@jarun jarun merged commit af9bfd3 into jarun:master Feb 5, 2025
8 checks passed
@jarun
Copy link
Owner

jarun commented Feb 5, 2025

Thank you!

@psicho2000 psicho2000 deleted the fix/paste-from-wsl branch February 5, 2025 18:09
@psicho2000
Copy link
Contributor Author

@jarun: Thanks for merging!
Regarding my question above: is nnn even supposed to read from the selection file, and if so, how?

@jarun
Copy link
Owner

jarun commented Feb 5, 2025

Please follow selpath in the code. It is read in several use cases, e.g. move, copy, list selection etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants