maximilian attems
2005-Jan-06 12:45 UTC
[Logcheck-devel] Re: [Logcheck-commits] CVS logcheck/debian
minor nitpicking. On Wed, 05 Jan 2005, CVS User ttroxell wrote:> @@ -70,6 +70,10 @@ > chown logcheck /var/lock/logcheck > /dev/null 2>&1 || true > fi > > + # fix for #284788 > + # update timestamp on cron > + touch /etc/cron.d/logcheck || true > + > ;; > > abort-upgrade|abort-remove|abort-deconfigure)this should be conditionalized as only need on install time and not on upgrade times. thanks for solving that! -- maks
Todd Troxell
2005-Jan-06 21:16 UTC
[Logcheck-devel] Re: [Logcheck-commits] CVS logcheck/debian
On Thu, Jan 06, 2005 at 01:45:58PM +0100, maximilian attems wrote:> On Wed, 05 Jan 2005, CVS User ttroxell wrote: > > + touch /etc/cron.d/logcheck || true > this should be conditionalized as only need on install time > and not on upgrade times. > thanks for solving that!The thing is, if we do upgrade from an old version that is running as root, we may need to update the timestamp to fix the bug. I guess it could be _version_ conditionalized though. -- [ Todd J. Troxell ,''`. Student, Debian GNU/Linux Developer, SysAdmin, Geek : :' : http://debian.org || http://rapidpacket.com/~xtat `. `' `- ] -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: Digital signature Url : http://lists.alioth.debian.org/pipermail/logcheck-devel/attachments/20050106/0d374455/attachment.pgp
maximilian attems
2005-Jan-07 16:01 UTC
[Logcheck-devel] Re: [Logcheck-commits] CVS logcheck/debian
On Thu, 06 Jan 2005, Todd Troxell wrote:> On Thu, Jan 06, 2005 at 01:45:58PM +0100, maximilian attems wrote: > > On Wed, 05 Jan 2005, CVS User ttroxell wrote: > > > + touch /etc/cron.d/logcheck || true > > this should be conditionalized as only need on install time > > and not on upgrade times. > > thanks for solving that! > > The thing is, if we do upgrade from an old version that is running as root, > we may need to update the timestamp to fix the bug.agreed.> I guess it could be _version_ conditionalized though.we can do it on next release. :) -- maks