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
1=== modified file 'debian/changelog'
2--- debian/changelog 2010-10-25 11:50:24 +0000
3+++ debian/changelog 2011-08-12 21:48:24 +0000
4@@ -1,3 +1,9 @@
5+cryptonit (0.9.7-2.4ubuntu2) oneiric; urgency=low
6+
7+ * Updated to build with OpenSSL 1.0.0
8+
9+ -- Mat Scales <mat@wibbly.org.uk> Fri, 12 Aug 2011 22:20:42 +0100
10+
11 cryptonit (0.9.7-2.4ubuntu1) natty; urgency=low
12
13 * link against -ldl to fix FTBFS
14
15=== modified file 'debian/control'
16--- debian/control 2010-10-25 11:50:24 +0000
17+++ debian/control 2011-08-12 21:48:24 +0000
18@@ -3,7 +3,7 @@
19 Priority: optional
20 Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
21 XSBC-Original-Maintainer: Pierre Machard <pmachard@debian.org>
22-Build-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, libtool
23+Build-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
24 Standards-Version: 3.8.0
25 Homepage: http://www.cryptonit.org
26
27
28=== modified file 'src/pkcs7.cpp'
29--- src/pkcs7.cpp 2006-06-05 14:33:40 +0000
30+++ src/pkcs7.cpp 2011-08-12 21:48:24 +0000
31@@ -227,15 +227,21 @@
32 void pkcs7::setDigest(DigestName name)
33 {
34 switch(name) {
35+#ifndef OPENSSL_NO_MD2
36 case D_MD2:
37 digest = EVP_md2();
38 break;
39+#endif
40+#ifndef OPENSSL_NO_MD5
41 case D_MD5:
42 digest = EVP_md5();
43 break;
44+#endif
45+#ifndef OPENSSL_NO_SHA
46 case D_SHA1:
47 digest = EVP_sha1();
48 break;
49+#endif
50 default:
51 break;
52 }
53@@ -456,7 +462,7 @@
54 int pkcs7::sign(const char *filein, Certificate &cert, Key &privateKey,
55 std::vector<Certificate> &certChain, bool detached)
56 {
57- STACK_OF(509) *chain=NULL;
58+ STACK_OF(X509) *chain=NULL;
59 std::vector<Certificate>::iterator certIt;
60 X509 *certif = cert.getX509Certificate();
61 EVP_PKEY *pkey = privateKey.getPrivateKey();
62@@ -925,7 +931,7 @@
63 else if(isSignedAndEncrypted())
64 recipientStack = sk_PKCS7_RECIP_INFO_dup(p7->d.signed_and_enveloped->recipientinfo);
65
66- while((cert=sk_PKCS7_RECIP_INFO_pop(recipientStack)->cert) != NULL) {
67+ while((cert=((PKCS7_RECIP_INFO *)sk_PKCS7_RECIP_INFO_pop(recipientStack))->cert) != NULL) {
68 Certificate *certif = new Certificate();
69 certif->setX509Certificate(cert);
70 #ifdef DEBUG

Subscribers

People subscribed via source and target branches

to all changes: