Summary: | libdbus-init-win library not be build on autotools cross compiling | ||
---|---|---|---|
Product: | dbus | Reporter: | Ralf Habacker <ralf.habacker> |
Component: | core | Assignee: | Simon McVittie <smcv> |
Status: | RESOLVED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | medium | Keywords: | patch |
Version: | unspecified | ||
Hardware: | Other | ||
OS: | Linux (All) | ||
Whiteboard: | review? | ||
i915 platform: | i915 features: | ||
Attachments: | libdbus-1.la: have proper dependencies on Windows |
Description
Ralf Habacker
2013-08-24 12:42:31 UTC
Have you tried running autoreconf? I'm pretty sure this cross-compile worked for me. (In reply to comment #1) > Have you tried running autoreconf? I'm pretty sure this cross-compile worked > for me. yes, same results. Further investigation shows that this seems to be a Makefile.am dependency problem because manual compiling the target works. > cd <buildroot>/dbus > make libdbus-init-win.la CXX dbus-init-win.lo CXXLD libdbus-init-win.la Linking than works. Build logs with "silent rules" aren't great for this sort of thing - "make ... V=1" to see what the compiler/linker is doing. I see what the problem is, though: libdbus_1_la_DEPENDENCIES = $(dbus_res) That's overwriting the dependencies completely - libdbus-1.la won't be recompiled unless the $(dbus_res) change, which they usually don't. Created attachment 84709 [details] [review] libdbus-1.la: have proper dependencies on Windows Assigning to libdbus_1_la_DEPENDENCIES defeats Automake's normal dependency logic, which makes libdbus-1.la depend on all the static libraries that will go into it (it still had a corrct dependency on the other objects, which go through a separate variable). This meant libdbus-init-win wasn't necessarily built first. Use EXTRA_libdbus_1_la_DEPENDENCIES to avoid that problem. (In reply to comment #4) > Created attachment 84709 [details] [review] [review] > libdbus-1.la: have proper dependencies on Windows > > Assigning to libdbus_1_la_DEPENDENCIES defeats Automake's normal > dependency logic, which makes libdbus-1.la depend on all the > static libraries that will go into it (it still had a corrct dependency > on the other objects, which go through a separate variable). > This meant libdbus-init-win wasn't necessarily built first. > > Use EXTRA_libdbus_1_la_DEPENDENCIES to avoid that problem. works as expected. (In reply to comment #5) > works as expected. I'll assume that was a positive review... (please be clear about whether you're saying "yes, I reviewed this and it looks good" or not, our review policy relies on it). Fixed in git, will be in 1.7.6. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.