On Sa, 2010-06-19 at 19:45 +0200, Toni Mueller wrote: > Hi, > > On Sat, 19.06.2010 at 16:54:37 +0200, Toni Mueller <[email protected]> wrote: > > NEED_UPDATE_CHECK=no > > > > for i in /$EtcDir/$SourceList /$EtcDir/${SourceList}.d/*; do > > test -f $i && NEED_UPDATE_CHECK=yes > > done > > I suggest the following change instead: > > > $ diff -uw update-motd-updates-available > /usr/lib/update-notifier/update-motd-updates-available > --- update-motd-updates-available 2010-05-05 12:27:47.000000000 +0200 > +++ /usr/lib/update-notifier/update-motd-updates-available 2010-06-19 > 18:56:52.000000000 +0200 > @@ -40,9 +40,17 @@ > SourceList="sources.list" > eval $(apt-config shell EtcDir Dir::Etc) > eval $(apt-config shell SourceList Dir::Etc::sourcelist) > -if [ $(stat -c %Y "/$EtcDir/$SourceList") -gt $stampt ]; then > + > +NEED_UPDATE_CHECK=no > + > +for i in /$EtcDir/$SourceList /$EtcDir/${SourceList}.d/*; do > + if [ -f $i ] && [ $(stat -c %Y "$i") -gt $stampt ]; then > NEED_UPDATE_CHECK=yes > fi > +done > + > + > +
The whole thing is broken anyway and needs to be reworked. BTW, the location of sources.list.d is not the location of sources.list (Dir::Etc::sourcelist) and a '.d', but Dir::Etc::sourceparts. I couldn't care less about this bug (OK, maybe a bit less is possible), as the functionality causing it is disabled in the version in unstable. -- Julian Andres Klode - Debian Developer, Ubuntu Member See http://wiki.debian.org/JulianAndresKlode and http://jak-linux.org/. -- To UNSUBSCRIBE, email to [email protected] with a subject of "unsubscribe". Trouble? Contact [email protected]
