Colin, I'll rather propose that (simplifying the code):

     # Prevent from updating if parameters already present (persistent usb key)
     if ! `grep -qs 'AutomaticLoginEnable' $GDMCustomFile` ; then
        if ! `grep -qs '\[daemon\]' $GDMCustomFile` ; then
            printf '[daemon]' >> $GDMCustomFile
        fi
        sed -i "s/\[daemon\]/\[daemon\]\n$AutologinParameters/" $GDMCustomFile
     fi

With that, we handle every cases:
- no GDMCustomFile, create the autologin part with [daemon]
- GDMCustomFile already exists with  [daemon] section but without the autologin 
part (case of ubutu derivatives): add it
- GDMCustomFile exists without [daemon] section: add all the autologin stuff 
and the section (don't know if that can happen with current GDM implementation, 
but it'll surely later)
- GDMCustomFile exists with  [daemon] section and autologin stuff in place 
(case of persistent usb key): don't do anything

And we get our \n interpreted by sed :)
This new proposal is in my linked branch.

** Changed in: casper (Ubuntu)
   Importance: Undecided => High

-- 
No more autologin in live CD
https://bugs.launchpad.net/bugs/505140
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to