[PATCH] firstboot-tweak.patch

[Essien Essien]

Hiya,

The attached patch checks for firstboot and if its _not_ installed disables
it from running, hence eliminating the [FAIL] boottime message that will
occur otherwise.

Cheers,
Essien

[Essien Essien]

On 2/24/06, Essien Essien <essiene(a)gmail.com&gt; wrote:
>  Hiya,
>
>  The attached patch checks for firstboot and if its _not_ installed
>  disables it from running, hence eliminating the [FAIL] boottime message that
>  will occur otherwise.
>
>  Cheers,
>  Essien
>  
oops... forgot the attachement :D

[J. Hartline]

Essien Essien wrote:

>      The attached patch checks for firstboot and if its _not_ installed
>      disables it from running, hence eliminating the [FAIL] boottime
>      message that will occur otherwise.
>
>  def tweak_firstboot_config (rootdir):
>  	shf = sysconfig_file (rootdir, "firstboot")
> -	shf["RUN_FIRSTBOOT"] = "YES"
> +	if os.path.exists('/usr/sbin/'):
> +		shf["RUN_FIRSTBOOT"] = "YES"
> +	else:
> +		shf["RUN_FIRSTBOOT"] = "NO"
>  	shf.write ()
>  
>   
>  
Whether /usr/sbin exists or not, doesn't play any part if firstboot was 
installed, or am I overlooking something?

J. Hartline

[Essien Essien]

On 2/24/06, J. Hartline <jasperhartline(a)adelphia.net&gt; wrote:
>  Essien Essien wrote:
>
>  >
>  >
>  >     The attached patch checks for firstboot and if its _not_ installed
>  >     disables it from running, hence eliminating the [FAIL] boottime
>  >     message that will occur otherwise.
>  >
>  > def tweak_firstboot_config (rootdir):
>  >       shf = sysconfig_file (rootdir, "firstboot")
>  >-      shf["RUN_FIRSTBOOT"] = "YES"
>  >+      if os.path.exists('/usr/sbin/'):
>  >+              shf["RUN_FIRSTBOOT"] = "YES"
>  >+      else:
>  >+              shf["RUN_FIRSTBOOT"] = "NO"
>  >       shf.write ()
>  >
>  >
>  >
>  Whether /usr/sbin exists or not, doesn't play any part if firstboot was
>  installed, or am I overlooking something? 

my bad. corrected patch attached. that should be /usr/sbin/firstboot.

[Jeremy Katz]

On Fri, 2006-02-24 at 14:41 +0100, Essien Essien wrote:
>  The attached patch checks for firstboot and if its _not_ installed
>  disables it from running, hence eliminating the [FAIL] boottime
>  message that will occur otherwise. 
Ermm, the firstboot initscript is part of the firstboot package and thus
should only be installed (and try to run) if you actually install
firstboot.

Jeremy

[Essien Essien]

On 2/24/06, Jeremy Katz <katzj(a)redhat.com&gt; wrote:
>  On Fri, 2006-02-24 at 14:41 +0100, Essien Essien wrote:
>  > The attached patch checks for firstboot and if its _not_ installed
>  > disables it from running, hence eliminating the [FAIL] boottime
>  > message that will occur otherwise.
>
>  Ermm, the firstboot initscript is part of the firstboot package and thus
>  should only be installed (and try to run) if you actually install
>  firstboot.
>
>  Jeremy 

ahhh correct!!! well... some folks on irc seemed to be having a problem. I
guess its a weird corner case? ;)

[J. Hartline]

Essien Essien wrote:

>  ahhh correct!!! well... some folks on irc seemed to be having a 
>  problem. I guess its a weird corner case? ;) 
Chitlesh said he was getting firstboot servive errors.