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

Extra single quotes are getting stripped in webconfig #8917

Closed
YmirKhang opened this issue Apr 27, 2022 · 3 comments
Closed

Extra single quotes are getting stripped in webconfig #8917

YmirKhang opened this issue Apr 27, 2022 · 3 comments
Labels
bug Something that's not working as intended
Milestone

Comments

@YmirKhang
Copy link
Contributor

Fish version 3.1.0
Ubuntu 20.04
xterm-256color

This issue can be reproduced regardless of third party customizations are added or not.

Whenever an abbreviation ending with a single quote is added via fish_config UI, the single quote at the end gets stripped. Even if the command is wrapped with one more layer of single quotes.

So setting the word _first to awk '{print $1'} actually executes the following command:

abbr --add '_first' 'awk \'{print $1}'

Also setting the work _first with another layer of single quotes: 'awk '{print $1'}', results in the execution of the same command:

abbr --add '_first' 'awk \'{print $1}'
@faho
Copy link
Member

faho commented Apr 28, 2022

Note: Your reproducer has what I assume is a typo:

awk '{print $1'}

works - there's no ' at the end, so there's nothing to "strip".

However I can still reproduce this with

awk '{print $1}'

which also makes much more sense.


Also:

Fish version 3.1.0

fish 3.1.0 was already outdated by the time Ubuntu 20.04 was released, and that's not even the current LTS anymore (the current LTS 22.04, incidentally also ships an already outdated fish).

It is 3930 released commits behind, out of a total 15379 released commits - that's 25% of all commits ever made to fish (and released)!

Please update if you want to report any further issues.

@faho
Copy link
Member

faho commented Apr 28, 2022

Also: We do not require separate issues to be filed if the PR is simple.

In future please just file the PR if you have one, otherwise it's just more clicking for me.

That being said: Thank you for your report, I believe it is fixed now!

@faho faho closed this as completed Apr 28, 2022
@faho faho added the bug Something that's not working as intended label Apr 28, 2022
@faho faho added this to the fish 3.5.0 milestone Apr 28, 2022
@YmirKhang
Copy link
Contributor Author

Also: We do not require separate issues to be filed if the PR is simple.

In future please just file the PR if you have one, otherwise it's just more clicking for me.

That being said: Thank you for your report, I believe it is fixed now!

Notes. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something that's not working as intended
Projects
None yet
Development

No branches or pull requests

2 participants