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

Update 50_guess_bootloader.sh #1042

Merged
merged 1 commit into from Oct 20, 2016

Conversation

jsmeix
Copy link
Member

@jsmeix jsmeix commented Oct 20, 2016

Added 'tail -n1' to get only the last value if there are more than one
and because getting values from sysconfig files is like sourcing
a shell script it is the last setting that matters, see
c291991#commitcomment-19500870
and
#1038

Cf.
http://stackoverflow.com/questions/2871930/how-to-retrieve-value-from-etc-sysconfig-in-python
that reads (excerpt):
source /etc/sysconfig/FOO
echo $MY_VALUE
and
http://ftp.novell.hu/pub/mirrors/ftp.novell.com/forge/library/SUSE%20Package%20Conventions/spc_sysconfig.html
that reads (excerpts):
The configuration is easily readable by shell scripts
The sysconfig files are plain text files.
The setting is defined by pairs {VARIABLE,value} the following way,
like shell variables:
VARIABLE=”value”

Added 'tail -n1' to get onyl the last value if there are more than one
and because getting values from sysconfig files is like sourcing a shell script
it is the last setting that matters.

Cf.
http://stackoverflow.com/questions/2871930/how-to-retrieve-value-from-etc-sysconfig-in-python
that reads (excerpt):
  source /etc/sysconfig/FOO
  echo $MY_VALUE
and
http://ftp.novell.hu/pub/mirrors/ftp.novell.com/forge/library/SUSE%20Package%20Conventions/spc_sysconfig.html
that reads (excerpts):
  The configuration is easily readable by shell scripts
  The sysconfig files are plain text files.
  The setting is defined by pairs {VARIABLE,value} the following way,
  like shell variables:
  VARIABLE=”value”
@jsmeix jsmeix added enhancement Adaptions and new features bug The code does not do what it is meant to do labels Oct 20, 2016
@jsmeix jsmeix added this to the Rear v1.20 milestone Oct 20, 2016
@jsmeix jsmeix self-assigned this Oct 20, 2016
@jsmeix jsmeix merged commit 62bdb52 into master Oct 20, 2016
@jsmeix jsmeix deleted the jsmeix-only-last-LOADER_TYPE-matters_issue1038 branch October 20, 2016 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug The code does not do what it is meant to do enhancement Adaptions and new features fixed / solved / done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant