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

[jsk_tools] Merge 99.jsk_tools.[bash|zsh] to 99.jsk_tools.sh #870

Merged
merged 1 commit into from
Apr 23, 2015

Conversation

wkentaro
Copy link
Member

@k-okada @garaemon I merged them.

issue: #869

@wkentaro wkentaro force-pushed the merge-env-hooks-shell-sh branch 15 times, most recently from ba49347 to fd5f197 Compare April 21, 2015 21:04
@wkentaro wkentaro force-pushed the merge-env-hooks-shell-sh branch from fd5f197 to 53c58ac Compare April 22, 2015 16:19
@garaemon garaemon closed this Apr 22, 2015
@garaemon garaemon reopened this Apr 22, 2015
@garaemon
Copy link
Member

travis not yet started.. wait a little bit more

garaemon added a commit that referenced this pull request Apr 23, 2015
[jsk_tools] Merge 99.jsk_tools.[bash|zsh] to 99.jsk_tools.sh
@garaemon garaemon merged commit 6dbc518 into jsk-ros-pkg:master Apr 23, 2015
@wkentaro wkentaro deleted the merge-env-hooks-shell-sh branch April 23, 2015 15:16
}

# source 99.jsk_tools.sh from same directory as this file
_THIS_DIR=$(buildin cd -q "`dirname "${BASH_SOURCE[0]}"`" > /dev/null && pwd)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bash does not have neither buildin nor cd -q

k-okada@kokada-t440s:~/ros/indigo/src/jsk-ros-pkg/jsk_common/jsk_tools/env-hooks$ source 99.jsk_tools.bash 
buildin: command not found
bash: /99.jsk_tools.sh: No such file or directory
k-okada@kokada-t440s:~/ros/indigo/src/jsk-ros-pkg/jsk_common/jsk_tools/env-hooks$ emacs -nw  99.jsk_tools.bash 

[3]+  Stopped                 emacs -nw 99.jsk_tools.bash
k-okada@kokada-t440s:~/ros/indigo/src/jsk-ros-pkg/jsk_common/jsk_tools/env-hooks$ source 99.jsk_tools.bash 
bash: cd: -q: invalid option
cd: usage: cd [-L|[-P [-e]] [-@]] [dir]
bash: /99.jsk_tools.sh: No such file or directory

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed in 42f0892

@wkentaro
Copy link
Member Author

sorry, it should be builtin cd without -q.
thanks for fixing it.

@wkentaro
Copy link
Member Author

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.

3 participants