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

install script incorrectly append PATH if it contains spaces #18291

Closed
ezimore opened this issue Aug 17, 2023 · 1 comment · Fixed by #18294
Closed

install script incorrectly append PATH if it contains spaces #18291

ezimore opened this issue Aug 17, 2023 · 1 comment · Fixed by #18294
Labels
domain: releasing Anything related to releasing Vector type: bug A code related bug.

Comments

@ezimore
Copy link

ezimore commented Aug 17, 2023

A note for the community

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

Problem

Vector installation removes the apostrophe ('mydir with spaces') and escape characters (mydir\ with\ spaces) from the PATH variable which makes existing configured commands and vector command not found.

Configuration

No response

Version

vector 0.31.0 (x86_64-unknown-linux-gnu 0f13b22 2023-07-06 13:52:34.591204470)

Debug Output

No response

Example Data

Prerequisites:

  • vector not installed
  • PATH variable contains spaces e.g. PATH=/home/myuser/mydir\ with\ spaces:'/home/myuser/2nddir with spaces'

After vector installation, the PATH variable looks like:
PATH=/home/myuser/mydir with spaces:/home/myuser/2nddir with spaces:/home/myuser/.vector/bin

PATH variable is corrupted. Vector command is not available, even more other commands on the directories added earlier into PATH are not available as well.

the correct behaviour would be keeping the original PATH string as it is and appending only the vector path at the end
PATH=/home/myuser/mydir\ with\ spaces:'/home/myuser/2nddir with spaces':/home/myuser/.vector/bin

Additional Context

No response

References

No response

@ezimore ezimore added the type: bug A code related bug. label Aug 17, 2023
jszwedko added a commit that referenced this issue Aug 17, 2023
Closes: #18291

Signed-off-by: Jesse Szwedko <jesse.szwedko@datadoghq.com>
@jszwedko
Copy link
Member

Thanks for this report @ezimore . I opened #18294 to resolve it.

@jszwedko jszwedko added the domain: releasing Anything related to releasing Vector label Aug 17, 2023
github-merge-queue bot pushed a commit that referenced this issue Aug 17, 2023
Closes: #18291

Signed-off-by: Jesse Szwedko <jesse.szwedko@datadoghq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain: releasing Anything related to releasing Vector type: bug A code related bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants