Skip to content
This repository has been archived by the owner on Dec 8, 2023. It is now read-only.

quote $0 occurances in scripts #717

Closed
wants to merge 1 commit into from
Closed

quote $0 occurances in scripts #717

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Jun 8, 2021

quote $0 occurances in scripts to avoid breakage on space characters in directory names

@rgl
Copy link

rgl commented Jun 29, 2021

I think you missed another critical step, everything that uses the quoted $0 must also be quoted. For example, $(dirname "$0")/version should be "$(dirname "$0")/version".

But I have no horse in this race. Using directories with spaces should be banished upfront :-D

@dweomer
Copy link
Contributor

dweomer commented Sep 23, 2021

@netzverweigerer please re-open after making the changes described by @rgl: #717 (comment)

I think you missed another critical step, everything that uses the quoted $0 must also be quoted. For example, $(dirname "$0")/version should be "$(dirname "$0")/version".

But I have no horse in this race. Using directories with spaces should be banished upfront :-D

@dweomer dweomer closed this Sep 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants