On 09:42 Wed 30 Mar     , Kelly O'Hair wrote:
> 
> 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.
> 

Thanks, so now I just need a HotSpot dev to give it a bug ID and push it through
their JPRT thing...

> ---
> 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, ...
> 

I've never really used := either, and I see your point, but that's not really 
something to change
in this changeset :-)

> -kto
> 

-- 
Andrew :)

Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

Support Free Java!
Contribute to GNU Classpath and IcedTea
http://www.gnu.org/software/classpath
http://icedtea.classpath.org
PGP Key: F5862A37 (https://keys.indymedia.org/)
Fingerprint = EA30 D855 D50F 90CD F54D  0698 0713 C3ED F586 2A37

Reply via email to