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.

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

musl build #35

Merged
merged 4 commits into from
Nov 30, 2023
Merged

musl build #35

merged 4 commits into from
Nov 30, 2023

Conversation

ericonr
Copy link
Member

@ericonr ericonr commented Nov 27, 2023

Doesn't work yet, needs a patch, which is still under investigation. Considering simply forwarding this PR to tech-talk to see if we get any help.

diff --git a/base/install-functions.sh b/base/install-functions.sh
index f1e6a87..f856a65 100644
--- a/base/install-functions.sh
+++ b/base/install-functions.sh
@@ -18,6 +18,9 @@ install_module() {
     if [ -n "$NEEDS_TIRPC" ]; then
         echo "TIRPC=YES" > configure/CONFIG_SITE.local
     fi
+    #if [ -f asyn/Makefile ]; then
+    #    sed -i s/USR_INCLUDES_Linux/USR_INCLUDES/ asyn/Makefile
+    #fi
 
     make -j${JOBS} install
     make clean

@ericonr
Copy link
Member Author

ericonr commented Nov 27, 2023

So, I think the issue was caused by some bad cache. I just got it to build without the workaround.

@ericonr ericonr force-pushed the musl branch 4 times, most recently from cd980ac to 5bc536b Compare November 28, 2023 11:38
Copy link
Collaborator

@henriquesimoes henriquesimoes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Documentation is also missing in the README.md and CHANGES.md.

base/install_epics.sh Outdated Show resolved Hide resolved
.github/workflows/base-image.yml Outdated Show resolved Hide resolved
base/epics-base-musl.patch Outdated Show resolved Hide resolved
base/install-functions.sh Outdated Show resolved Hide resolved
base/musl/docker-compose.yml Outdated Show resolved Hide resolved
This is necessary for the asyn module with modern glibc or musl.
This is intended mainly for building fully static IOCs to be deployed on
older machines. We are using Alpine since it's the main musl-based
distro with LTS releases.

We also document it in README and CHANGES.
It was also necessary to reorganize the env section for CI to work.
@ericonr ericonr merged commit c07047c into main Nov 30, 2023
2 checks passed
@ericonr ericonr deleted the musl branch November 30, 2023 14:28
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.

2 participants