alexcrichton / openssl-src-rs

Source code and logic to build OpenSSL from source
Apache License 2.0
69 stars 114 forks source link

Bump to 1.1.1o #136

Closed amousset closed 2 years ago

alexcrichton commented 2 years ago

Looks like the Android build on CI is broken, can you either work to fix it or remove it here as well?

amousset commented 2 years ago

I checked with current main branch to be sure and it fails too, so I'm disabling it.

alexcrichton commented 2 years ago

Ah unfortunately I cannot publish this due to:

   Uploading openssl-src v111.19.0+1.1.1o (/home/acrichto/code/openssl-src-rs)
error: failed to publish to registry at https://crates.io

Caused by:
  the remote server responded with an error: max upload size is: 10485760

where the tarball looks to be 14MB:

$ l target/package/openssl-src-111.19.0+1.1.1o.crate
-rw-r--r-- 1 acrichto acrichto 14M May  5 10:44 target/package/openssl-src-111.19.0+1.1.1o.crate
amousset commented 2 years ago

I'll have a look at what is included

amousset commented 2 years ago

It happens because of the new wycheproof submodule (which seems to be initialized automatically when updating the openssl submodule), when cloning openssl-src-rs from scratch and non-recursively initializing submodules, I get a size similar to previous releases.

alexcrichton commented 2 years ago

Ah ok that was fixed on main awhile back. Would you be interested in backporting that?

amousset commented 2 years ago

Sure!

amousset commented 2 years ago

Opened #139