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

command -v "$emptyvar" returns zero #1974

Closed
oguz-ismail opened this issue May 18, 2024 · 3 comments
Closed

command -v "$emptyvar" returns zero #1974

oguz-ismail opened this issue May 18, 2024 · 3 comments

Comments

@oguz-ismail
Copy link

osh-0.21.0$ x=
osh-0.21.0$ command -v "$x"

osh-0.21.0$ echo $?
0
osh-0.21.0$

No other shell does this.

andychu pushed a commit that referenced this issue May 18, 2024
Issue #1974

Seems like this was fixed recently.
@andychu
Copy link
Contributor

andychu commented May 18, 2024

Thanks for testing! I reproduced it and it was indeed a bug, though it was fixed several weeks ago as part of this commit:

1bd8841

This will be out with the next release!

@andychu
Copy link
Contributor

andychu commented May 18, 2024

If you like, the CI creates a tarball on every commit, which is easy to download and install

http://travis-ci.oilshell.org/github-jobs/6976/

cpp-tarball -> make-tar -> results

http://travis-ci.oilshell.org/github-jobs/6976/cpp-tarball.wwz/_release/oils-for-unix.tar

@andychu
Copy link
Contributor

andychu commented Jun 20, 2024

@andychu andychu closed this as completed Jun 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants