Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Refactor and further simplify the build script #64

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Refactor and further simplify the build script #64

wants to merge 4 commits into from

Commits on Sep 20, 2024

  1. Simplifications and cleanup to the build script 

    * The script now supports zsh and bash to allow building from macOS and
  Linux with builtin zsh and bash version 3 or 4.
* It now uses curl instead of wget, wget is not builtin on macOS.
* Use the default NDk path from the Android SDK.
* Allow to set custom build and output folders.
* The various build folders for each version are now kept around under
  the 'build' folder.
* At the same time add the build folder to .gitignore.
* Create relative symlinks KDAB#61.
    @Issam-b
    Issam-b committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    99cefa3 View commit details
    Browse the repository at this point in the history

  2. Get rid of patchelf requirement 

    OpenSSL has the configuration option shlib_variant, so we can use that instead.

This works for version 1.x and 3.x, so it would make the build script more similar between the two versions.

Also, this avoid issues that can come from patchelf, as this patch comes after a bug found in patchelf 0.18 that created wrongly aligned libraries. See NixOS/patchelf#492.
    @Issam-b
    Issam-b committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    c310893 View commit details
    Browse the repository at this point in the history

  3. Add test to check the libs are working properly 

    It's a simple check at runtime to see if ssl is supported using the included libs.
    @Issam-b
    Issam-b committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    eb5d0fd View commit details
    Browse the repository at this point in the history

  4. Update library artefacts after script changes

    @Issam-b
    Issam-b committed Sep 20, 2024
    Configuration menu
    Copy the full SHA
    d12a622 View commit details
    Browse the repository at this point in the history