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

Allow goaccess to be cross compiled. #2038

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ericonr
Copy link

@ericonr ericonr commented Feb 23, 2021

When cross compiling, bin2c needs to be built for the build system, so
building it with the CC compiler won't work. We use an autoconf macro in
order to obtain a good value for CC_FOR_BUILD, and define a rule using
it directly instead.

Reference
https://www.gnu.org/software/autoconf-archive/ax_cc_for_build.html


If you want to avoid the dependency on autoconf-archive for the new macro, the logic is reasonably easy to replicate.

I noticed that bin2c.c has some code for using bzip2, but I don't think that can be enabled in any way, at least for now?

When cross compiling, bin2c needs to be built for the build system, so
building it with the CC compiler won't work. We use an autoconf macro in
order to obtain a good value for CC_FOR_BUILD, and define a rule using
it directly instead.

Reference
https://www.gnu.org/software/autoconf-archive/ax_cc_for_build.html
@ericonr
Copy link
Author

ericonr commented Feb 23, 2021

CI seems to be failing because CC_FOR_BUILD isn't defined, which I would assume happened because it lacks autoconf-archive.

@allinurl
Copy link
Owner

Thanks for submitting this and sorry for the delay. Are you able to avoid the autoconf-archive dependency? I'd certainly like to it keep without that dependency.

Yeah I did not include the compression part as an option. Do you need it in your case?

Thanks again!

@allinurl allinurl force-pushed the master branch 2 times, most recently from 0550432 to b1332f5 Compare December 22, 2023 22:26
@allinurl allinurl force-pushed the master branch 5 times, most recently from 3d5333b to 6c68a6e Compare January 11, 2024 01:36
@allinurl allinurl force-pushed the master branch 7 times, most recently from 31e2ac1 to c6199f6 Compare May 14, 2024 22:48
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