[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: one package altering other package's postrm



> : Russ Allbery <rra@debian.org>
> Well, ideally this is something that should be coordinated with the cron package.

I've sent a one-liner patch, as this put the least work on cron maintainers:

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=773095

I guess they can apply it right away, even during the freeze;
if there were a new RC bug popping up;
this could even be included in Jessie.

>  How do the other cron replacements, like bcron, handle this?
It seems not to implement this feature.

> It feels cleaner to me
To me too, I guess I could check in preinst
[1] that /etc/cron.allow or /etc/cron.deny even exists (by default no)
[2] that some version of cron without this patch is installed
before deleting cron.postrm as last resort.

> : Guillem Jover <guillem@debian.org>
> Move the handling of those (and any other) common files or dirs
>    (like /etc/cron.allow, /etc/cron.deny, crontab.5, /etc/crontab,
>    the /etc/cron.* dirs and placeholders, and possibly also the cron
>    spool) to a third package (say cron-common/cron-support/cron-base/etc)
>    that both packages depend on.

systemd-cron must ship it's own (blanks) /etc/crontab & /etc/anacrontab
to avoid duplicate execution and also has it's own crontab.1 & .5 that
documents the "PERSITENT=true" feature and other quirks.

Ok for the dirs & placeholders tough;
and to move "rm -f /etc/cron.allow|deny" in cron-base postrm.

For exemple: Gentoo does ship "cronbase" with only the folders:
http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/sys-process/cronbase/cronbase-0.3.3.ebuild

> : Marco d'Itri <md@Linux.IT>
> Ugly, but sometimes there is no other option.
> I am quite sure that I had to do this once or twice in my packages and 
> no adverse effects have ever been reported.
Ok, I'll first let cron's maintainer a chance to apply the patch & DTRT
... and I'll wait anyway till I find a new sponsor to update this package:

http://lists.alioth.debian.org/pipermail/pkg-systemd-maintainers/2014-December/005181.html

Alexandre Detiste

Attachment: signature.asc
Description: This is a digitally signed message part.


Reply to: