I'm working on a script using bash to determine the os being used and print to the screen the version.
The only line giving me issues is this one, could you please help me?:
4 Answers
An alternative is to use
lsb_release -d
for getting the distribution name, or
lsb_release -a
to get all info about the distro.
More info about this linux standard base utility.
Comments
You can't have spaces between the two sides of the = sign, and you need to execute the right side as a subcommand (using $() syntax).
current_os=$(cat /etc/*-release | grep CentOS)
With spaces like current_os = ... bash will interpret it as you trying to run a command called current_os.
12 Comments
wkl
@think - you'll need to post the contents of your script for help with that.
Mat
@think: you probably have windows line endings in your script.
dos2unix can fix that.wkl
@think - Mat is likely correct and you may have line-endings that are causing problems. Run
dos2unix os-script.txt and try again. |
source /etc/os-release
current_os=$PRETTY_NAME
or if you don't want to pull everything in
current_os=$(source /etc/os-release && echo $PRETTY_NAME)
Comments
Put the commands in backticks (``) and the statement will assign the output of the command.
current_os=`cat /etc/*-release | grep CentOS`
4 Comments
Mat
Syntax still invalid, and backticks are not recommended,
$() is better
René Pijl
$() will give you exactly the same result in this case, so it's not better, it's just another solution.
Mat
It's better advice in general.
chepner
Backticks are considered obsolete, as they are harder to read and much harder to nest.