Update watch files for polenum and acccheck
Marcos Fouces
mfouces at yahoo.es
Tue Nov 8 22:53:22 UTC 2016
El 08/11/16 a las 15:53, Raphael Hertzog escribió:
> Hello Marcos,
>
> sorry for the delay.
>
> On Mon, 24 Oct 2016, Marcos Fouces wrote:
>> All three packages are (hopefully) ready for upload.
> Uploaded polenum, acccheck and sucrack.
>
>> BTW, there is also acct and bbqsql packages waiting.
> Looking at bbqsql, it's good, so I uploaded it.
>
> Looking at acct, it's definitely not good to go. You merged patches
> from the BTS without any critical review... not everything that
> is submitted are good ideas. Sometimes bug reporters suggest ugly work
> arounds...
>
> Commenting on commits:
> 3747373540d0ea8402e102982843b35fe7ba768b More robust cron.daily.
>
> => you dropped the most important line: the one calling "savelog"
Yes, this is a typo on my side...
>
> ca6e05f26923ef32f7616bf2d8c25b44c2bfd922 Fix broken cron.monthly
>
> => the bug reporter basically reverted the change of the maintainer
> at that time, it was certainly broken but I would still keep some
> of the improvements like the early exit when "/usr/bin/accton" does
> not exist (instead of the big "if)
>
> even better would be to work with the logrotate maintainer to get
> a hook into a "prerotate" script associated to wtmp handling. Because
> this script is really hackish.
I decided to include it because it is a better solution than existent
one, as you stated.
Surely, the early exit if the executable does not exist is a good idea.
I will rewrote the script to keep it.
>
> 4fe61651c1088c87fc3c31b9a9c84f6915f99eeb keep stopping and starting acct
>
> => this is definitely not a good fix, if the problem reported is true,
> then it needs to be fixed at the glibc or kernel level, I suspect the
> problem does no longer exist as we haven't gotten many "me too" in years
> in that old bug report. Please revert that change and close the bug.
I included this patch because i jbelieve that the test included doesn't
create any trouble and it solves the issue.
The submiter send it just a year ago (15 Jun 2015) so it is not so old.
Anyway, i will revert and close the bug.
>
> The other changes looked fine from a cursory look. Did you test the
> package?
I will do the suggested changes and i'll test the package. In 2-3 weeks
it should be ready.
Cheers,
Marcos
More information about the Pkg-security-team
mailing list