Skip to content

need to use $@ for all arguments#15

Open
k-okada wants to merge 25 commits intojbohren-forks:shell-verbsfrom
k-okada:fix_orig_args
Open

need to use $@ for all arguments#15
k-okada wants to merge 25 commits intojbohren-forks:shell-verbsfrom
k-okada:fix_orig_args

Conversation

@k-okada
Copy link
Copy Markdown

@k-okada k-okada commented Jul 16, 2015

I think using ${@[*]} does not work for bash

$ catkin cd
bash: ${@[*]}: bad substitution

jbohren and others added 25 commits April 15, 2015 14:25
specifically when using --jobserver
Use os.environ.get instead of dictionary,
which raises KeyError when SHELL is not set.
Catch KeyError when SHELL is not set
(some restrictive systems default to 600 for new files)
augmented bash completion, fixed installed file permissions
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.

6 participants