Developers of Sympa

[IKEDA Soji <address@concealed>]

Hi,

# During merge by David and Etienne, I personally had been working
# up merging -cleanup branch to -6.2.  Now it has mostly finished.
# This doesn't aim at preciseness (in fact resulting code doesn't
# work), but helps for my understanding on Guillaume's work.
# So hereafter I may ask questions about some commits.

> sympa[9135] branches/sympa-cleanup/src/sbin/bulk.pl.in: [-bug] use PIDDIR 
> for stderr file
> sympa[9139] branches/sympa-cleanup/src/sbin/bulk.pl.in: [-bug] use PIDDIR 
> for stderr file

By 6.2a, STDERR of daemons were directed to files under tmpdir.
What is the reason to switch them to PIDDIR?

I feel it is undesirable that /var/run directory is filled by
temporary files.  And, cleanup process by sympa.pl does not work:
it targets tmpdir.


Regards,

-- Soji

On Thu, 18 Apr 2013 10:02:43 +0200 (CEST)
address@concealed wrote:

> sympa[9135] branches/sympa-cleanup/src/sbin/bulk.pl.in: [-bug] use PIDDIR 
> for stderr file
> Revision 9135 Author rousse Date 2013-04-18 10:02:43 +0200 (jeu. 18 avril 
> 2013)
> Log Message[-bug] use PIDDIR for stderr file
> Modified Paths
> branches/sympa-cleanup/src/sbin/bulk.pl.in
> Diff
> Modified: branches/sympa-cleanup/src/sbin/bulk.pl.in (9134 => 9135)
> --- branches/sympa-cleanup/src/sbin/bulk.pl.in        2013-04-18 08:01:57 
> UTC (rev 9134)
> +++ branches/sympa-cleanup/src/sbin/bulk.pl.in        2013-04-18 08:02:43 
> UTC (rev 9135)
> @@ -155,10 +155,10 @@
>  # If process is running in foreground, don't write STDERR to a dedicated 
> file
>  unless ($options{foreground}) {
>       Sympa::Tools::Daemon::direct_stderr_to_file(
> -             tmpdir => $Sympa::Configuration::Conf{'tmpdir'},
> -             pid    => $PID,
> -             user   => Sympa::Constants::USER,
> -             group  => Sympa::Constants::GROUP,
> +             directory => Sympa::Constants::PIDDIR,
> +             pid       => $PID,
> +             user      => Sympa::Constants::USER,
> +             group     => Sympa::Constants::GROUP,
>       );
>  }
>  ## Set the UserID & GroupID for the process


-- 
株式会社 コンバージョン  セキュリティ&OSSソリューション部   池田荘児
〒231-0004　神奈川県横浜市中区元浜町3-21-2 ヘリオス関内ビル7F
e-mail address@concealed  TEL 045-640-3550
http://www.conversion.co.jp/