Merge lp:~wibblymat/ubuntu/oneiric/cryptonit/openssl-transition into lp:ubuntu/oneiric/cryptonit

Proposed by Mat Scales
Status: Merged
Merge reported by: Colin Watson
Merged at revision: not available
Proposed branch: lp:~wibblymat/ubuntu/oneiric/cryptonit/openssl-transition
Merge into: lp:ubuntu/oneiric/cryptonit
Diff against target: 70 lines (+15/-3)
3 files modified
debian/changelog (+6/-0)
debian/control (+1/-1)
src/pkcs7.cpp (+8/-2)
To merge this branch: bzr merge lp:~wibblymat/ubuntu/oneiric/cryptonit/openssl-transition
Reviewer Review Type Date Requested Status
Ubuntu branches Pending
Review via email: mp+71431@code.launchpad.net

Description of the change

Cryptonit FTBFS with openssl-1.0.0, which this branch fixes.

To post a comment you must log in.
Revision history for this message
Mat Scales (wibblymat) wrote :

The package is a mess. Not maintained upstream at all since 2006. Only FTBFS fixes in Debian or Ubuntu since then.

I could tidy the package, make it 3.0 (quilt), bump the standards version, etc. if required, but it seems unlikely.

Revision history for this message
Stefano Rivera (stefanor) wrote :

Yeah, it should probably be removed from Debian (http://wiki.debian.org/qa.debian.org/removals). It looks like an interesting package, but is clearly unmaintained. I'll file a bug to that effect.

Revision history for this message
Colin Watson (cjwatson) wrote :

This appears to have been effectively merged by way of your patch sent to the Debian bug, so marking as such:

cryptonit (0.9.7-2.4ubuntu2) oneiric; urgency=low

  * src/pkcs7.cpp: fix FTBFS with OpenSSL 1.0.0. Patch from debbugs #636956,
    thanks to Mat Scales.
  * debian/rules: don't copy config.* files to have a cleaner diff.
  * Rebuild gets rid of binary dependency on openssl0.9.8.

 -- Ilya Barygin <email address hidden> Thu, 18 Aug 2011 23:53:29 +0400

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2010-10-25 11:50:24 +0000
+++ debian/changelog 2011-08-12 21:48:24 +0000
@@ -1,3 +1,9 @@
1cryptonit (0.9.7-2.4ubuntu2) oneiric; urgency=low
2
3 * Updated to build with OpenSSL 1.0.0
4
5 -- Mat Scales <mat@wibbly.org.uk> Fri, 12 Aug 2011 22:20:42 +0100
6
1cryptonit (0.9.7-2.4ubuntu1) natty; urgency=low7cryptonit (0.9.7-2.4ubuntu1) natty; urgency=low
28
3 * link against -ldl to fix FTBFS9 * link against -ldl to fix FTBFS
410
=== modified file 'debian/control'
--- debian/control 2010-10-25 11:50:24 +0000
+++ debian/control 2011-08-12 21:48:24 +0000
@@ -3,7 +3,7 @@
3Priority: optional3Priority: optional
4Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>4Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
5XSBC-Original-Maintainer: Pierre Machard <pmachard@debian.org>5XSBC-Original-Maintainer: Pierre Machard <pmachard@debian.org>
6Build-Depends: debhelper (>= 5.0.0), libldap2-dev (>= 2.1.22-1), libssl-dev (>= 0.9.7c-5), wx2.6-headers, libwxgtk2.6-dev, automake1.9, autoconf, libtool6Build-Depends: debhelper (>= 5.0.0), libldap2-dev (>= 2.1.22-1), libssl-dev (>= 1.0.0), wx2.6-headers, libwxgtk2.6-dev, automake1.9, autoconf, libtool
7Standards-Version: 3.8.07Standards-Version: 3.8.0
8Homepage: http://www.cryptonit.org8Homepage: http://www.cryptonit.org
99
1010
=== modified file 'src/pkcs7.cpp'
--- src/pkcs7.cpp 2006-06-05 14:33:40 +0000
+++ src/pkcs7.cpp 2011-08-12 21:48:24 +0000
@@ -227,15 +227,21 @@
227 void pkcs7::setDigest(DigestName name)227 void pkcs7::setDigest(DigestName name)
228 {228 {
229 switch(name) {229 switch(name) {
230#ifndef OPENSSL_NO_MD2
230 case D_MD2:231 case D_MD2:
231 digest = EVP_md2();232 digest = EVP_md2();
232 break;233 break;
234#endif
235#ifndef OPENSSL_NO_MD5
233 case D_MD5:236 case D_MD5:
234 digest = EVP_md5();237 digest = EVP_md5();
235 break;238 break;
239#endif
240#ifndef OPENSSL_NO_SHA
236 case D_SHA1:241 case D_SHA1:
237 digest = EVP_sha1();242 digest = EVP_sha1();
238 break;243 break;
244#endif
239 default:245 default:
240 break;246 break;
241 }247 }
@@ -456,7 +462,7 @@
456 int pkcs7::sign(const char *filein, Certificate &cert, Key &privateKey,462 int pkcs7::sign(const char *filein, Certificate &cert, Key &privateKey,
457 std::vector<Certificate> &certChain, bool detached)463 std::vector<Certificate> &certChain, bool detached)
458 {464 {
459 STACK_OF(509) *chain=NULL;465 STACK_OF(X509) *chain=NULL;
460 std::vector<Certificate>::iterator certIt;466 std::vector<Certificate>::iterator certIt;
461 X509 *certif = cert.getX509Certificate();467 X509 *certif = cert.getX509Certificate();
462 EVP_PKEY *pkey = privateKey.getPrivateKey();468 EVP_PKEY *pkey = privateKey.getPrivateKey();
@@ -925,7 +931,7 @@
925 else if(isSignedAndEncrypted())931 else if(isSignedAndEncrypted())
926 recipientStack = sk_PKCS7_RECIP_INFO_dup(p7->d.signed_and_enveloped->recipientinfo);932 recipientStack = sk_PKCS7_RECIP_INFO_dup(p7->d.signed_and_enveloped->recipientinfo);
927 933
928 while((cert=sk_PKCS7_RECIP_INFO_pop(recipientStack)->cert) != NULL) {934 while((cert=((PKCS7_RECIP_INFO *)sk_PKCS7_RECIP_INFO_pop(recipientStack))->cert) != NULL) {
929 Certificate *certif = new Certificate();935 Certificate *certif = new Certificate();
930 certif->setX509Certificate(cert);936 certif->setX509Certificate(cert);
931#ifdef DEBUG937#ifdef DEBUG

Subscribers

People subscribed via source and target branches

to all changes: