[PATCH REVIEW]: Include Shark code in the build again

[Kelly O'Hair]

On Mar 30, 2011, at 9:26 AM, Dr Andrew John Hughes wrote:

> This change (arrived at by both myself and Gary Benson separately):
> 
> http://cr.openjdk.java.net/~gbenson/zero-shark-fixes-04-1/
> 
> fixes the issue by adding Shark back into the mix.
> 
> Kelly, can you approve these build changes?

Looks fine to me.

---
Although overall, I don't understand why someone used so many := assignments.
I usually reserve := for special assignments that require a shell or complex evaluation.
The := forces an expansion of the right side on the spot, so any later changes to the
variables used on the right will have no impact on your variable. In Makefiles, that's
usually what you don't want, but I suppose there are cases where that is expected.
By using := all the time, you have created a specific order dependency on the variables, but
whatever, ...

-kto

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/build-dev/attachments/20110330/6f2e7f92/attachment.htm>