[Pkg-clamav-devel] New upload?

[Stephen Gran]

This one time, at band camp, Michael Tautschnig said:
> > Hello all,
> > 
> > I've done a little cleanup on the various branches and think it's ready
> > for another upload.  Any comments before it's uploaded?
> > 
> 
> Sorry for hijacking this thread, but I just wanted to thank you for doing all
> the work.
> 
> I was and still am quite busy with real life, and therefore don't find the time
> to stay tuned in on IRC. Could somebody just briefly explain the current status
> and what is to be done at the moment? I think uploads to volatile might still be
> pending, but didn't really check that.

Yes, we need volatile uploads quite badly.  I haven't had the time to do
the final items on my todo before uploading:

 - redo variable usage in milter postinst (more below)
 - write VUA for upload
 - check installability/Depends of -dev package on etch
 - check milter upgrade from 0.94 and 0.95 versions and do any cleanup
   there
 - check milter functionality

The milter postinst currently does things like:
    db_metaget clamav-milter/MilterSocket value || true
    MilterSocket="$RET"
Then
    slurp_config "$CLAMAVMILTERCONF"

Where $CLAMAVMILTERCONF will have an entry like
MilterSocket /var/spool/postfix/run/milter.sock

What this does is overwrite the data returned from debconf with the data
returned from parsing the config file.  We need to use different variables
for the debconf part of it (I defaulted to lower case versions with the
same name in clamav-base.postinst.in, so in this case MilterSocket and
miltersocket would do).  Then we need to make sure that where debconf
and the config file differ, the variable from debconf takes precedence.
This can be achieved by something like:

    db_metaget clamav-milter/MilterSocket value || true
    miltersocket="$RET"
    ...
    slurp_config "$CLAMAVMILTERCONF"
    ...
    [ -n "$miltersocket" ] || miltersocket="$MilterSocket"
    ...
    cat > $DEBCONFFILE <<EOF
    MilterSocket $miltersocket

And needs to be repeated for each variable that can be gotten from
debconf.  This is obviously a lot of cleanup and needs to be tested
afterwards fairly carefully.  Since I haven't used the milter in literally
5 years, I am not confident that I will catch all the corner cases that
could go wrong.  I'm willing to do the work if no one else gets to it
first, but I have quite a lot going on right now and it won't be soon.

Cheers,
-- 
 -----------------------------------------------------------------
|   ,''`.                                            Stephen Gran |
|  : :' :                                        sgran at debian.org |
|  `. `'                        Debian user, admin, and developer |
|    `-                                     http://www.debian.org |
 -----------------------------------------------------------------
-------------- 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/pkg-clamav-devel/attachments/20090628/b705b588/attachment.pgp>