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: fix strip program stdout and destination hyphen handling #5718 #5848

Merged

Conversation

cre4ture
Copy link
Contributor

@cre4ture cre4ture commented Jan 16, 2024

adresses issue #5718

there where two issues:

  1. stdout from strip program wasn't forwarded
  2. "./" was applied redundantly (././-dest) as the check was on the filename (without path)

Tests are extented accordingly

@cre4ture cre4ture changed the title install: fix stdout and hyphen handling #5718 install: fix strip program stdout and destination hyphen handling #5718 Jan 16, 2024
@cre4ture cre4ture force-pushed the fix/install_stdout_and_hyphen_handling branch from 78c6445 to 32c1d61 Compare January 16, 2024 22:13
@cakebaker cakebaker linked an issue Jan 17, 2024 that may be closed by this pull request
@cakebaker cakebaker merged commit dc533a9 into uutils:main Jan 17, 2024
60 of 61 checks passed
@cakebaker
Copy link
Contributor

Good job, thanks :)

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.

install: no output when running -s --strip-program
2 participants