Why doesn't using multiple commands with a || or && conditional work?How can I test if a variable is empty or contains only spaces?Using sshpass, return code (exit status) differs for reasons unknown using valid commandsScript in C shell skipping over last else statement?Modify bash arguments if only one argument is setFunction that calls another function with list of arguments doesn't workC Shell Script syntax error “unexpected end of of file”“dash” arguments to shell scriptsHow to use as argument the characters + , - , x , / in a scriptCreate a menu driven script with do loop and case statement in bashA star is being printed for an empty directory after running a script to list the subfolderecho $HISTSIZE not printing when executed via shell script but works in command line
Why is it a bad idea to hire a hitman to eliminate most corrupt politicians?
What about the virus in 12 Monkeys?
Mathematica command that allows it to read my intentions
Assassin's bullet with mercury
What killed these X2 caps?
In 'Revenger,' what does 'cove' come from?
What mechanic is there to disable a threat instead of killing it?
Why was the shrinking from 8″ made only to 5.25″ and not smaller (4″ or less)?
Forgetting the musical notes while performing in concert
Why didn't Miles's spider sense work before?
How much of data wrangling is a data scientist's job?
Should I cover my bicycle overnight while bikepacking?
What does the expression "A Mann!" means
Can compressed videos be decoded back to their uncompresed original format?
Can we compute the area of a quadrilateral with one right angle when we only know the lengths of any three sides?
Why didn't Boeing produce its own regional jet?
How to compactly explain secondary and tertiary characters without resorting to stereotypes?
CAST throwing error when run in stored procedure but not when run as raw query
GFCI outlets - can they be repaired? Are they really needed at the end of a circuit?
Venezuelan girlfriend wants to travel the USA to be with me. What is the process?
How do I gain back my faith in my PhD degree?
How do I know where to place holes on an instrument?
What does “the session was packed” mean in this context?
What reasons are there for a Capitalist to oppose a 100% inheritance tax?
Why doesn't using multiple commands with a || or && conditional work?
How can I test if a variable is empty or contains only spaces?Using sshpass, return code (exit status) differs for reasons unknown using valid commandsScript in C shell skipping over last else statement?Modify bash arguments if only one argument is setFunction that calls another function with list of arguments doesn't workC Shell Script syntax error “unexpected end of of file”“dash” arguments to shell scriptsHow to use as argument the characters + , - , x , / in a scriptCreate a menu driven script with do loop and case statement in bashA star is being printed for an empty directory after running a script to list the subfolderecho $HISTSIZE not printing when executed via shell script but works in command line
This works on a shell (bash, dash) prompt:
[ -z "" ] && echo A || echo B
A
However, I am trying to write a POSIX shell script, it starts like this:
#!/bin/sh
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."; exit 1
readonly raw_input_string=$1
[ -z "$raw_input_string" ] && echo "The given argument is empty."; exit 1
And I don't know why, but I don't get the message:
The given argument is empty.
if I call the script like this:
./test_empty_argument ""
Why is that?
shell-script arguments
|
show 3 more comments
This works on a shell (bash, dash) prompt:
[ -z "" ] && echo A || echo B
A
However, I am trying to write a POSIX shell script, it starts like this:
#!/bin/sh
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."; exit 1
readonly raw_input_string=$1
[ -z "$raw_input_string" ] && echo "The given argument is empty."; exit 1
And I don't know why, but I don't get the message:
The given argument is empty.
if I call the script like this:
./test_empty_argument ""
Why is that?
shell-script arguments
3
See How can I test if a variable is empty or contains only spaces? for ways on testing if a variable is empty, unset, or only contains blanks. The issue in this question has nothing to do with that.
– ilkkachu
12 hours ago
Just useif [ X”” = X”$var” ] ; then echo isempty ; fi
– user2497
12 hours ago
2
@user2497 There is no reason to use that in any shell released in the last 20 years. That's a workaround for old, buggy shells.
– chepner
11 hours ago
@chepner So it is not a valid solution? Something else must be used?
– user2497
11 hours ago
3
[ "" = "$var" ]
would work fine; a quoted empty string will not be removed from the argument list of[
. But that's not necessary either, because[ -z "$var" ]
also works just fine.
– chepner
10 hours ago
|
show 3 more comments
This works on a shell (bash, dash) prompt:
[ -z "" ] && echo A || echo B
A
However, I am trying to write a POSIX shell script, it starts like this:
#!/bin/sh
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."; exit 1
readonly raw_input_string=$1
[ -z "$raw_input_string" ] && echo "The given argument is empty."; exit 1
And I don't know why, but I don't get the message:
The given argument is empty.
if I call the script like this:
./test_empty_argument ""
Why is that?
shell-script arguments
This works on a shell (bash, dash) prompt:
[ -z "" ] && echo A || echo B
A
However, I am trying to write a POSIX shell script, it starts like this:
#!/bin/sh
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."; exit 1
readonly raw_input_string=$1
[ -z "$raw_input_string" ] && echo "The given argument is empty."; exit 1
And I don't know why, but I don't get the message:
The given argument is empty.
if I call the script like this:
./test_empty_argument ""
Why is that?
shell-script arguments
shell-script arguments
edited 8 hours ago
ilkkachu
63k10103180
63k10103180
asked 22 hours ago
VlastimilVlastimil
8,4451565145
8,4451565145
3
See How can I test if a variable is empty or contains only spaces? for ways on testing if a variable is empty, unset, or only contains blanks. The issue in this question has nothing to do with that.
– ilkkachu
12 hours ago
Just useif [ X”” = X”$var” ] ; then echo isempty ; fi
– user2497
12 hours ago
2
@user2497 There is no reason to use that in any shell released in the last 20 years. That's a workaround for old, buggy shells.
– chepner
11 hours ago
@chepner So it is not a valid solution? Something else must be used?
– user2497
11 hours ago
3
[ "" = "$var" ]
would work fine; a quoted empty string will not be removed from the argument list of[
. But that's not necessary either, because[ -z "$var" ]
also works just fine.
– chepner
10 hours ago
|
show 3 more comments
3
See How can I test if a variable is empty or contains only spaces? for ways on testing if a variable is empty, unset, or only contains blanks. The issue in this question has nothing to do with that.
– ilkkachu
12 hours ago
Just useif [ X”” = X”$var” ] ; then echo isempty ; fi
– user2497
12 hours ago
2
@user2497 There is no reason to use that in any shell released in the last 20 years. That's a workaround for old, buggy shells.
– chepner
11 hours ago
@chepner So it is not a valid solution? Something else must be used?
– user2497
11 hours ago
3
[ "" = "$var" ]
would work fine; a quoted empty string will not be removed from the argument list of[
. But that's not necessary either, because[ -z "$var" ]
also works just fine.
– chepner
10 hours ago
3
3
See How can I test if a variable is empty or contains only spaces? for ways on testing if a variable is empty, unset, or only contains blanks. The issue in this question has nothing to do with that.
– ilkkachu
12 hours ago
See How can I test if a variable is empty or contains only spaces? for ways on testing if a variable is empty, unset, or only contains blanks. The issue in this question has nothing to do with that.
– ilkkachu
12 hours ago
Just use
if [ X”” = X”$var” ] ; then echo isempty ; fi
– user2497
12 hours ago
Just use
if [ X”” = X”$var” ] ; then echo isempty ; fi
– user2497
12 hours ago
2
2
@user2497 There is no reason to use that in any shell released in the last 20 years. That's a workaround for old, buggy shells.
– chepner
11 hours ago
@user2497 There is no reason to use that in any shell released in the last 20 years. That's a workaround for old, buggy shells.
– chepner
11 hours ago
@chepner So it is not a valid solution? Something else must be used?
– user2497
11 hours ago
@chepner So it is not a valid solution? Something else must be used?
– user2497
11 hours ago
3
3
[ "" = "$var" ]
would work fine; a quoted empty string will not be removed from the argument list of [
. But that's not necessary either, because [ -z "$var" ]
also works just fine.– chepner
10 hours ago
[ "" = "$var" ]
would work fine; a quoted empty string will not be removed from the argument list of [
. But that's not necessary either, because [ -z "$var" ]
also works just fine.– chepner
10 hours ago
|
show 3 more comments
5 Answers
5
active
oldest
votes
Note that your line
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."; exit 1
this is the same as
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."
exit 1
(a ;
can, in most circumstances, be replaced by a newline character)
Which means that the exit 1
statement is always executed regardless of how many arguments were passed to the script. This means that the message The given argument is empty.
would never have a chance of getting printed.
To execute more than a single statement after a test using the "short-circuit syntax", group the statements in ...;
. The alternative is to use a proper if
test (which, IMHO, looks cleaner in a script):
if [ "$#" -ne 1 ]; then
echo 'Invalid number of arguments, expected one.' >&2
exit 1
fi
You have the same issue with your second test.
Regarding
[ -z "" ] && echo A || echo B
This would work for the given example, but the generic
some-test && command1 || command2
would not be the same as
if some-test; then
command1
else
command2
fi
Instead, it is more like
if ! some-test && command1; ; then
command2
fi
That is, if either the test or the first command fails, the second command executes, which means it has the potential to execute all three involved statements.
add a comment |
This:
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."; exit 1
is not:
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."; exit 1;
But instead is:
exit 1
Your script is exiting regardless of how many arguments you passed to it.
add a comment |
One way to make it more readable is to define a die
function (à la perl
) like:
die()
printf >&2 '%sn' "$@"
exit 1
# then:
[ "$#" -eq 1 ] || die "Expected one argument, got $#"
[ -n "$1" ] || die "Empty argument not supported"
You can add more bells and whistles like colours, prefix... if need be.
In practice, do you ever call yourdie
function with multiple arguments? (If so, can you give an example?) I use an almost identicaldie
function, but use"$*"
instead, which may be more what you're intending?
– jrw32982
11 hours ago
The value of"$@"
is that it allows multi-line messages without needing to add literal newlines.
– Charles Duffy
8 hours ago
add a comment |
I've often seen this as a test for an empty string:
if [ "x$foo" = "x" ]; then ...
Should have been "=" - fixed.
– wef
22 hours ago
2
That practice is literally from the 1970s. There is no reason whatsoever to use it with any shell that is compliant with the 1992 POSIX sh standard (so long as correct quoting is used and now-obsolescent functionality such as-a
,-o
,(
and)
as derectives to telltest
to combine multiple operations in a single invocation are avoided; see the OB markers in pubs.opengroup.org/onlinepubs/9699919799/utilities/test.html).
– Charles Duffy
8 hours ago
Non-linux unices shipped with the original 'sh' well into the '90's - maybe still do. In my job at that time, I had to write portable install scripts and I used this construct. I just looked at the install scripts that NVidia ships for linux and they still use this construct.
– wef
7 hours ago
NVidia may use it, but that's not to say they have any technical justification to do so; cargo-cult development in commercial UNIX is sadly prevalent. Even Heirloom Bourne doesn't have the bug in question -- so the SunOS codebase (that being the last commercial UNIX to ship a non-POSIX/bin/sh
, and Heirloom Bourne's immediate predecessor) didn't have it either.
– Charles Duffy
6 hours ago
1
I don't wear a hat, so I can't promise to post a YouTube video eating it should someone turn up a shell published on a commercial UNIX with this bug post-1990... but if I did, it would be tempting. :)
– Charles Duffy
6 hours ago
add a comment |
I would also add to the other answers that a POSIX shell may detect variable assignment errors.
#!/bin/sh -u
foo=$1:? # choose a meaninful variable name and possibly use an error message.
# ... if the first parameter is unset or empty the shell exits.
bar=$2- # if the second parameter is set, then the shell assigns its value, else
# it assigns the empty string.
# displays the date if "var" is empty, else, prints the working directory.
[ -z "$var-" ] && date || pwd
# "var" is set to the empty string. I assume the nineth parameter is not set.
# `echo "$9"' returns 1 so the shell also runs `echo "var not empty: $var"'
[ -z "$var-" ] && echo "$9" || echo "var not empty: $var"
# `echo "var empty"' returns 0
[ -z "$var-" ] && echo "var empty" || echo "var not empty: $var"
var=0101
[ -z "$var-" ] && echo "var empty" || echo "var not empty: $var"
The shell exits if any variable is unset while the shell option -u
is enabled. The shell expansions used in the sample script are also defined by POSIX.
add a comment |
Your Answer
StackExchange.ready(function()
var channelOptions =
tags: "".split(" "),
id: "106"
;
initTagRenderer("".split(" "), "".split(" "), channelOptions);
StackExchange.using("externalEditor", function()
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled)
StackExchange.using("snippets", function()
createEditor();
);
else
createEditor();
);
function createEditor()
StackExchange.prepareEditor(
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: false,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: null,
bindNavPrevention: true,
postfix: "",
imageUploader:
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
,
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
);
);
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2funix.stackexchange.com%2fquestions%2f510216%2fwhy-doesnt-using-multiple-commands-with-a-or-conditional-work%23new-answer', 'question_page');
);
Post as a guest
Required, but never shown
5 Answers
5
active
oldest
votes
5 Answers
5
active
oldest
votes
active
oldest
votes
active
oldest
votes
Note that your line
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."; exit 1
this is the same as
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."
exit 1
(a ;
can, in most circumstances, be replaced by a newline character)
Which means that the exit 1
statement is always executed regardless of how many arguments were passed to the script. This means that the message The given argument is empty.
would never have a chance of getting printed.
To execute more than a single statement after a test using the "short-circuit syntax", group the statements in ...;
. The alternative is to use a proper if
test (which, IMHO, looks cleaner in a script):
if [ "$#" -ne 1 ]; then
echo 'Invalid number of arguments, expected one.' >&2
exit 1
fi
You have the same issue with your second test.
Regarding
[ -z "" ] && echo A || echo B
This would work for the given example, but the generic
some-test && command1 || command2
would not be the same as
if some-test; then
command1
else
command2
fi
Instead, it is more like
if ! some-test && command1; ; then
command2
fi
That is, if either the test or the first command fails, the second command executes, which means it has the potential to execute all three involved statements.
add a comment |
Note that your line
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."; exit 1
this is the same as
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."
exit 1
(a ;
can, in most circumstances, be replaced by a newline character)
Which means that the exit 1
statement is always executed regardless of how many arguments were passed to the script. This means that the message The given argument is empty.
would never have a chance of getting printed.
To execute more than a single statement after a test using the "short-circuit syntax", group the statements in ...;
. The alternative is to use a proper if
test (which, IMHO, looks cleaner in a script):
if [ "$#" -ne 1 ]; then
echo 'Invalid number of arguments, expected one.' >&2
exit 1
fi
You have the same issue with your second test.
Regarding
[ -z "" ] && echo A || echo B
This would work for the given example, but the generic
some-test && command1 || command2
would not be the same as
if some-test; then
command1
else
command2
fi
Instead, it is more like
if ! some-test && command1; ; then
command2
fi
That is, if either the test or the first command fails, the second command executes, which means it has the potential to execute all three involved statements.
add a comment |
Note that your line
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."; exit 1
this is the same as
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."
exit 1
(a ;
can, in most circumstances, be replaced by a newline character)
Which means that the exit 1
statement is always executed regardless of how many arguments were passed to the script. This means that the message The given argument is empty.
would never have a chance of getting printed.
To execute more than a single statement after a test using the "short-circuit syntax", group the statements in ...;
. The alternative is to use a proper if
test (which, IMHO, looks cleaner in a script):
if [ "$#" -ne 1 ]; then
echo 'Invalid number of arguments, expected one.' >&2
exit 1
fi
You have the same issue with your second test.
Regarding
[ -z "" ] && echo A || echo B
This would work for the given example, but the generic
some-test && command1 || command2
would not be the same as
if some-test; then
command1
else
command2
fi
Instead, it is more like
if ! some-test && command1; ; then
command2
fi
That is, if either the test or the first command fails, the second command executes, which means it has the potential to execute all three involved statements.
Note that your line
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."; exit 1
this is the same as
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."
exit 1
(a ;
can, in most circumstances, be replaced by a newline character)
Which means that the exit 1
statement is always executed regardless of how many arguments were passed to the script. This means that the message The given argument is empty.
would never have a chance of getting printed.
To execute more than a single statement after a test using the "short-circuit syntax", group the statements in ...;
. The alternative is to use a proper if
test (which, IMHO, looks cleaner in a script):
if [ "$#" -ne 1 ]; then
echo 'Invalid number of arguments, expected one.' >&2
exit 1
fi
You have the same issue with your second test.
Regarding
[ -z "" ] && echo A || echo B
This would work for the given example, but the generic
some-test && command1 || command2
would not be the same as
if some-test; then
command1
else
command2
fi
Instead, it is more like
if ! some-test && command1; ; then
command2
fi
That is, if either the test or the first command fails, the second command executes, which means it has the potential to execute all three involved statements.
edited 8 hours ago
answered 22 hours ago
Kusalananda♦Kusalananda
139k17260432
139k17260432
add a comment |
add a comment |
This:
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."; exit 1
is not:
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."; exit 1;
But instead is:
exit 1
Your script is exiting regardless of how many arguments you passed to it.
add a comment |
This:
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."; exit 1
is not:
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."; exit 1;
But instead is:
exit 1
Your script is exiting regardless of how many arguments you passed to it.
add a comment |
This:
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."; exit 1
is not:
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."; exit 1;
But instead is:
exit 1
Your script is exiting regardless of how many arguments you passed to it.
This:
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."; exit 1
is not:
[ "$#" -eq 1 ] || echo "Invalid number of arguments, expected one."; exit 1;
But instead is:
exit 1
Your script is exiting regardless of how many arguments you passed to it.
answered 22 hours ago
murumuru
37k589164
37k589164
add a comment |
add a comment |
One way to make it more readable is to define a die
function (à la perl
) like:
die()
printf >&2 '%sn' "$@"
exit 1
# then:
[ "$#" -eq 1 ] || die "Expected one argument, got $#"
[ -n "$1" ] || die "Empty argument not supported"
You can add more bells and whistles like colours, prefix... if need be.
In practice, do you ever call yourdie
function with multiple arguments? (If so, can you give an example?) I use an almost identicaldie
function, but use"$*"
instead, which may be more what you're intending?
– jrw32982
11 hours ago
The value of"$@"
is that it allows multi-line messages without needing to add literal newlines.
– Charles Duffy
8 hours ago
add a comment |
One way to make it more readable is to define a die
function (à la perl
) like:
die()
printf >&2 '%sn' "$@"
exit 1
# then:
[ "$#" -eq 1 ] || die "Expected one argument, got $#"
[ -n "$1" ] || die "Empty argument not supported"
You can add more bells and whistles like colours, prefix... if need be.
In practice, do you ever call yourdie
function with multiple arguments? (If so, can you give an example?) I use an almost identicaldie
function, but use"$*"
instead, which may be more what you're intending?
– jrw32982
11 hours ago
The value of"$@"
is that it allows multi-line messages without needing to add literal newlines.
– Charles Duffy
8 hours ago
add a comment |
One way to make it more readable is to define a die
function (à la perl
) like:
die()
printf >&2 '%sn' "$@"
exit 1
# then:
[ "$#" -eq 1 ] || die "Expected one argument, got $#"
[ -n "$1" ] || die "Empty argument not supported"
You can add more bells and whistles like colours, prefix... if need be.
One way to make it more readable is to define a die
function (à la perl
) like:
die()
printf >&2 '%sn' "$@"
exit 1
# then:
[ "$#" -eq 1 ] || die "Expected one argument, got $#"
[ -n "$1" ] || die "Empty argument not supported"
You can add more bells and whistles like colours, prefix... if need be.
answered 12 hours ago
Stéphane ChazelasStéphane Chazelas
313k57592948
313k57592948
In practice, do you ever call yourdie
function with multiple arguments? (If so, can you give an example?) I use an almost identicaldie
function, but use"$*"
instead, which may be more what you're intending?
– jrw32982
11 hours ago
The value of"$@"
is that it allows multi-line messages without needing to add literal newlines.
– Charles Duffy
8 hours ago
add a comment |
In practice, do you ever call yourdie
function with multiple arguments? (If so, can you give an example?) I use an almost identicaldie
function, but use"$*"
instead, which may be more what you're intending?
– jrw32982
11 hours ago
The value of"$@"
is that it allows multi-line messages without needing to add literal newlines.
– Charles Duffy
8 hours ago
In practice, do you ever call your
die
function with multiple arguments? (If so, can you give an example?) I use an almost identical die
function, but use "$*"
instead, which may be more what you're intending?– jrw32982
11 hours ago
In practice, do you ever call your
die
function with multiple arguments? (If so, can you give an example?) I use an almost identical die
function, but use "$*"
instead, which may be more what you're intending?– jrw32982
11 hours ago
The value of
"$@"
is that it allows multi-line messages without needing to add literal newlines.– Charles Duffy
8 hours ago
The value of
"$@"
is that it allows multi-line messages without needing to add literal newlines.– Charles Duffy
8 hours ago
add a comment |
I've often seen this as a test for an empty string:
if [ "x$foo" = "x" ]; then ...
Should have been "=" - fixed.
– wef
22 hours ago
2
That practice is literally from the 1970s. There is no reason whatsoever to use it with any shell that is compliant with the 1992 POSIX sh standard (so long as correct quoting is used and now-obsolescent functionality such as-a
,-o
,(
and)
as derectives to telltest
to combine multiple operations in a single invocation are avoided; see the OB markers in pubs.opengroup.org/onlinepubs/9699919799/utilities/test.html).
– Charles Duffy
8 hours ago
Non-linux unices shipped with the original 'sh' well into the '90's - maybe still do. In my job at that time, I had to write portable install scripts and I used this construct. I just looked at the install scripts that NVidia ships for linux and they still use this construct.
– wef
7 hours ago
NVidia may use it, but that's not to say they have any technical justification to do so; cargo-cult development in commercial UNIX is sadly prevalent. Even Heirloom Bourne doesn't have the bug in question -- so the SunOS codebase (that being the last commercial UNIX to ship a non-POSIX/bin/sh
, and Heirloom Bourne's immediate predecessor) didn't have it either.
– Charles Duffy
6 hours ago
1
I don't wear a hat, so I can't promise to post a YouTube video eating it should someone turn up a shell published on a commercial UNIX with this bug post-1990... but if I did, it would be tempting. :)
– Charles Duffy
6 hours ago
add a comment |
I've often seen this as a test for an empty string:
if [ "x$foo" = "x" ]; then ...
Should have been "=" - fixed.
– wef
22 hours ago
2
That practice is literally from the 1970s. There is no reason whatsoever to use it with any shell that is compliant with the 1992 POSIX sh standard (so long as correct quoting is used and now-obsolescent functionality such as-a
,-o
,(
and)
as derectives to telltest
to combine multiple operations in a single invocation are avoided; see the OB markers in pubs.opengroup.org/onlinepubs/9699919799/utilities/test.html).
– Charles Duffy
8 hours ago
Non-linux unices shipped with the original 'sh' well into the '90's - maybe still do. In my job at that time, I had to write portable install scripts and I used this construct. I just looked at the install scripts that NVidia ships for linux and they still use this construct.
– wef
7 hours ago
NVidia may use it, but that's not to say they have any technical justification to do so; cargo-cult development in commercial UNIX is sadly prevalent. Even Heirloom Bourne doesn't have the bug in question -- so the SunOS codebase (that being the last commercial UNIX to ship a non-POSIX/bin/sh
, and Heirloom Bourne's immediate predecessor) didn't have it either.
– Charles Duffy
6 hours ago
1
I don't wear a hat, so I can't promise to post a YouTube video eating it should someone turn up a shell published on a commercial UNIX with this bug post-1990... but if I did, it would be tempting. :)
– Charles Duffy
6 hours ago
add a comment |
I've often seen this as a test for an empty string:
if [ "x$foo" = "x" ]; then ...
I've often seen this as a test for an empty string:
if [ "x$foo" = "x" ]; then ...
edited 22 hours ago
answered 22 hours ago
wefwef
31415
31415
Should have been "=" - fixed.
– wef
22 hours ago
2
That practice is literally from the 1970s. There is no reason whatsoever to use it with any shell that is compliant with the 1992 POSIX sh standard (so long as correct quoting is used and now-obsolescent functionality such as-a
,-o
,(
and)
as derectives to telltest
to combine multiple operations in a single invocation are avoided; see the OB markers in pubs.opengroup.org/onlinepubs/9699919799/utilities/test.html).
– Charles Duffy
8 hours ago
Non-linux unices shipped with the original 'sh' well into the '90's - maybe still do. In my job at that time, I had to write portable install scripts and I used this construct. I just looked at the install scripts that NVidia ships for linux and they still use this construct.
– wef
7 hours ago
NVidia may use it, but that's not to say they have any technical justification to do so; cargo-cult development in commercial UNIX is sadly prevalent. Even Heirloom Bourne doesn't have the bug in question -- so the SunOS codebase (that being the last commercial UNIX to ship a non-POSIX/bin/sh
, and Heirloom Bourne's immediate predecessor) didn't have it either.
– Charles Duffy
6 hours ago
1
I don't wear a hat, so I can't promise to post a YouTube video eating it should someone turn up a shell published on a commercial UNIX with this bug post-1990... but if I did, it would be tempting. :)
– Charles Duffy
6 hours ago
add a comment |
Should have been "=" - fixed.
– wef
22 hours ago
2
That practice is literally from the 1970s. There is no reason whatsoever to use it with any shell that is compliant with the 1992 POSIX sh standard (so long as correct quoting is used and now-obsolescent functionality such as-a
,-o
,(
and)
as derectives to telltest
to combine multiple operations in a single invocation are avoided; see the OB markers in pubs.opengroup.org/onlinepubs/9699919799/utilities/test.html).
– Charles Duffy
8 hours ago
Non-linux unices shipped with the original 'sh' well into the '90's - maybe still do. In my job at that time, I had to write portable install scripts and I used this construct. I just looked at the install scripts that NVidia ships for linux and they still use this construct.
– wef
7 hours ago
NVidia may use it, but that's not to say they have any technical justification to do so; cargo-cult development in commercial UNIX is sadly prevalent. Even Heirloom Bourne doesn't have the bug in question -- so the SunOS codebase (that being the last commercial UNIX to ship a non-POSIX/bin/sh
, and Heirloom Bourne's immediate predecessor) didn't have it either.
– Charles Duffy
6 hours ago
1
I don't wear a hat, so I can't promise to post a YouTube video eating it should someone turn up a shell published on a commercial UNIX with this bug post-1990... but if I did, it would be tempting. :)
– Charles Duffy
6 hours ago
Should have been "=" - fixed.
– wef
22 hours ago
Should have been "=" - fixed.
– wef
22 hours ago
2
2
That practice is literally from the 1970s. There is no reason whatsoever to use it with any shell that is compliant with the 1992 POSIX sh standard (so long as correct quoting is used and now-obsolescent functionality such as
-a
, -o
, (
and )
as derectives to tell test
to combine multiple operations in a single invocation are avoided; see the OB markers in pubs.opengroup.org/onlinepubs/9699919799/utilities/test.html).– Charles Duffy
8 hours ago
That practice is literally from the 1970s. There is no reason whatsoever to use it with any shell that is compliant with the 1992 POSIX sh standard (so long as correct quoting is used and now-obsolescent functionality such as
-a
, -o
, (
and )
as derectives to tell test
to combine multiple operations in a single invocation are avoided; see the OB markers in pubs.opengroup.org/onlinepubs/9699919799/utilities/test.html).– Charles Duffy
8 hours ago
Non-linux unices shipped with the original 'sh' well into the '90's - maybe still do. In my job at that time, I had to write portable install scripts and I used this construct. I just looked at the install scripts that NVidia ships for linux and they still use this construct.
– wef
7 hours ago
Non-linux unices shipped with the original 'sh' well into the '90's - maybe still do. In my job at that time, I had to write portable install scripts and I used this construct. I just looked at the install scripts that NVidia ships for linux and they still use this construct.
– wef
7 hours ago
NVidia may use it, but that's not to say they have any technical justification to do so; cargo-cult development in commercial UNIX is sadly prevalent. Even Heirloom Bourne doesn't have the bug in question -- so the SunOS codebase (that being the last commercial UNIX to ship a non-POSIX
/bin/sh
, and Heirloom Bourne's immediate predecessor) didn't have it either.– Charles Duffy
6 hours ago
NVidia may use it, but that's not to say they have any technical justification to do so; cargo-cult development in commercial UNIX is sadly prevalent. Even Heirloom Bourne doesn't have the bug in question -- so the SunOS codebase (that being the last commercial UNIX to ship a non-POSIX
/bin/sh
, and Heirloom Bourne's immediate predecessor) didn't have it either.– Charles Duffy
6 hours ago
1
1
I don't wear a hat, so I can't promise to post a YouTube video eating it should someone turn up a shell published on a commercial UNIX with this bug post-1990... but if I did, it would be tempting. :)
– Charles Duffy
6 hours ago
I don't wear a hat, so I can't promise to post a YouTube video eating it should someone turn up a shell published on a commercial UNIX with this bug post-1990... but if I did, it would be tempting. :)
– Charles Duffy
6 hours ago
add a comment |
I would also add to the other answers that a POSIX shell may detect variable assignment errors.
#!/bin/sh -u
foo=$1:? # choose a meaninful variable name and possibly use an error message.
# ... if the first parameter is unset or empty the shell exits.
bar=$2- # if the second parameter is set, then the shell assigns its value, else
# it assigns the empty string.
# displays the date if "var" is empty, else, prints the working directory.
[ -z "$var-" ] && date || pwd
# "var" is set to the empty string. I assume the nineth parameter is not set.
# `echo "$9"' returns 1 so the shell also runs `echo "var not empty: $var"'
[ -z "$var-" ] && echo "$9" || echo "var not empty: $var"
# `echo "var empty"' returns 0
[ -z "$var-" ] && echo "var empty" || echo "var not empty: $var"
var=0101
[ -z "$var-" ] && echo "var empty" || echo "var not empty: $var"
The shell exits if any variable is unset while the shell option -u
is enabled. The shell expansions used in the sample script are also defined by POSIX.
add a comment |
I would also add to the other answers that a POSIX shell may detect variable assignment errors.
#!/bin/sh -u
foo=$1:? # choose a meaninful variable name and possibly use an error message.
# ... if the first parameter is unset or empty the shell exits.
bar=$2- # if the second parameter is set, then the shell assigns its value, else
# it assigns the empty string.
# displays the date if "var" is empty, else, prints the working directory.
[ -z "$var-" ] && date || pwd
# "var" is set to the empty string. I assume the nineth parameter is not set.
# `echo "$9"' returns 1 so the shell also runs `echo "var not empty: $var"'
[ -z "$var-" ] && echo "$9" || echo "var not empty: $var"
# `echo "var empty"' returns 0
[ -z "$var-" ] && echo "var empty" || echo "var not empty: $var"
var=0101
[ -z "$var-" ] && echo "var empty" || echo "var not empty: $var"
The shell exits if any variable is unset while the shell option -u
is enabled. The shell expansions used in the sample script are also defined by POSIX.
add a comment |
I would also add to the other answers that a POSIX shell may detect variable assignment errors.
#!/bin/sh -u
foo=$1:? # choose a meaninful variable name and possibly use an error message.
# ... if the first parameter is unset or empty the shell exits.
bar=$2- # if the second parameter is set, then the shell assigns its value, else
# it assigns the empty string.
# displays the date if "var" is empty, else, prints the working directory.
[ -z "$var-" ] && date || pwd
# "var" is set to the empty string. I assume the nineth parameter is not set.
# `echo "$9"' returns 1 so the shell also runs `echo "var not empty: $var"'
[ -z "$var-" ] && echo "$9" || echo "var not empty: $var"
# `echo "var empty"' returns 0
[ -z "$var-" ] && echo "var empty" || echo "var not empty: $var"
var=0101
[ -z "$var-" ] && echo "var empty" || echo "var not empty: $var"
The shell exits if any variable is unset while the shell option -u
is enabled. The shell expansions used in the sample script are also defined by POSIX.
I would also add to the other answers that a POSIX shell may detect variable assignment errors.
#!/bin/sh -u
foo=$1:? # choose a meaninful variable name and possibly use an error message.
# ... if the first parameter is unset or empty the shell exits.
bar=$2- # if the second parameter is set, then the shell assigns its value, else
# it assigns the empty string.
# displays the date if "var" is empty, else, prints the working directory.
[ -z "$var-" ] && date || pwd
# "var" is set to the empty string. I assume the nineth parameter is not set.
# `echo "$9"' returns 1 so the shell also runs `echo "var not empty: $var"'
[ -z "$var-" ] && echo "$9" || echo "var not empty: $var"
# `echo "var empty"' returns 0
[ -z "$var-" ] && echo "var empty" || echo "var not empty: $var"
var=0101
[ -z "$var-" ] && echo "var empty" || echo "var not empty: $var"
The shell exits if any variable is unset while the shell option -u
is enabled. The shell expansions used in the sample script are also defined by POSIX.
edited 12 hours ago
answered 12 hours ago
FólkvangrFólkvangr
32913
32913
add a comment |
add a comment |
Thanks for contributing an answer to Unix & Linux Stack Exchange!
- Please be sure to answer the question. Provide details and share your research!
But avoid …
- Asking for help, clarification, or responding to other answers.
- Making statements based on opinion; back them up with references or personal experience.
To learn more, see our tips on writing great answers.
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2funix.stackexchange.com%2fquestions%2f510216%2fwhy-doesnt-using-multiple-commands-with-a-or-conditional-work%23new-answer', 'question_page');
);
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
3
See How can I test if a variable is empty or contains only spaces? for ways on testing if a variable is empty, unset, or only contains blanks. The issue in this question has nothing to do with that.
– ilkkachu
12 hours ago
Just use
if [ X”” = X”$var” ] ; then echo isempty ; fi
– user2497
12 hours ago
2
@user2497 There is no reason to use that in any shell released in the last 20 years. That's a workaround for old, buggy shells.
– chepner
11 hours ago
@chepner So it is not a valid solution? Something else must be used?
– user2497
11 hours ago
3
[ "" = "$var" ]
would work fine; a quoted empty string will not be removed from the argument list of[
. But that's not necessary either, because[ -z "$var" ]
also works just fine.– chepner
10 hours ago