[RFC][icedtea6-1.9, icedtea6-1.10] Backport for PR732: Use xsltproc for bootstrap xslt in place of Xerces/Xalan

[Dr Andrew John Hughes]

On 14:57 Tue 20 Dec     , Danesh Dadachanji wrote:
> Adding CC. =)
> 
> On 15/11/11 04:54 PM, Danesh Dadachanji wrote:
> > Ping?
> >
> > On 26/10/11 04:13 PM, Danesh Dadachanji wrote:
> >> Hi,
> >>
> >> Attached are patches for 6-1.9 and 6-1.10 to replace the use of
> >> xalan/xerces with xsltproc.
> >>
> >> It is pretty similar to PR732's fix[1] with some minor changes in
> >> syntax/line numbers for diffs:
> >> - The changeset from HEAD that added
> >> patches/ecj/endorsed-dir-for-jvmti.patch was never backported so I did
> >> not need to deal with it.
> >> - I had to adjust patches/ecj/xsltproc.patch to get it to apply,
> >> jvmti.make is a bit offset when compared to HEAD's build.
> >> - For 1.9, the configure.ac macros in question do not start with "IT_"
> >> (e.g. FIND_TOOL vs IT_FIND_TOOL).
> >>
> >> 1.10 ChangeLog:
> >> +2011-10-26 Danesh Dadachanji <ddadacha at redhat.com>
> >> +
> >> + * Makefile.am:
> >> + (ENDORSED_DIR): Removed.
> >> + (ICEDTEA_ECJ_PATCHES): Add xsltproc.patch.
> >> + (ICEDTEA_ENV_ECJ): Set XSLT. Don't
> >> + set ICEDTEA_ENDORSED_DIR or ENDORSED.
> >> + (bootstrap-directory-stage1): Don't link
> >> + xalan2, xerces2 or xalan2 serialiser.
> >> + * acinclude.m4:
> >> + (IT_FIND_XALAN2_JAR): Removed.
> >> + (IT_FIND_XALAN2_SERIALIZER_JAR): Removed.
> >> + (IT_FIND_XERCES2_JAR): Removed.
> >> + * configure.ac:
> >> + Replace calls to FIND_XALAN2_JAR,
> >> + IT_FIND_XALAN2_SERIALIZER_JAR and
> >> + IT_FIND_XERCES2_JAR with call to
> >> + IT_FIND_TOOL for xsltproc.
> >> + * patches/ecj/xsltproc.patch:
> >> + Patch jvmti.make in HotSpot to use
> >> + xsltproc instead of Java's XSLT
> >> + when bootstrapping.
> >> + * NEWS: Updated.
> >>
> >>
> >> 1.9 ChangeLog:
> >> +2011-10-26 Danesh Dadachanji <ddadacha at redhat.com>
> >> +
> >> + * Makefile.am:
> >> + (ENDORSED_DIR): Removed.
> >> + (ICEDTEA_ECJ_PATCHES): Add xsltproc.patch.
> >> + (ICEDTEA_ENV_ECJ): Set XSLT. Don't
> >> + set ICEDTEA_ENDORSED_DIR or ENDORSED.
> >> + (bootstrap-directory-stage1): Don't link
> >> + xalan2, xerces2 or xalan2 serialiser.
> >> + * acinclude.m4:
> >> + (FIND_XALAN2_JAR): Removed.
> >> + (FIND_XALAN2_SERIALIZER_JAR): Removed.
> >> + (FIND_XERCES2_JAR): Removed.
> >> + * configure.ac:
> >> + Replace calls to FIND_XALAN2_JAR,
> >> + FIND_XALAN2_SERIALIZER_JAR and
> >> + FIND_XERCES2_JAR with call to
> >> + FIND_TOOL for xsltproc.
> >> + * patches/ecj/xsltproc.patch:
> >> + Patch jvmti.make in HotSpot to use
> >> + xsltproc instead of Java's XSLT
> >> + when bootstrapping.
> >> + * NEWS: Updated.
> >>
> >> Regards,
> >> Danesh
> >> [1] http://icedtea.classpath.org/hg/icedtea6/rev/32fa8c401cee

What's the status of this?

I don't think we should make such extensive build changes on a release branch.
-- 
Andrew :)

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

PGP Key: 248BDC07 (https://keys.indymedia.org/)
Fingerprint = EC5A 1F5E C0AD 1D15 8F1F  8F91 3B96 A578 248B DC07
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20111221/bab72216/attachment.bin