On Fri, Oct 01, 2010 at 10:54:10 +0200, Alan Pevec <[email protected]> wrote: > -Requires: syslinux > Requires: /sbin/extlinux
I didn't think is was worth counting on the dependency in syslinux, so I didn't drop the requirement. > But what about Paul's actual patch: I didn't notice that before. I'll review that. For now I don't think missing a chattr is likely to be a big deal, so this can probably just go upstream and end up in 035. > - chattr -i $USBMNT/$SYSLINUXPATH/extlinux.sys > + chattr -i $USBMNT/$SYSLINUXPATH/ldlinux.sys > > This is not in git master, and to preserve compatibility it should > check for both: > - chattr -i $USBMNT/$SYSLINUXPATH/extlinux.sys > + if [ -f "$USBMNT/$SYSLINUXPATH/extlinux.sys" ]; then > - chattr -i $USBMNT/$SYSLINUXPATH/extlinux.sys > + elif [ -f "$USBMNT/$SYSLINUXPATH/ldlinux.sys" ]; then > + chattr -i $USBMNT/$SYSLINUXPATH/ldlinux.sys > + fi > > Alan > -- > livecd mailing list > [email protected] > https://admin.fedoraproject.org/mailman/listinfo/livecd -- livecd mailing list [email protected] https://admin.fedoraproject.org/mailman/listinfo/livecd
