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

transitive extract_all_objects no longer works #3401

Closed
bruce-richardson opened this issue Apr 13, 2018 · 5 comments
Closed

transitive extract_all_objects no longer works #3401

bruce-richardson opened this issue Apr 13, 2018 · 5 comments

Comments

@bruce-richardson
Copy link
Contributor

There is new error added by commit 809f018: "extract_all_objects: Also extract generated sources".

ERROR: Cannot extract objects from a target that itself has extracted objects

There are many cases where this might be needed, and it seemed to work fine in previous versions of meson. This is breaking the build of the DPDK project where we are compiling for older x86_64 targets without AVX2 instruction set support. In that case, we compile the AVX2 code separately as a static lib, extact it and then merge with the rest of the code into the official library static lib. That lib in turn is converted to a shared lib again by extract_all_objects(). [Yes, I have a TODO to look at SIMD module and "both_library" support for this, but it is what it is now.]

Please fix before next release. This used to work and is a regression IMHO.

@tp-m tp-m added this to the meson-next milestone Apr 13, 2018
@nirbheek
Copy link
Member

@xclaesse

@xclaesse
Copy link
Member

It never worked, it was silently not including those objects.

@nirbheek
Copy link
Member

Then we should convert that error into a warning.

@xclaesse
Copy link
Member

We should fix it instead. Shouldn't be hard ;-)

@nirbheek
Copy link
Member

I'm assuming you're volunteering for that 😉

xclaesse added a commit to xclaesse/meson that referenced this issue Apr 13, 2018
xclaesse added a commit to xclaesse/meson that referenced this issue Apr 13, 2018
xclaesse added a commit to xclaesse/meson that referenced this issue Apr 13, 2018
xclaesse added a commit to xclaesse/meson that referenced this issue Apr 13, 2018
xclaesse added a commit to xclaesse/meson that referenced this issue Apr 16, 2018
xclaesse added a commit to xclaesse/meson that referenced this issue Apr 17, 2018
xclaesse added a commit to xclaesse/meson that referenced this issue Apr 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants