Skip to content

Conversation

@padelsbach
Copy link
Contributor

When generating a Debian package for WolfProvider, also include a separate package for openssl (Alternate approach)

@padelsbach padelsbach force-pushed the wp_debian_ossl_nodefault branch 15 times, most recently from b6b0bc8 to 2d9f1cc Compare August 29, 2025 16:01
@padelsbach padelsbach changed the title Draft: Alternate: Add openssl debian package Add openssl debian package without libdefault Aug 29, 2025
@padelsbach padelsbach marked this pull request as ready for review August 29, 2025 16:04
@padelsbach padelsbach force-pushed the wp_debian_ossl_nodefault branch 4 times, most recently from fc44b09 to d0c8838 Compare August 29, 2025 21:51
@padelsbach padelsbach force-pushed the wp_debian_ossl_nodefault branch 5 times, most recently from f943f98 to 02450dc Compare September 2, 2025 03:22
@padelsbach padelsbach force-pushed the wp_debian_ossl_nodefault branch 2 times, most recently from 43d1b6d to 86be742 Compare September 2, 2025 03:30
@padelsbach padelsbach changed the title Add openssl debian package without libdefault Add openssl debian package Sep 2, 2025
@padelsbach padelsbach force-pushed the wp_debian_ossl_nodefault branch from 86be742 to 3beac35 Compare September 2, 2025 16:36
@padelsbach padelsbach force-pushed the wp_debian_ossl_nodefault branch 9 times, most recently from c5aeb36 to e1138e4 Compare September 9, 2025 15:46
@LinuxJedi
Copy link
Member

I don't think we need to have the entire wolfSSL source and binaries in this git repo. It would be better to have GitHub Actions to build and cache them. Once wolfSSL/wolfssl#9161 is merged, this should become quite easy.

@padelsbach
Copy link
Contributor Author

I don't think we need to have the entire wolfSSL source and binaries in this git repo. It would be better to have GitHub Actions to build and cache them. Once wolfSSL/wolfssl#9161 is merged, this should become quite easy.

Sure. The tarball here is just a stop-gap. However, from what I can tell, the packaging.yml in wolfSSL does not seem to save artifacts. Should we add that @LinuxJedi ?

@LinuxJedi
Copy link
Member

Sure. The tarball here is just a stop-gap. However, from what I can tell, the packaging.yml in wolfSSL does not seem to save artifacts. Should we add that @LinuxJedi ?

It shouldn't be required. You can pin whatever version of wolfSSL you want without it and just copy the Debian directory over + patch the configure.ac (for older than master once merged).

Unless there is a use case I'm missing?

@padelsbach
Copy link
Contributor Author

padelsbach commented Sep 9, 2025

Sure. The tarball here is just a stop-gap. However, from what I can tell, the packaging.yml in wolfSSL does not seem to save artifacts. Should we add that @LinuxJedi ?

It shouldn't be required. You can pin whatever version of wolfSSL you want without it and just copy the Debian directory over + patch the configure.ac (for older than master once merged).

Unless there is a use case I'm missing?

After some offline discussion, the basic flow is:

  1. checkout the chosen tag/version of wolfssl
  2. copy the debian directory from wolfssl master (post merge of PR 9161)
  3. patch wolfssl/configure.ac with the changes from PR 9161
  4. build the wolfssl debian package as part of the WP github action (with make deb)

@padelsbach padelsbach force-pushed the wp_debian_ossl_nodefault branch from e1138e4 to 0671182 Compare September 9, 2025 17:00
@ColtonWilley ColtonWilley merged commit c5e7219 into wolfSSL:master Sep 9, 2025
446 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants