
<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">[adding build-dev and macosx-port-dev]<div><br></div><div>On 21 jun 2012, at 14:43, David Holmes wrote:</div><div><div><br class="Apple-interchange-newline"><blockquote type="cite"><div>On 21/06/2012 10:30 PM, Staffan Larsen wrote:<br><blockquote type="cite">Do you mean:<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">.PHONY: $(UNIVERSAL_LIPO_LIST) $(UNIVERSAL_COPY_LIST)<br></blockquote><br>Yes. Now they will always be rebuilt.<br><br><blockquote type="cite">Yes, that seems to have the same effect. Probably a better solution.<br></blockquote><br>I think both of these simply mask the real problem. I still don't understand how only some of the list items get "rebuilt". The CR says<br><br>"These targets will only be run for the last item in the xxx_LIST variables (which happens to be the client jvm)"<br><br>but I don't understand why that is?<br></div></blockquote><div><br></div><div>Neither do I. Makefiles is black magic to me. I only discovered that building the complete JDK from the top-level directory did not update the hotspot bits in the j2sdk-image and this was the ultimate cause.</div><div><br></div><div>Here is an updated webrev: <a href="http://cr.openjdk.java.net/~sla/7178667/webrev.02/">http://cr.openjdk.java.net/~sla/7178667/webrev.02/</a></div><div><br></div><div><div>Thanks,</div><div>/Staffan</div><div><br></div></div><br><blockquote type="cite"><div>But I also don't understand this universalization process.<br><br>BTW you might want to run this past the bsd-port folks (don't recall the exact alias) and/or build-dev. I seem to recall that last time we changed something to do with universal builds it actually broke something.</div></blockquote><blockquote type="cite"><div><br>David<br><br><blockquote type="cite">Thanks,<br></blockquote><blockquote type="cite">/Staffan<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">On 21 jun 2012, at 14:12, David Holmes wrote:<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite"><blockquote type="cite">Hi Staffan,<br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite">On 21/06/2012 6:33 PM, Staffan Larsen wrote:<br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite">Please review the following fix to makefiles for universal binaries on<br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite">max os x. The idea is to force the target to be executed for all items<br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite">in the list.<br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite"><br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite">Fix contributed by Rickard Bäckman (rbackman).<br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite"><br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><blockquote type="cite">webrev: <a href="http://cr.openjdk.java.net/~sla/7178667/webrev.01/">http://cr.openjdk.java.net/~sla/7178667/webrev.01/</a><br></blockquote></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite">I don't understand the problem that this addresses but wouldn't you get the same affect by declaring those targets as PHONY ?<br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite">David<br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite"><br></blockquote></blockquote><blockquote type="cite"><blockquote type="cite">PS. Unrelated but I was astounded to see that bsd/Makefile and linux/Makefile both have a chunk of code conditional on "ifeq ($(OSNAME),solaris)" Huh!<br></blockquote></blockquote><blockquote type="cite"><br></blockquote></div></blockquote></div><br></div></body></html>