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

compile error w/ apxs #21

Open
AerisG222 opened this issue Jan 1, 2013 · 5 comments
Open

compile error w/ apxs #21

AerisG222 opened this issue Jan 1, 2013 · 5 comments

Comments

@AerisG222
Copy link

Hello,

I was not able to compile on my system (Fedora 17), as it was failing on the call to apxs, with an error stating:

apxs:Error: Unknown option: f.

After looking at what was going on, the results on my system of MagickWand-config --cppflags --ldflags --libs|xargs resulted in the following:

-fopenmp -I/usr/include/ImageMagick -lMagickWand -lMagickCore -lMagickWand -lMagickCore

I was able to get past this by updating that portion of the Makefile with the following to strip out that option (believe real change would have to go into Makefile.am):

MagickWand-config --cppflags --ldflags --libs |xargs |sed "s/-fopenmp\s//"

Afterwards, I was able to get it to compile. I do not know if this negatively affects what is built, I have no intention of using the apache module. It might also be nice to be able to override whether the module is built by including an --disable option for apache (sorry if it is there and I did not see it, am terrible with autotools).

Thank you for this tool, am excited to see how much it will shave off my images!

-Mike

@rflynn
Copy link
Owner

rflynn commented Jan 2, 2013

What's the output of

cc --version

@AerisG222
Copy link
Author

cc (GCC) 4.7.2 20121109 (Red Hat 4.7.2-8)

@rflynn
Copy link
Owner

rflynn commented Feb 21, 2013

Thanks. I'll have to add proper autotools detection for openmp, though I don't know it well.

@Magotchi
Copy link

Magotchi commented Feb 6, 2014

AerisG222's fix works in src/apache2/Makefile.am, but on mine I needed the "global" flag in sed, as follows:
|sed "s/-fopenmp\s//g"

micmro pushed a commit to micmro/imgmin that referenced this issue Aug 3, 2015
rflynn added a commit that referenced this issue Aug 9, 2015
issue #28 fixed with @AerisG222 solution from #21
@bagipriyank
Copy link

@rflynn any update on this?

zvezdochiot pushed a commit to ImageProcessing-ElectronicPublications/imgmin that referenced this issue Dec 24, 2022
zvezdochiot pushed a commit to ImageProcessing-ElectronicPublications/imgmin that referenced this issue Dec 24, 2022
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

No branches or pull requests

4 participants