Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Fix libssl dependency for deb build script. #6542

Closed
wants to merge 1 commit into from
Closed

Conversation

5chdn
Copy link
Contributor

@5chdn 5chdn commented Sep 18, 2017

Reverts #4260

Closes #5893

@5chdn 5chdn added A0-pleasereview 🤓 Pull request needs code review. M0-build 🏗 Building and build system. labels Sep 18, 2017
@arkpar
Copy link
Collaborator

arkpar commented Sep 18, 2017

Does it break compatibility with ubuntu 14.x?

@5chdn
Copy link
Contributor Author

5chdn commented Sep 19, 2017

Not trying to become emotional about this :-p

Just did a quick search, this will break Debian oldstable (Jessie) and all Ubuntu versions. I do not fully understand how it's impossible to ship the 1.1 version of openssl with new Ubuntu versions.

Ubuntu 14.04 openssl 1.0.1f libssl-dev 1.0.1f
Ubuntu 16.04 openssl 1.0.2g libssl-dev 1.0.2g
Ubuntu 17.04 openssl 1.0.2g libssl-dev 1.0.2g
Ubuntu 17.10 openssl 1.0.2g libssl-dev 1.0.2g

Debian 08 OldStable   (Jessi) openssl 1.0.1t libssl-dev 1.0.1t
Debian 09    Stable (Stretch) openssl 1.1.0f libssl-dev 1.1.0f
Debian 10   Testing  (Buster) openssl 1.1.0f libssl-dev 1.1.0f

Closing because I have no idea how to tackle this. I was assuming that if a dinosaur like debian introduces openssl 1.1, ubuntu would have long been there,

@5chdn 5chdn closed this Sep 19, 2017
@5chdn 5chdn added A4-gotissues 💥 Pull request is reviewed and has significant issues which must be addressed. and removed A0-pleasereview 🤓 Pull request needs code review. labels Sep 19, 2017
@5chdn
Copy link
Contributor Author

5chdn commented Sep 19, 2017

The only solution that comes to my mind is providing different packages for different linux derivatives and versions. Or move away from debs at all.

@5chdn 5chdn deleted the a5-build-openssl branch September 19, 2017 10:03
@arkpar
Copy link
Collaborator

arkpar commented Sep 19, 2017

Won't something like libssl1.0.0 (>=1.0.0) | libssl (>=1.1.0) work?

@5chdn
Copy link
Contributor Author

5chdn commented Sep 20, 2017

No. Well, maybe, but it will get pretty complex. The issue is debian and ubuntu use different naming convention. In Debian 10 it's libssl1.0.2 for 1.0.2g and in Ubuntu it's libssl1.0.0 for 1.0.2l. Here is a matrix that displays same packages by name and colors them by different versions:

screenshot at 2017-09-20 11-55-18

With the current configuration, only Debian stable and testing is broken. The following could work (I'm not sure if the syntax allows it):

libssl (>=1.0.0) | libssl1.0.0 (>=1.0.0) | libssl1.0.2 (>=1.0.0)

But I would need to find a way to test that first.

@jpritikin
Copy link

Is this merged for parity 1.8? Because I still can't install parity 1.8.

@5chdn
Copy link
Contributor Author

5chdn commented Oct 16, 2017

No, I closed this because this breaks all Ubuntu versions. If you have any idea, please let me know.

@iFA88
Copy link

iFA88 commented Nov 1, 2017

If you wish to use parity on Debian 9 ( stretch ) you need download the package from https://packages.debian.org/jessie/libssl1.0.0 and install like dpkg --install libssl1.0.0_1.0.1t-+deb8u6_amd64.deb as root. For me works.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A4-gotissues 💥 Pull request is reviewed and has significant issues which must be addressed. M0-build 🏗 Building and build system.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants