3356 2003-12-09 20:30:49 +0300 bogus check for detectloader 2005-07-13 15:45:38 +0400 1 1 4 Development Sisyphus lilo unstable all Linux CLOSED FIXED P2 normal --- 1 at rider led rider qa-sisyphus oldest_to_newest 10858 0 at 2003-12-09 20:30:49 +0300 Two problems with lilo.spec: 1) bogus check for /usr/sbin/detectloader (actually the test is for the given string, not for the given file); 2) missing deps on /usr/sbin/detectloader, so the post script may or may not achieve the desired effect. --- /home/at/RPM/SPECS/lilo.spec- 2003-08-20 21:50:52 +0400 +++ /home/at/RPM/SPECS/lilo.spec 2003-12-09 16:44:00 +0300 @@ -22,6 +22,7 @@ ExclusiveArch: %ix86 +Requires: /usr/sbin/detectloader BuildPreReq: kernel-headers-std # Automatically added by buildreq on Fri May 30 2003 @@ -73,7 +74,7 @@ bzip2 -9 $RPM_BUILD_ROOT%docdir/{README,CHANGES,*.ps} %post -if [ -f /etc/%name.conf -a /usr/sbin/detectloader -a -f /proc/partitions ]; then +if [ -f /etc/%name.conf -a -x /usr/sbin/detectloader -a -f /proc/partitions ]; then if [ "`/usr/sbin/detectloader`" = LILO ]; then /sbin/%name || echo "Please run %name manually." >&2 fi 10970 1 ldv 2003-12-22 02:16:28 +0300 Thanks for the typo report. I think that detectloader dependency is not required though. 15620 2 ldv 2004-06-25 19:53:17 +0400 Fixed the typo in lilo-22.4.1-alt5