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!

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.

None yet

2 participants