Subject: Developers of Sympa
List archive
Re: [sympa-developpers] [sympa-commits] sympa[9135] branches/sympa-cleanup/src/sbin/bulk.pl.in: [-bug] use PIDDIR for stderr file
- From: IKEDA Soji <address@concealed>
- To: address@concealed
- Subject: Re: [sympa-developpers] [sympa-commits] sympa[9135] branches/sympa-cleanup/src/sbin/bulk.pl.in: [-bug] use PIDDIR for stderr file
- Date: Thu, 8 Aug 2013 13:03:20 +0900
On Tue, 06 Aug 2013 16:36:01 +0200
Guillaume Rousse <address@concealed> wrote:
> Le 05/08/2013 15:52, IKEDA Soji a écrit :
> >> Bascally, those PID files or STDERR files are both temporary files of
> >> the same class: they normally exist only during the lifetime of a single
> >> process, and should be removed during normal process termination. If
> >> they are left behind, that's because a crash occured, and they should be
> >> removed on next process startup. So I think it makes sense to host both
> >> kind of files on /var/run/sympa, and to use a single build-time
> >> configuration directive for it. It could use a better name than PIDDIR,
> >> however, such as RUNDIR for instance.
> >
> > It's in sense that PID files will be removed when the daemons
> > stopeed normally.
> As does the stderr ones, according to my own understanding of their purpose.
>
> So, what's about my proposal ?
> - use File::Temp for all create-and-forget files
File::Temp seems to cleanup files even in case of abnormal
termination (e.g. runtime error by a typo in method names).
On PID files and STDERR outputs, no traces of crash can't be
left.
However, this module will be useful for other kind of temporary
files discussed below.
> - deprecated tmpdir configuration variable
> - rename --piddir configure switch to --rundir
> - use RUNDIR for both pid and stderr files
The mountpoint for PID files does not always intend files with
gradual increase or large mass. I suppose these two would be
defined separately.
There is the other use of tmpdir which may be rather primary use:
Saving messages to feed them to external utilities (MHOnArc,
antivirus, openssl, ...). Though generic temporary partition such
as /var/tmp may be used, a dedicated place for sensitive data may
be desirable.
So, I suppose tmpdir parameter is useful and it would be distinct
from PIDDIR.
Regards,
--- Soji
--
株式会社 コンバージョン セキュリティ&OSSソリューション部 池田荘児
〒231-0004 神奈川県横浜市中区元浜町3-21-2 ヘリオス関内ビル7F
e-mail address@concealed TEL 045-640-3550
http://www.conversion.co.jp/
-
Re: [sympa-developpers] [sympa-commits] sympa[9135] branches/sympa-cleanup/src/sbin/bulk.pl.in: [-bug] use PIDDIR for stderr file,
IKEDA Soji, 08/02/2013
-
Re: [sympa-developpers] [sympa-commits] sympa[9135] branches/sympa-cleanup/src/sbin/bulk.pl.in: [-bug] use PIDDIR for stderr file,
Guillaume Rousse, 08/05/2013
-
Re: [sympa-developpers] [sympa-commits] sympa[9135] branches/sympa-cleanup/src/sbin/bulk.pl.in: [-bug] use PIDDIR for stderr file,
IKEDA Soji, 08/05/2013
-
Re: [sympa-developpers] [sympa-commits] sympa[9135] branches/sympa-cleanup/src/sbin/bulk.pl.in: [-bug] use PIDDIR for stderr file,
Guillaume Rousse, 08/06/2013
- Re: [sympa-developpers] [sympa-commits] sympa[9135] branches/sympa-cleanup/src/sbin/bulk.pl.in: [-bug] use PIDDIR for stderr file, IKEDA Soji, 08/08/2013
-
Re: [sympa-developpers] [sympa-commits] sympa[9135] branches/sympa-cleanup/src/sbin/bulk.pl.in: [-bug] use PIDDIR for stderr file,
Guillaume Rousse, 08/06/2013
-
Re: [sympa-developpers] [sympa-commits] sympa[9135] branches/sympa-cleanup/src/sbin/bulk.pl.in: [-bug] use PIDDIR for stderr file,
IKEDA Soji, 08/05/2013
-
Re: [sympa-developpers] [sympa-commits] sympa[9135] branches/sympa-cleanup/src/sbin/bulk.pl.in: [-bug] use PIDDIR for stderr file,
Guillaume Rousse, 08/05/2013
Archive powered by MHonArc 2.6.19+.