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.

Sign up for GitHub

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

Jump to bottom

need to use $@ for all arguments #15

Open
wants to merge 25 commits into
base: shell-verbs
Choose a base branch
from
Open

need to use $@ for all arguments #15

wants to merge 25 commits into from

Conversation

k-okada
Copy link

@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
@jbohren
doc: adding distcc usage example
a90fb3c
@wjwwood
Merge pull request catkin#197 from jbohren-forks/distcc-doc
4e4e312 
doc: adding distcc usage example
@jbohren @wjwwood
verbs: adding find verb to get workspace paths
d964d0f
@wjwwood
catkin find => catkin locate
56a9cab
@wjwwood
touchup catkin locate
700481d
@davetcoleman @wjwwood
Add documentation about warnings
13c7898
@wjwwood
touchup doc
8b41301
@wjwwood
working on shell completion still
cca632b
@jbohren @wjwwood
completion: adding bash/zsh completion for most verbs
ddcc2bd
@wjwwood
improve /etc/* asset installation
f797eb6
@wjwwood
add check to prevent dangling jobs after ctrl-c
f3dc7ab 
specifically when using --jobserver
@wjwwood
0.3.0
1cacc06
@wjwwood
adding MANIFEST.in file
219cc99
@scpeters
Don't raise when SHELL is not set
e945e48 
Use os.environ.get instead of dictionary,
which raises KeyError when SHELL is not set.
@wjwwood
Merge pull request catkin#204 from scpeters/issue_203
c5e597c 
Catch KeyError when SHELL is not set
@rhaschke
added --whitelist and --blacklist to bash completion
55fcfaf
@rhaschke
install with read permissions for everybody
3337267 
(some restrictive systems default to 600 for new files)
@rhaschke
fixed octal permission values
562f11e
@rhaschke
added --no-whitelist and --no-blacklist to bash completion
47b3e65
@wjwwood
Merge pull request catkin#206 from rhaschke/master
1c9609a 
augmented bash completion, fixed installed file permissions
@wjwwood
ordering imports
fc660e6
@wjwwood
fix key error on executor shutdown
3848f96
@wjwwood
Fix logic error for enable/disable color output
9568fa8
@k-okada
Merge remote-tracking branch 'remotes/jbohren-forks/shell-verbs' into…
ecdb0d4 
… fix_orig_args
@k-okada
need to use $@ for all arguments
bb67808
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@k-okada @jbohren @wjwwood @davetcoleman @scpeters @rhaschke