Bug#717493: pu: package sympa/6.1.11~dfsg-5.1~deb7u1

2014-04-19 Thread Adam D. Barratt 

Control: tags -1 + pending

On 2014-04-19 0:43, Emmanuel Bouthenot wrote:

On Fri, Apr 18, 2014 at 11:12:07AM +0100, Adam D. Barratt wrote:
[...]


Given how close we are to the upload window for 7.5 closing (over
this weekend), I'm prepared to accept the fix as-is on the assumption
that it will get to unstable in the very near future, so please go
ahead with the upload for stable.

Done a few minutes ago.


Flagged for acceptance; thanks.

Regards,

Adam


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#717493: pu: package sympa/6.1.11~dfsg-5.1~deb7u1

2014-04-18 Thread Adam D. Barratt 

Control: tags -1 + pending

On 2014-04-17 22:46, Emmanuel Bouthenot wrote:

On Thu, Apr 17, 2014 at 02:24:24PM +0100, Adam D. Barratt wrote:

I'm slightly torn on this one. Whilst the patch seems obviously
correct, it's not applied to the package in unstable, which is
usually a requirement for accepting a patch for stable. There's an
argument that it's unlikely that anyone will try to update from
squeeze's sympa to jessie's without having upgraded perl in between,
but conversely the patch should work fine with the package in
unstable.

Are there any plans to apply the patch, or otherwise update the
package, in unstable?


This fix apply on a Debian patch[1] which has been merged upstream[2] 
in

sympa 6.1.19 and the fix itself was applied a few commits later[3].

I'm currently working on the packaging of sympa 6.1.20. It should be
ready to be uploaded in unstable within the next few days.


Thanks.

Given how close we are to the upload window for 7.5 closing (over this 
weekend), I'm prepared to accept the fix as-is on the assumption that it 
will get to unstable in the very near future, so please go ahead with 
the upload for stable.


Regards,

Adam


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#717493: pu: package sympa/6.1.11~dfsg-5.1~deb7u1

2014-04-18 Thread Emmanuel Bouthenot 
On Fri, Apr 18, 2014 at 11:12:07AM +0100, Adam D. Barratt wrote:
[...]

 Given how close we are to the upload window for 7.5 closing (over
 this weekend), I'm prepared to accept the fix as-is on the assumption
 that it will get to unstable in the very near future, so please go
 ahead with the upload for stable.
Done a few minutes ago.

Thanks,

M.

-- 
Emmanuel Bouthenot
  mail: kolter@{openics,debian}.orggpg: 4096R/0x929D42C3
  xmpp: kol...@im.openics.org  irc: kolter@{freenode,oftc}


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#717493: pu: package sympa/6.1.11~dfsg-5.1~deb7u1

2014-04-17 Thread Adam D. Barratt 

On 2014-04-16 23:10, Emmanuel Bouthenot wrote:

On Tue, Apr 15, 2014 at 09:25:11PM +0100, Adam D. Barratt wrote:

Please go ahead with the upload; thanks.


If you agree I'd like to fix 2 other bugs: #709240 and #714388

#709240 is fixed by a 2 lines patch (syntax error with Perl 5.14
present in Squeeze)


I'm slightly torn on this one. Whilst the patch seems obviously 
correct, it's not applied to the package in unstable, which is usually 
a requirement for accepting a patch for stable. There's an argument that 
it's unlikely that anyone will try to update from squeeze's sympa to 
jessie's without having upgraded perl in between, but conversely the 
patch should work fine with the package in unstable.


Are there any plans to apply the patch, or otherwise update the package, 
in unstable?



#714388 is fixed by adding a missing file


That looks fine.

Regards,

Adam


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#717493: pu: package sympa/6.1.11~dfsg-5.1~deb7u1

2014-04-17 Thread Emmanuel Bouthenot 
On Thu, Apr 17, 2014 at 02:24:24PM +0100, Adam D. Barratt wrote:
[...]

 I'm slightly torn on this one. Whilst the patch seems obviously
 correct, it's not applied to the package in unstable, which is
 usually a requirement for accepting a patch for stable. There's an
 argument that it's unlikely that anyone will try to update from
 squeeze's sympa to jessie's without having upgraded perl in between,
 but conversely the patch should work fine with the package in
 unstable.
 
 Are there any plans to apply the patch, or otherwise update the
 package, in unstable?

This fix apply on a Debian patch[1] which has been merged upstream[2] in
sympa 6.1.19 and the fix itself was applied a few commits later[3].

I'm currently working on the packaging of sympa 6.1.20. It should be
ready to be uploaded in unstable within the next few days.


[1] 
http://patch-tracker.debian.org/patch/series/view/sympa/6.1.11~dfsg-5/1011_fix_sqlite_db_upgrade
[2] 
https://sourcesup.renater.fr/scm/viewvc.php/branches/sympa-6.1-branch/src/lib/Upgrade.pm?root=sympar1=9974r2=9995
[3] 
https://sourcesup.renater.fr/scm/viewvc.php/branches/sympa-6.1-branch/src/lib/Upgrade.pm?root=sympar1=9996r2=10003


Regards,

M.

-- 
Emmanuel Bouthenot
  mail: kolter@{openics,debian}.orggpg: 4096R/0x929D42C3
  xmpp: kol...@im.openics.org  irc: kolter@{freenode,oftc}


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#717493: pu: package sympa/6.1.11~dfsg-5.1~deb7u1

2014-04-16 Thread Emmanuel Bouthenot 
Adam,

On Tue, Apr 15, 2014 at 09:25:11PM +0100, Adam D. Barratt wrote:
[...]

 Apologies for the huge delay in getting back to you regarding this.
 Please go ahead with the upload; thanks.

If you agree I'd like to fix 2 other bugs: #709240 and #714388

#709240 is fixed by a 2 lines patch (syntax error with Perl 5.14
present in Squeeze)

#714388 is fixed by adding a missing file

Attached is the debdiff.


Regards,

M.

-- 
Emmanuel Bouthenot
  mail: kolter@{openics,debian}.orggpg: 4096R/0x929D42C3
  xmpp: kol...@im.openics.org  irc: kolter@{freenode,oftc}
diff -Nru sympa-6.1.11~dfsg/debian/changelog sympa-6.1.11~dfsg/debian/changelog
--- sympa-6.1.11~dfsg/debian/changelog  2012-11-30 19:01:18.0 +
+++ sympa-6.1.11~dfsg/debian/changelog  2014-04-16 21:46:20.0 +
@@ -1,3 +1,15 @@
+sympa (6.1.11~dfsg-5+deb7u1) stable-proposed-updates; urgency=low
+
+  * Add a patch to raise a warning instead of an error when the CA
+bundle file is not readable.
+  * Add a patch to fix CAS authentication issues (Closes: #720383)
+  * Fix SQLite upgrade patch to avoid errors with perl = 5.14
+(Closes: #709240)
+  * Add a patch to provide the missing template help_suspend.tt2
+(Closes: #714388)
+
+ -- Emmanuel Bouthenot kol...@debian.org  Wed, 16 Apr 2014 21:45:53 +
+
 sympa (6.1.11~dfsg-5) unstable; urgency=low
 
   * Fix SQLite patch to avoid crash during installation from scratch and
diff -Nru sympa-6.1.11~dfsg/debian/patches/0001_fix_cas_authentication.patch 
sympa-6.1.11~dfsg/debian/patches/0001_fix_cas_authentication.patch
--- sympa-6.1.11~dfsg/debian/patches/0001_fix_cas_authentication.patch  
1970-01-01 00:00:00.0 +
+++ sympa-6.1.11~dfsg/debian/patches/0001_fix_cas_authentication.patch  
2013-10-03 14:04:12.0 +
@@ -0,0 +1,18 @@
+Description: Fix CAS/LDAP authentication
+Author: F. Périchon
+Origin: upstream, 
https://sourcesup.renater.fr/scm/viewvc.php?view=revisionroot=symparevision=7917
+Bug-Debian: http://bug.debian.org/720383
+Last-Update: 2013-10-03
+---
+This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
+--- a/wwsympa/Auth.pm
 b/wwsympa/Auth.pm
+@@ -330,7 +330,7 @@
+ filter = $filter,
+ scope = $ldap-{'ldap_scope'},
+ timeout = $ldap-{'ldap_timeout'},
+-attrs =  $ldap-{'ldap_email_attribute'}
++attrs =  
[$ldap-{'ldap_email_attribute'}],
+ );
+   my $count = $emails-count();
+ 
diff -Nru sympa-6.1.11~dfsg/debian/patches/1011_fix_sqlite_db_upgrade 
sympa-6.1.11~dfsg/debian/patches/1011_fix_sqlite_db_upgrade
--- sympa-6.1.11~dfsg/debian/patches/1011_fix_sqlite_db_upgrade 2012-11-30 
16:32:19.0 +
+++ sympa-6.1.11~dfsg/debian/patches/1011_fix_sqlite_db_upgrade 2014-04-16 
20:09:54.0 +
@@ -207,7 +207,7 @@
 +  }
 +  }
 +  my $sqloldfields = join(, , 
@oldfields);
-+  my $sqlnewfieldstype = join(, 
, map { $_ $db_struct{$Conf::Conf{'db_type'}}{${t}}{$_} } keys 
$db_struct{$Conf::Conf{'db_type'}}{${t}});
++  my $sqlnewfieldstype = join(, 
, map { $_ $db_struct{$Conf::Conf{'db_type'}}{$t}{$_} } keys 
%{$db_struct{$Conf::Conf{'db_type'}}{$t}});
 +
 +  # SQLite doesn't support 
changes on columns types hence we use a dirty workaround here:
 +  # we create a temporary table 
(a clone of the table containing the field to update)
@@ -246,8 +246,8 @@
 +  do_log('notice', $sqlcode);
 +  $dbh-do($sqlcode);
 +  # Update real structure
-+  foreach my $newfield (sort keys 
$db_struct{$Conf::Conf{'db_type'}}{${t}}) {
-+  
$real_struct{$t}{$newfield} = 
$db_struct{$Conf::Conf{'db_type'}}{${t}}{$newfield};
++  foreach my $newfield (sort keys 
%{$db_struct{$Conf::Conf{'db_type'}}{$t}}) {
++  
$real_struct{$t}{$newfield} = $db_struct{$Conf::Conf{'db_type'}}{$t}{$newfield};
 +  }
 +  }
 +  push @report, sprintf('Field %s in 
table %s, structure updated', $f, $t);
diff -Nru 
sympa-6.1.11~dfsg/debian/patches/2001_ca_bundle_check_as_warning.patch 
sympa-6.1.11~dfsg/debian/patches/2001_ca_bundle_check_as_warning.patch
---

Bug#717493: pu: package sympa/6.1.11~dfsg-5.1~deb7u1

2014-04-15 Thread Adam D. Barratt 
Control: tags -1 + confirmed

On Thu, 2013-10-03 at 19:01 +0200, Emmanuel Bouthenot wrote:
 On Mon, Sep 30, 2013 at 01:15:54AM +0200, Cyril Brulebois wrote:
 [...]
 
  To reduce user perplexity, I guess we could consider getting sympa to
  catch this situation instead, which would reduce side effects. I think
  something along the lines of this commit would go in a better direction
  that adding a Pre-Depends:
 I fully agree.
 
 I've prepared a pu upload (I hope It will be ok for the next point
 release) which fixes this issue and another one (#720383).
 
 About #720383, the fix is a one line patch.
 
 Attached is the debdiff.

Apologies for the huge delay in getting back to you regarding this.
Please go ahead with the upload; thanks.

Regards,

Adam


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#717493: pu: package sympa/6.1.11~dfsg-5.1~deb7u1

2013-10-03 Thread Emmanuel Bouthenot 
On Mon, Sep 30, 2013 at 01:15:54AM +0200, Cyril Brulebois wrote:
[...]

 To reduce user perplexity, I guess we could consider getting sympa to
 catch this situation instead, which would reduce side effects. I think
 something along the lines of this commit would go in a better direction
 that adding a Pre-Depends:
I fully agree.

I've prepared a pu upload (I hope It will be ok for the next point
release) which fixes this issue and another one (#720383).

About #720383, the fix is a one line patch.

Attached is the debdiff.

Regards,

M.

-- 
Emmanuel Bouthenot
  mail: kolter@{openics,debian}.orggpg: 4096R/0x929D42C3
  xmpp: kol...@im.openics.org  irc: kolter@{freenode,oftc}
diff -Nru sympa-6.1.11~dfsg/debian/changelog sympa-6.1.11~dfsg/debian/changelog
--- sympa-6.1.11~dfsg/debian/changelog  2012-11-30 19:01:18.0 +
+++ sympa-6.1.11~dfsg/debian/changelog  2013-10-03 14:54:00.0 +
@@ -1,3 +1,11 @@
+sympa (6.1.11~dfsg-5+deb7u1) stable-proposed-updates; urgency=low
+
+  * Add a patch to raise a warning instead of an error when the CA
+bundle file is not readable.
+  * Add a patch to fix CAS authentication issues (Closes: #720383)
+
+ -- Emmanuel Bouthenot kol...@debian.org  Thu, 03 Oct 2013 19:54:58 +
+
 sympa (6.1.11~dfsg-5) unstable; urgency=low
 
   * Fix SQLite patch to avoid crash during installation from scratch and
diff -Nru sympa-6.1.11~dfsg/debian/patches/0001_fix_cas_authentication.patch 
sympa-6.1.11~dfsg/debian/patches/0001_fix_cas_authentication.patch
--- sympa-6.1.11~dfsg/debian/patches/0001_fix_cas_authentication.patch  
1970-01-01 00:00:00.0 +
+++ sympa-6.1.11~dfsg/debian/patches/0001_fix_cas_authentication.patch  
2013-10-03 14:04:12.0 +
@@ -0,0 +1,18 @@
+Description: Fix CAS/LDAP authentication
+Author: F. Périchon
+Origin: upstream, 
https://sourcesup.renater.fr/scm/viewvc.php?view=revisionroot=symparevision=7917
+Bug-Debian: http://bug.debian.org/720383
+Last-Update: 2013-10-03
+---
+This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
+--- a/wwsympa/Auth.pm
 b/wwsympa/Auth.pm
+@@ -330,7 +330,7 @@
+ filter = $filter,
+ scope = $ldap-{'ldap_scope'},
+ timeout = $ldap-{'ldap_timeout'},
+-attrs =  $ldap-{'ldap_email_attribute'}
++attrs =  
[$ldap-{'ldap_email_attribute'}],
+ );
+   my $count = $emails-count();
+ 
diff -Nru 
sympa-6.1.11~dfsg/debian/patches/2001_ca_bundle_check_as_warning.patch 
sympa-6.1.11~dfsg/debian/patches/2001_ca_bundle_check_as_warning.patch
--- sympa-6.1.11~dfsg/debian/patches/2001_ca_bundle_check_as_warning.patch  
1970-01-01 00:00:00.0 +
+++ sympa-6.1.11~dfsg/debian/patches/2001_ca_bundle_check_as_warning.patch  
2013-10-03 13:48:51.0 +
@@ -0,0 +1,50 @@
+Description: Raise a warning instead of an error when the CA bundle file is 
not readable.
+  For its internal usage (to make https request and verify smime mail 
signatures) Sympa
+  needs access to the Certificates Authorities installed on the system.
+
+  Certificates Authorities are available in the package ca-certificates and
+  this package also provides a single file with all the CA concatenated.
+  The location of this file is /etc/ssl/certs/ca-certificates.crt
+
+  Sympa package provides a symlink from /usr/share/sympa/default/ca-bundle.crt
+  to /etc/ssl/certs/ca-certificates.crt
+
+  Recently, to fix bug #537051 the ca-certificates package maintainer has
+  decided to replace the CA bundle generation
+  (/etc/ssl/certs/ca-certificates.crt) in the postinst by a trigger.
+
+  During installation of Sympa, if ca-certificates is not present, it is
+  installed as a Sympa dependency but the sympa postinst will always be
+  executed before the ca-certificates triggers.
+  It means that Sympa start will fail with an error because
+  /etc/ssl/certs/ca-certificates.crt does not exists (not yet generated by the
+  ca-certificates trigger).
+
+  The following patch tries to detect if the CA bundle configured in sympa is a
+  symlink to /etc/ssl/certs/ca-certificates.crt. In that case, a warning raised
+  instead of en error.
+
+  This is a purely Debian specific fix, which is quite ugly but it works.
+
+  A better patch will be very much appreciated.
+Author: Emmanuel Bouthenot kol...@debian.org
+Bug-Debian: http://bugs.debian.org/706965
+Forwarded: not-needed
+Last-Update: 2013-07-02
+---
+This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
+--- a/src/lib/Conf.pm
 b/src/lib/Conf.pm
+@@ -876,7 +876,11 @@
+ 
+ ## Check cafile and capath access
+ if (defined $Conf{'cafile'}  $Conf{'cafile'}) {
+-  unless (-f $Conf{'cafile'}  -r $Conf{'cafile'}) {
++  my $debian_ca_bundle = '/etc/ssl/certs/ca-certificates.crt';
++  if (-l $Conf{'cafile'}  ! -r

Bug#717493: pu: package sympa/6.1.11~dfsg-5.1~deb7u1

2013-09-29 Thread Cyril Brulebois 
Julien Cristau jcris...@debian.org (2013-07-21):
 On Sun, Jul 21, 2013 at 15:27:17 +0200, Andreas Beckmann wrote:
 
  On 2013-07-21 15:13, Adam D. Barratt wrote:
   Whichever solution was chosen, it looks like this bug is not fixed in
   unstable yet?
  
  Yes, I wanted to write a sentence about this as well ...
  
  If the release team considers adding a pre-depends in stable as
  acceptable, I would first upload a NMU to sid (same patch, just omitting
  the wheezy changelog entry) and later on to wheezy.
  Instead of doing NMUs, maintainer uploads would be welcome of course!
  
 I think adding a pre-depends in stable would be worse than this bug.

Indeed.

To reduce user perplexity, I guess we could consider getting sympa to
catch this situation instead, which would reduce side effects. I think
something along the lines of this commit would go in a better direction
that adding a Pre-Depends:
  
http://anonscm.debian.org/gitweb/?p=collab-maint/sympa.git;a=commitdiff;h=7ab6b9b7de9ec7cd27ebf2d8046f7ea227b583a6

(I'm attaching it for reference.)

Mentioning how to fix the issue would be even nicer.

Mraw,
KiBi.
Description: Raise a warning instead of an error when the CA bundle file is not 
readable.
  For its internal usage (to make https request and verify smime mail 
signatures) Sympa
  needs access to the Certificates Authorities installed on the system.

  Certificates Authorities are available in the package ca-certificates and
  this package also provides a single file with all the CA concatenated.
  The location of this file is /etc/ssl/certs/ca-certificates.crt

  Sympa package provides a symlink from /usr/share/sympa/default/ca-bundle.crt
  to /etc/ssl/certs/ca-certificates.crt

  Recently, to fix bug #537051 the ca-certificates package maintainer has
  decided to replace the CA bundle generation
  (/etc/ssl/certs/ca-certificates.crt) in the postinst by a trigger.

  During installation of Sympa, if ca-certificates is not present, it is
  installed as a Sympa dependency but the sympa postinst will always be
  executed before the ca-certificates triggers.
  It means that Sympa start will fail with an error because
  /etc/ssl/certs/ca-certificates.crt does not exists (not yet generated by the
  ca-certificates trigger).

  The following patch tries to detect if the CA bundle configured in sympa is a
  symlink to /etc/ssl/certs/ca-certificates.crt. In that case, a warning raised
  instead of en error.

  This is a purely Debian specific fix, which is quite ugly but it works.

  A better patch will be very much appreciated.
Author: Emmanuel Bouthenot kol...@debian.org
Bug-Debian: http://bugs.debian.org/706965
Forwarded: not-needed
Last-Update: 2013-07-02
---
This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
--- a/src/lib/Conf.pm
+++ b/src/lib/Conf.pm
@@ -920,7 +920,11 @@
 
 ## Check cafile and capath access
 if (defined $Conf{'cafile'}  $Conf{'cafile'}) {
-   unless (-f $Conf{'cafile'}  -r $Conf{'cafile'}) {
+   my $debian_ca_bundle = '/etc/ssl/certs/ca-certificates.crt';
+   if (-l $Conf{'cafile'}  ! -r $Conf{'cafile'}  
readlink($Conf{'cafile'}) == $debian_ca_bundle) {
+   do_log('warning', 'cafile (%s) is a symlink to the Debian CA 
certificates bundle (%s) which is not yet installed yet (see #706965)', 
$Conf{'cafile'}, $debian_ca_bundle);
+   }
+   elsif (! -f $Conf{'cafile'} || ! -r $Conf{'cafile'}) {
do_log('err', 'Cannot access cafile %s', $Conf{'cafile'});
unless (List::send_notify_to_listmaster('cannot_access_cafile', 
$Conf{'domain'}, [$Conf{'cafile'}])) {
do_log('err', 'Unable to send notify cannot access cafile to 
listmaster');   


signature.asc
Description: Digital signature

Bug#717493: pu: package sympa/6.1.11~dfsg-5.1~deb7u1

2013-07-21 Thread Adam D. Barratt 
Control: tags -1 + wheezy moreinfo

On Sun, 2013-07-21 at 15:07 +0200, Andreas Beckmann wrote:
 Sympa fails to install if the ca-certificates trigger has never been
 run, yet.  #717435
 One solution is to add a Pre-Depends: ca-certificates to sympa
 (suggested patch attached).
 The other solution would be to fix this in ca-certificates and force a
 trigger run during postinst (not sure if that is really feasible due to
 the interaction needed with ca-certificates-java). Fixing this in
 ca-certificates would prevent similar bugs to happen in other packages,
 although I haven't seen any more in my piuparts tests.

Whichever solution was chosen, it looks like this bug is not fixed in
unstable yet?

Regards,

Adam


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#717493: pu: package sympa/6.1.11~dfsg-5.1~deb7u1

2013-07-21 Thread Andreas Beckmann 
On 2013-07-21 15:13, Adam D. Barratt wrote:
 Whichever solution was chosen, it looks like this bug is not fixed in
 unstable yet?

Yes, I wanted to write a sentence about this as well ...

If the release team considers adding a pre-depends in stable as
acceptable, I would first upload a NMU to sid (same patch, just omitting
the wheezy changelog entry) and later on to wheezy.
Instead of doing NMUs, maintainer uploads would be welcome of course!


Andreas


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#717493: pu: package sympa/6.1.11~dfsg-5.1~deb7u1

2013-07-21 Thread Julien Cristau 
On Sun, Jul 21, 2013 at 15:27:17 +0200, Andreas Beckmann wrote:

 On 2013-07-21 15:13, Adam D. Barratt wrote:
  Whichever solution was chosen, it looks like this bug is not fixed in
  unstable yet?
 
 Yes, I wanted to write a sentence about this as well ...
 
 If the release team considers adding a pre-depends in stable as
 acceptable, I would first upload a NMU to sid (same patch, just omitting
 the wheezy changelog entry) and later on to wheezy.
 Instead of doing NMUs, maintainer uploads would be welcome of course!
 
I think adding a pre-depends in stable would be worse than this bug.

Cheers,
Julien


signature.asc
Description: Digital signature

Bug#717493: pu: package sympa/6.1.11~dfsg-5.1~deb7u1

2013-07-21 Thread Andreas Beckmann 
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: pu

Sympa fails to install if the ca-certificates trigger has never been
run, yet.  #717435
One solution is to add a Pre-Depends: ca-certificates to sympa
(suggested patch attached).
The other solution would be to fix this in ca-certificates and force a
trigger run during postinst (not sure if that is really feasible due to
the interaction needed with ca-certificates-java). Fixing this in
ca-certificates would prevent similar bugs to happen in other packages,
although I haven't seen any more in my piuparts tests.


Andreas
diff -Nru sympa-6.1.11~dfsg/debian/changelog sympa-6.1.11~dfsg/debian/changelog
--- sympa-6.1.11~dfsg/debian/changelog	2012-11-30 20:01:18.0 +0100
+++ sympa-6.1.11~dfsg/debian/changelog	2013-07-21 14:41:49.0 +0200
@@ -1,3 +1,17 @@
+sympa (6.1.11~dfsg-5.1~deb7u1) wheezy; urgency=low
+
+  * Non-maintainer upload.
+  * Rebuild for wheezy.
+
+ -- Andreas Beckmann a...@debian.org  Sun, 21 Jul 2013 14:41:08 +0200
+
+sympa (6.1.11~dfsg-5.1) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * Add Pre-Depends: ca-certificates.  (Closes: #717435)
+
+ -- Andreas Beckmann a...@debian.org  Sun, 21 Jul 2013 14:37:04 +0200
+
 sympa (6.1.11~dfsg-5) unstable; urgency=low
 
   * Fix SQLite patch to avoid crash during installation from scratch and
diff -Nru sympa-6.1.11~dfsg/debian/control sympa-6.1.11~dfsg/debian/control
--- sympa-6.1.11~dfsg/debian/control	2012-05-21 14:23:32.0 +0200
+++ sympa-6.1.11~dfsg/debian/control	2013-07-21 13:59:24.0 +0200
@@ -36,7 +36,7 @@
 
 Package: sympa
 Architecture: any
-Pre-Depends: dpkg (= 1.15.7.2~)
+Pre-Depends: dpkg (= 1.15.7.2~), ca-certificates
 Depends: ${perl:Depends}, ${shlibs:Depends}, ${misc:Depends}, ${cdbs:Depends}
 Replaces: wwsympa
 Provides: wwsympa