After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 549693 - ORBit2 might enter an infinite make loop when used on parallel make
ORBit2 might enter an infinite make loop when used on parallel make
Status: RESOLVED FIXED
Product: ORBit2
Classification: Deprecated
Component: general
2.14.x
Other All
: Normal normal
: ---
Assigned To: ORBit maintainers
ORBit maintainers
Depends on:
Blocks:
 
 
Reported: 2008-08-28 10:26 UTC by Diego Elio Pettenò
Modified: 2008-09-08 17:27 UTC
See Also:
GNOME target: ---
GNOME version: 2.21/2.22


Attachments
Patch to fix. (757 bytes, patch)
2008-08-28 10:27 UTC, Diego Elio Pettenò
none Details | Review

Description Diego Elio Pettenò 2008-08-28 10:26:57 UTC
Please describe the problem:
When building iwth parallel make (make -jN with N > 1), the build might get stuck in an infinite loop. The reason for this is that the .PHONY directive tells make that a given rule has _always_ to be rebuilt, so it will be stuck in calling the make for idl-compiler.

The attached patch sanitises the build a bit by removing the phony rule, and making the rule look "real" to make. It also removes the "true" calls, as it's just declaring dependencies for the .c file rather than having a rule generate it.

Steps to reproduce:


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Diego Elio Pettenò 2008-08-28 10:27:19 UTC
Created attachment 117514 [details] [review]
Patch to fix.
Comment 2 Michael Meeks 2008-08-28 11:01:41 UTC
looks fine to commit to me. Of course, the IDL compilation itself is horrible in ORBit2 - just running 'make' will re-build the IDL and hence a chunk of the code itself - some broken Makefile rules from the past.
It'd be great to fix that too, if you're passing :-)
Comment 3 Kjartan Maraas 2008-08-28 11:18:22 UTC
I commited this. If you want to look at the other issues there's a patch in bug 319379.
Comment 4 Kjartan Maraas 2008-09-08 17:27:41 UTC
Closing as well.