Skip to content

Conversation

SuperGNUS
Copy link

This adds much more flexibility to the script to not download the Boost Library or clear it out except when you add --clean for it.

Also added some more options to where to build the library and where to source it from.

@SuperGNUS SuperGNUS closed this Dec 20, 2022
@SuperGNUS SuperGNUS reopened this Dec 20, 2022
@SuperGNUS
Copy link
Author

This finally fixes the Linux building of Android, at least on Ubuntu 2.7.6, because patch ver 2.7.6 doesn't patch symlinks, you need to patch the original file. it took a long time to figure this out!

@pelya
Copy link
Collaborator

pelya commented Dec 21, 2022

You have missed some variable, because CI script downloads 1.80.0 but tries to build 1.78.0

Building boost version: 1.80.0
Detected Android NDK version 25.1
Building with TOOLSET=clang CONFIG_VARIANT=ndk23 CXXPATH=/usr/local/lib/android/sdk/ndk/25.1.8937393/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++ CFLAGS= CXXFLAGS=-std=c++14
./build-android.sh: 491: cd: can't cd to boost_1_78_0
Performing boost bootstrap
ERROR: Could not perform boostrap! See /home/runner/work/Boost-for-Android/Boost-

@pelya
Copy link
Collaborator

pelya commented Dec 22, 2022

Still the same problem with CI, the script uses incorrect Boost version

Building boost version: 1.79.0
Detected Android NDK version 25.0
Building with TOOLSET=clang CONFIG_VARIANT=ndk23 CXXPATH=/usr/local/lib/android/sdk/ndk/25.0.8775105/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++ CFLAGS= CXXFLAGS=-std=c++14
./build-android.sh: 492: cd: can't cd to boost_1_78_0

@pelya
Copy link
Collaborator

pelya commented Jan 25, 2023

CI still fails, I cannot merge this

@pelya
Copy link
Collaborator

pelya commented Jul 3, 2024

Github actions are broken from the server side. 'Waiting for a runner' but no runner is available, previously rhe same CI script was working fine.

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.

6 participants