Differences between OpenJDK 6 hotspot tip and hsx14/master tip (Was: hg: jdk6/jdk6/hotspot: 557 new changesets)

[Andrew John Hughes]

2009/9/15 Dalibor Topic <Dalibor.Topic at sun.com>:
> Andrew John Hughes wrote:
>> I guess it was inevitable a change of this size wouldn't go through
>> without problems, even though both myself and Joe have been through
>> it.
>
> Stuff happens. ;)
>
>> I verified the GNU/Linux build but, being unable to do either a
>> Solaris or Windows build with Free tools, I haven't been able to
>> verify those.
>
> From what I could see in the history, the reason why your
> build worked on Linux is this 'Merge' changeset:
>
> http://hg.openjdk.java.net/jdk6/jdk6/hotspot/rev/327aeae2236a
>
> It's hard to say where it was merged from, though the 'minimal
> tweak to makefile' idea was something I tried with OpenSolaris
> first, too, before I wrote to the list.

Ah ok, I was thinking of the earlier big merge changeset.  This one is
a number of merge artefacts I found when building the resulting merged
copy.  The 'Merge' comment reflects that it should have been part of
the original merge, and also avoids having to obtain a bug ID for this
fix (though I believe that's disabled on 6 anyway).  I guess I didn't
spot the Solaris one.

Due to the whitespace changes, pretty much every file from the merge
had conflicts.  In some cases, where it was obvious the conflicts were
just whitespace I copied over the file from hs14 as a cleaner
alternative, but some had to be merged manually.  I guess this is why
there is still some whitespace oddities and other strange changes.  To
say that notionally the two versions are meant to be pretty close, it
seems the whitespace cleanup of the HotSpot sources in 7 has fouled
things up.

In the proposed patch,
> I picked the 'just drop unused lines' approach Erik suggested,
> though, which removes the differences between the merged and
> original makefiles in this case.
>
> There is a bunch of other small differences between the original hsx 14
> code and the merged code beside the whitespace changes. I'd prefer to
> see a whitespace cleanup patch first, before we go into the remaining
> deltas - like Martin, I'd like to see the OpenJDK 6 hotspot code as
> close as possible to the 'upstream' repository, so that any additional
> deltas stand out easily.
>

Ok, I'll look at fixing that when I'm back at work next week, unless
anyone wants to doing it in the meantime.

> cheers,
> dalibor topic
> --
> *******************************************************************
> Dalibor Topic                   Tel: (+49 40) 23 646 738
> Java F/OSS Ambassador           AIM: robiladonaim
> Sun Microsystems GmbH           Mobile: (+49 177) 2664 192
> Nagelsweg 55                    http://openjdk.java.net
> D-20097 Hamburg                 mailto:Dalibor.Topic at sun.com
> Sitz der Gesellschaft: Sonnenallee 1, D-85551 Kirchheim-Heimstetten
> Amtsgericht München: HRB 161028
> Geschäftsführer: Thomas Schröder, Wolfgang Engels, Wolf Frenkel
> Vorsitzender des Aufsichtsrates: Martin Häring
>
>
>

Thanks,
-- 
Andrew :-)

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

Support Free Java!
Contribute to GNU Classpath and the OpenJDK
http://www.gnu.org/software/classpath
http://openjdk.java.net

PGP Key: 94EFD9D8 (http://subkeys.pgp.net)
Fingerprint: F8EF F1EA 401E 2E60 15FA  7927 142C 2591 94EF D9D8