<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "http://bugs.gentoo.org/bugzilla.dtd">

<bugzilla version="2.22.7"
          urlbase="http://bugs.gentoo.org/"
          maintainer="bugzilla@gentoo.org"
>

    <bug>
          <bug_id>145568</bug_id>
          
          <creation_ts>2006-08-30 02:47 0000</creation_ts>
          <short_desc>dev-java/fop-0.20.5-r6 build.xml uses automagical tests to determine whether build optional stuff</short_desc>
          <delta_ts>2006-09-10 13:56:32 0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>Gentoo Linux</product>
          <component>Ebuilds</component>
          <version>unspecified</version>
          <rep_platform>All</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          
          <priority>P2</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          
          <everconfirmed>1</everconfirmed>
          <reporter>caster@gentoo.org</reporter>
          <assigned_to>java@gentoo.org</assigned_to>
          

      

      
          <long_desc isprivate="0">
            <who>caster@gentoo.org</who>
            <bug_when>2006-08-30 02:47:09 0000</bug_when>
            <thetext>It tests presence of the following on classpath:
jimi (sun-jimi), jai (sun-jai-bin), trax (xalan), jdk14, jce

This is bad because jimi and jai are controlled by useflags (linking appropriate jars into lib/ dir based on the flag), but the autodetection can mess with it because ant can be using jai itself, and that can result in (with USE=&quot;-jai&quot; for fop):
- java-strict is on, but the autodetection will still see jai on classpath when doing the test (-Dbuild.sysclasspath=ignore doesn&apos;t affect this, sadly), during compiling jai will be hidden and build will fail
- java-strict is off, it will build jai support because it&apos;s in ant&apos;s classpath, but it won&apos;t be recorded as dep in fop&apos;s package.env

One solution can be to put ignoresystemclasspath=&quot;true&quot; parameter in &lt;available&gt; tags in build.xml used to test jai and jimi presence. This will ensure that jai and jimi support will be checked only in lib/ dir, not on classpath ant uses. This could be done even without patching, by running xml-rewrite.py from ebuild. But since there&apos;s already a build.xml patch, I expanded it to do the following:

Instead of testing for jdk14 and jce, it enables is unconditionally. Also changed the dep from || (jdk1.4 jdk-1.3) to just jdk-1.4.
Same with the trax testing, it&apos;s just enabled, because xalan is unconditional dep anyway.
Tests for jai and jimi are removed and the associated properties are not set in build.xml at all. Ebuild passes these properties based on useflags.
No automagical detection is left :)</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>caster@gentoo.org</who>
            <bug_when>2006-08-30 02:48:37 0000</bug_when>
            <thetext>Created an attachment (id=95444)
new patch for build.xml

</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>caster@gentoo.org</who>
            <bug_when>2006-08-30 02:50:14 0000</bug_when>
            <thetext>Created an attachment (id=95447)
patch for fop&apos;s ebuild

</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>caster@gentoo.org</who>
            <bug_when>2006-09-10 13:56:32 0000</bug_when>
            <thetext>Released in fop-0.20.5-r7
</thetext>
          </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>95444</attachid>
            <date>2006-08-30 02:48 0000</date>
            <desc>new patch for build.xml</desc>
            <filename>0.20.5-buildxml.patch</filename>
            <type>text/plain</type>
            <data encoding="base64">LS0tIGJ1aWxkLnhtbC5vcmlnCTIwMDMtMDctMTUgMDQ6MDM6MjIuMDAwMDAwMDAwICswMjAwCisr
KyBidWlsZC54bWwJMjAwNi0wOC0zMCAxMTozMDoyMi4wMDAwMDAwMDAgKzAyMDAKQEAgLTE3Miwx
NiArMTcyLDcgQEAKICAgPC9maWxlc2V0PgogCiAgIDxwYXRoIGlkPSJsaWJzLWJ1aWxkLWNsYXNz
cGF0aCI+Ci0gICAgPGZpbGVzZXQgZGlyPSJsaWIiPgotICAgICAgPGluY2x1ZGUgbmFtZT0ieGFs
YW4qLmphciIvPgotICAgICAgPGluY2x1ZGUgbmFtZT0ieGVyY2VzKi5qYXIiLz4KLSAgICAgIDxp
bmNsdWRlIG5hbWU9InhtbC1hcGlzLmphciIvPgotICAgICAgPGluY2x1ZGUgbmFtZT0iYXZhbG9u
LWZyYW1ld29yayouamFyIi8+Ci0gICAgICA8aW5jbHVkZSBuYW1lPSJiYXRpayouamFyIi8+Ci0g
ICAgICA8aW5jbHVkZSBuYW1lPSJqaW1pKi5qYXIiLz4KLSAgICAgIDxpbmNsdWRlIG5hbWU9Imph
aSouamFyIi8+Ci0gICAgICA8aW5jbHVkZSBuYW1lPSJqY2UqLmphciIvPgotICAgIDwvZmlsZXNl
dD4KKyAgICA8ZmlsZXNldCBkaXI9ImxpYiIgaW5jbHVkZXM9IioqLyouamFyIi8+CiAgIDwvcGF0
aD4KICAgCiAgIDwhLS0gPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PSAtLT4KQEAgLTI5MywxMSArMjg0LDkgQEAKICAgPC90
YXJnZXQ+CiAKICAgPHRhcmdldCBuYW1lPSJpbml0LWF2YWlsIj4KLSAgICA8YXZhaWxhYmxlIHBy
b3BlcnR5PSJqaW1pLnByZXNlbnQiIGNsYXNzbmFtZT0iY29tLnN1bi5qaW1pLmNvcmUuSmltaSIg
Y2xhc3NwYXRocmVmPSJsaWJzLWJ1aWxkLWNsYXNzcGF0aCIvPgotICAgIDxhdmFpbGFibGUgcHJv
cGVydHk9ImphaS5wcmVzZW50IiBjbGFzc25hbWU9ImphdmF4Lm1lZGlhLmphaS5KQUkiIGNsYXNz
cGF0aHJlZj0ibGlicy1idWlsZC1jbGFzc3BhdGgiLz4KLSAgICA8YXZhaWxhYmxlIHByb3BlcnR5
PSJ0cmF4LnByZXNlbnQiIGNsYXNzbmFtZT0iamF2YXgueG1sLnRyYW5zZm9ybS5UcmFuc2Zvcm1l
ciIgY2xhc3NwYXRocmVmPSJsaWJzLWJ1aWxkLWNsYXNzcGF0aCIvPgotICAgIDxhdmFpbGFibGUg
cHJvcGVydHk9ImpkazE0LnByZXNlbnQiIGNsYXNzbmFtZT0iamF2YS5sYW5nLkNoYXJTZXF1ZW5j
ZSIvPgotICAgIDxhdmFpbGFibGUgcHJvcGVydHk9ImpjZS5wcmVzZW50IiBjbGFzc25hbWU9Imph
dmF4LmNyeXB0by5DaXBoZXIiIGNsYXNzcGF0aHJlZj0ibGlicy1idWlsZC1jbGFzc3BhdGgiLz4K
KyAgICA8cHJvcGVydHkgbmFtZT0idHJheC5wcmVzZW50IiB2YWx1ZT0idHJ1ZSIvPgorICAgIDxw
cm9wZXJ0eSBuYW1lPSJqZGsxNC5wcmVzZW50IiB2YWx1ZT0idHJ1ZSIvPgorICAgIDxwcm9wZXJ0
eSBuYW1lPSJqY2UucHJlc2VudCIgdmFsdWU9InRydWUiLz4KICAgPC90YXJnZXQ+CiAKICAgPHRh
cmdldCBuYW1lPSJpbml0LWZpbHRlcnMtamRrMTMiIGRlcGVuZHM9ImluaXQtYXZhaWwiIHVubGVz
cz0iamRrMTQucHJlc2VudCI+CkBAIC01ODgsNyArNTc3LDYgQEAKICAgICAgICAgIGluY2x1ZGVz
PSJvcmcvKiosY29uZi8qKixoeXBoLyoqIj4KICAgICA8bWFuaWZlc3Q+CiAgICAgICAgIDxhdHRy
aWJ1dGUgbmFtZT0iTWFpbi1DbGFzcyIgdmFsdWU9IiR7bWFpbi5jbGFzc30iLz4KLSAgICAgICAg
PGF0dHJpYnV0ZSBuYW1lPSJDbGFzcy1QYXRoIiB2YWx1ZT0iJHtydW50aW1lLmNsYXNzcGF0aH0i
Lz4KICAgICAgICAgPGF0dHJpYnV0ZSBuYW1lPSJJbXBsZW1lbnRhdGlvbi1UaXRsZSIgdmFsdWU9
IiR7TmFtZX0iLz4KICAgICAgICAgPGF0dHJpYnV0ZSBuYW1lPSJJbXBsZW1lbnRhdGlvbi1WZXJz
aW9uIiB2YWx1ZT0iJHt2ZXJzaW9ufSIvPgogICAgICAgICA8YXR0cmlidXRlIG5hbWU9IkltcGxl
bWVudGF0aW9uLVZlbmRvciIgdmFsdWU9IkFwYWNoZSBTb2Z0d2FyZSBGb3VuZGF0aW9uIChodHRw
Oi8veG1sLmFwYWNoZS5vcmcvZm9wLykiLz4K
</data>        

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>95447</attachid>
            <date>2006-08-30 02:50 0000</date>
            <desc>patch for fop&apos;s ebuild</desc>
            <filename>fop-0.20.5-r6.ebuild.patch</filename>
            <type>text/plain</type>
            <data encoding="base64">LS0tIGZvcC0wLjIwLjUtcjYuZWJ1aWxkCTIwMDYtMDgtMDYgMDU6NDA6MjYuMDAwMDAwMDAwICsw
MjAwCisrKyBmb3AtMC4yMC41LXI2LmVidWlsZAkyMDA2LTA4LTMwIDEwOjIxOjE0LjAwMDAwMDAw
MCArMDIwMApAQCAtMjIsOSArMjIsOSBAQAogCWRldi1qYXZhL3hhbGFuCiAJPj1kZXYtamF2YS94
ZXJjZXMtMi42LjIKIAkhZGV2LWphdmEvZm9wLWJpbiIKLVJERVBFTkQ9Inx8ICggPXZpcnR1YWwv
anJlLTEuNCogPXZpcnR1YWwvanJlLTEuMyogKQorUkRFUEVORD0iKCA9dmlydHVhbC9qcmUtMS40
KiApCiAJJHtDT01NT05fREVQfSIKLURFUEVORD0ifHwgKCA9dmlydHVhbC9qZGstMS40KiA9dmly
dHVhbC9qZGstMS4zKiApCitERVBFTkQ9Inx8ICggPXZpcnR1YWwvamRrLTEuNCogKQogCSR7Q09N
TU9OX0RFUH0KIAk+PWRldi1qYXZhL2FudC0xLjUuNCIKIApAQCAtNTAsNyArNTAsMTIgQEAKIHNy
Y19jb21waWxlKCkgewogCWphdmEtcGtnX2ZpbHRlci1jb21waWxlciBqaWtlcwogCi0JZWFudCBw
YWNrYWdlICQodXNlX2RvYyBqYXZhZG9jcykKKwlsb2NhbCBqYWlwIGppbWlwCisJCisJdXNlIGph
aSAmJiBqYWlwPSItRGphaS5wcmVzZW50PXRydWUiCisJdXNlIGppbWkgJiYgamltaXA9Ii1Eamlt
aS5wcmVzZW50PXRydWUiCisKKwllYW50ICR7amFpcH0gJHtqaW1pcH0gcGFja2FnZSAkKHVzZV9k
b2MgamF2YWRvY3MpCiB9CiAKIHNyY19pbnN0YWxsKCkgewo=
</data>        

          </attachment>
    </bug>

</bugzilla>