Change in vdsm[master]: vdsm.spec.in: use locale LC_ALL

danken at redhat.com danken at redhat.com
Thu Sep 20 06:48:01 UTC 2012


Dan Kenigsberg has posted comments on this change.

Change subject: vdsm.spec.in: use locale LC_ALL
......................................................................


Patch Set 1: I would prefer that you didn't submit this

(2 inline comments)

textual comments only

....................................................
Commit Message
Line 3: AuthorDate: 2012-09-19 14:23:12 -0400
Line 4: Commit:     Douglas Schilling Landgraf <dougsland at redhat.com>
Line 5: CommitDate: 2012-09-19 14:23:12 -0400
Line 6: 
Line 7: vdsm.spec.in: use locale LC_ALL
LC_ALL=C
Line 8: 
Line 9: During VDSM instalation, we verify if SELinux booleans are
Line 10: enabled with return of semanage command, output like: "on" or "off".
Line 11: These are English words and if the system contains a different


....................................................
File vdsm.spec.in
Line 429: virt_use_sanlock=1
Line 430: _EOF
Line 431: fi
Line 432: 
Line 433: # Change the current state of SELinux
we do not change selinux state, only set a couple of selinux booleans.
Line 434: if /usr/sbin/selinuxenabled; then
Line 435:     /usr/sbin/setsebool virt_use_nfs on
Line 436:     /usr/sbin/setsebool virt_use_sanlock on
Line 437: fi


--
To view, visit http://gerrit.ovirt.org/8068
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I22b5706e02c71f463200ce20eb0e426e1bfee6e6
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw at linux.vnet.ibm.com>


More information about the vdsm-patches mailing list