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

Outfile not recognized when -r is specified (box removal) #55

Closed
pidgeon777 opened this issue Feb 8, 2019 · 6 comments
Closed

Outfile not recognized when -r is specified (box removal) #55

pidgeon777 opened this issue Feb 8, 2019 · 6 comments
Labels

Comments

@pidgeon777
Copy link

@pidgeon777 pidgeon777 commented Feb 8, 2019

This is what I actually do into Notepad++ to generate a box around a text selection, according to the docs:

boxes.exe -d stone -s 80 -ac "$(ascfile)" "$(tempfile)"

By doing so, it works.

But, if I want to remove the box through:

boxes.exe -r -d stone "$(ascfile)" "$(tempfile)"

then the resulting output is sent to the stdout (shown in NppExec console), and not to the $(tempfile) file, thus the originally selected text is replaced with an empty string.

I'm using boxes version 1.2.

@tsjensen
Copy link
Member

@tsjensen tsjensen commented Feb 8, 2019

Good catch! While this is being fixed, you can use the following workaround:

cmd /c boxes -r -d stone "$(ascfile)" > "$(tempfile)"

The NppExec manual actually says one should do it like this, but according to the boxes docs, what you did should have worked, too.

@tsjensen tsjensen added the bug label Feb 8, 2019
@tsjensen
Copy link
Member

@tsjensen tsjensen commented Feb 8, 2019

I updated the docs to avoid further confusion.
Fixing this in the code will take much longer, but we should be good for now. 😉

@pidgeon777
Copy link
Author

@pidgeon777 pidgeon777 commented Feb 8, 2019

Thank you for the prompt reply, your proposed solution is actually working good. I also think it can be considered a good workaround until a solution to the bug will be found.

@tsjensen tsjensen changed the title Boxes removal doesn't work with Notepad++ Outfile not recognized when -r is specified (box removal) Feb 13, 2019
@tsjensen
Copy link
Member

@tsjensen tsjensen commented Feb 14, 2019

Fixed in master. This will go out with the next release, but be patient as that may take some time.

@tsjensen tsjensen closed this Feb 14, 2019
@pidgeon777
Copy link
Author

@pidgeon777 pidgeon777 commented Feb 14, 2019

No problem, I will wait, then.

@tsjensen
Copy link
Member

@tsjensen tsjensen commented Feb 21, 2019

Released as 1.3

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

Successfully merging a pull request may close this issue.

None yet
2 participants