[Buildroot] svn commit: trunk/buildroot/package/dbus
jacmet at uclibc.org
Sat Jan 31 22:26:42 UTC 2009
>>>>> "Ulf" == Ulf Samuelsson <ulf.samuelsson at atmel.com> writes:
Ulf> Why are you doing this?
Ulf> You are AGAIN breaking boards...
>> What? What is broken? A wee bit more detail would be handy.
Ulf> The problem is that you make dbus depend on expat/libxml2.
No, I make the configure target of dbus depend on expat/libxml2 being
installed into staging.
Ulf> To build dbus, you need to do a number of things
Ulf> install staging dir
Ulf> install target
Ulf> dbus = "ready"
Ulf> If you make "dbus" depend on "libxml2", then
Ulf> libxml2 can be built at ANY time between start of make
Ulf> until the "dbus" install target rule is executed
Ulf> which will complete the dbus build.
Which is why I don't do that.
Ulf> "dbus" needs to have access to the libraries already
Ulf> at "configure" time, or the build will abort.
Ulf> It so happens that "dbus" configure starts before
Ulf> libxml2 is built for some unclear reason.
Because the dbus makefile is listed before the libxml2 one.
Ulf> Therefore *"dbus-configured"* meeds to be dependent on
Ulf> "expat" | "libxml2".
Almost, it needs to depend on a non-phony target providing the
expat/libxml2 library/headers in staging dir.
Ulf> Then they are built before the configure starts
Ulf> and the build will complete without errors.
Ulf> You were right that it is OK to depend on libxml2|expat
Ulf> instead of strange libraries/goals.
Ulf> but the solution was not complete which broke the build.
Where isn't it complete? Your commit (besides removing valuable
comments) made dbus configure depend on the phony expat target instead
of the real rule, so dbus will alreads be considered out of date for
expat (just like for libxml2, but as the (deleted) comment states,
there's not much to do about that besides fixing up
Makefile.autotools.in which I DONT suggest we do before the release.
This should ofcourse be seen in the light of all your dbus.mk changes
since the 25th to "fix continous rebuilds" ;)
In other words, I don't see where I'm "AGAIN breaking boards..."
Just to check, I did a build of at91rm9200df_defconfig, and the dbus
build completed without any errors, so unless I hear otherwise from
you, I WILL revert your revert.
Bye, Peter Korsgaard
More information about the buildroot