Mark Hindess wrote:
Since most (all?) distributions provide versions of these libraries (and
maintain them - "regular" security fixes for example) why would we want
to maintain them ourselves?  It's not a job I'd want.

I'm not advocating maintenance, but simply defining the set that we build and test with.

I'm not a fan of having certified builds be dependent on whatever random stuff the user downloads to /usr/lib


Really the same is true for zlib and to an extent icu.  If someone
else is doing the work maintaining them, we should use what they are a
providing not make more work for ourselves.  We should also try to use
the dynamic libraries if possible.

Yes, on the first sentence, not so sure on the last, simply because there's value in testing a fixed set of versions of stuff.


With the exception of icu, most of these libraries change very little
over time so there should be few if any interoperability issues.

I can only really see a good argument for maintaining icu binaries since
it is changing more frequently and many distributions seem to have
rather old versions.

Regards,
 Mark.

On 4 October 2006 at 10:40, "Geir Magnusson Jr." <[EMAIL PROTECTED]> wrote:
Any reason why we couldn't do the same thing for linux that we're doing for windows in terms of having these libraries pre-compiled and easy to drop in?

geir

Mark Hindess wrote:
With this change, the awt dependencies should now be automated for
windows and at least fairly trivial (installing a few packages on
Linux[0]).  I think it is time we removed the with.awt.swing flag.
Anyone object?

Please test the current setup with -Dwith.awt.swing=true and report any
problems.

Regards,
 Mark.

[0] Details of the required packages for distributions other than
Debian/Ubuntu would be welcome.

On 4 October 2006 at 10:24, [EMAIL PROTECTED] wrote:
Author: hindessm
Date: Wed Oct  4 03:24:29 2006
New Revision: 452826

URL: http://svn.apache.org/viewvc?view=rev&rev=452826
Log:
Update check/fetch depends targets to handle the awt dependencies.

Modified:
    incubator/harmony/enhanced/classlib/trunk/depends/libs/build/   (props
 ch
anged)
incubator/harmony/enhanced/classlib/trunk/depends/libs/windows.x86/
(pr
ops changed)
    incubator/harmony/enhanced/classlib/trunk/make/depends.properties
    incubator/harmony/enhanced/classlib/trunk/make/depends.xml

Propchange: incubator/harmony/enhanced/classlib/trunk/depends/libs/build/
--------------------------------------------------------------------------
---
-
--- svn:ignore (original)
+++ svn:ignore Wed Oct  4 03:24:29 2006
@@ -1,3 +1,4 @@
 jpeg
 lcms
 png
+winxp_2006-09-28.txt

Propchange: incubator/harmony/enhanced/classlib/trunk/depends/libs/windows
.x8
6/
--------------------------------------------------------------------------
---
-
--- svn:ignore (original)
+++ svn:ignore Wed Oct  4 03:24:29 2006
@@ -1 +1,2 @@
 msvcr71.dll
+swing_awt_deps_winxp_2006-09-28.tgz

Modified: incubator/harmony/enhanced/classlib/trunk/make/depends.propertie
s
URL: http://svn.apache.org/viewvc/incubator/harmony/enhanced/classlib/trun
k/m
ake/depends.properties?view=diff&rev=452826&r1=452825&r2=452826
==========================================================================
===
=
--- incubator/harmony/enhanced/classlib/trunk/make/depends.properties (ori
gin
al)
+++ incubator/harmony/enhanced/classlib/trunk/make/depends.properties Wed
Oct
  4 03:24:29 2006
@@ -98,3 +98,11 @@
 servlet-api.jar=${jetty.dir}/servlet-api-2.5-6.0.0.jar
 servlet-api.md5=c27c02fb0a00cc3a7d05ea993a9bf56e
 servlet-api.url=${ibiblio.base}/maven2/jetty/servlet-api/2.5-6.0.0/servle
t-a
pi-2.5-6.0.0.jar
+
+people.apache.base=http://people.apache.org/~geirm/harmony/
+awtdeps.dir=${depends.dir}/libs/windows.x86
+awtdeps.tar=${awtdeps.dir}/swing_awt_deps_winxp_2006-09-28.tgz
+awtdeps.url=${people.apache.base}swing_awt_deps_winxp_2006-09-28.tgz
+awtdeps.md5=d61a27e4b305d9fcabaaacf34f8f534a
+awtdeps.extract.dir=${depends.dir}/libs/build
+awtdeps.testfile=${awtdeps.extract.dir}/winxp_2006-09-28.txt

Modified: incubator/harmony/enhanced/classlib/trunk/make/depends.xml
URL: http://svn.apache.org/viewvc/incubator/harmony/enhanced/classlib/trun
k/m
ake/depends.xml?view=diff&rev=452826&r1=452825&r2=452826
==========================================================================
===
=
--- incubator/harmony/enhanced/classlib/trunk/make/depends.xml (original)
+++ incubator/harmony/enhanced/classlib/trunk/make/depends.xml Wed Oct 4
03:
24:29 2006
@@ -72,17 +72,22 @@
</target> - <target name="-check-win" if="is.windows">
+    <target name="-check-win" if="is.windows"
+            depends="-really-check-win,-awt-tar-extract" />
+
+    <target name="-really-check-win" if="is.windows">
<check-one-file src="${msvcr71.url}" dest="${msvcr71.dll}" /> - </target>
+        <check-one-file src="${awtdeps.url}" dest="${awtdeps.tar}" />
+
+       <uptodate property="awtdeps.uptodate"
+                  srcfile="${awtdeps.tar}"
+                  targetfile="${awtdeps.testfile}" />
- <target name="-check-unix" if="with.awt.swing">
-            <antcall target="--check-unix" />
     </target>
- <target name="--check-unix" if="is.unix">
+    <target name="-check-unix" if="is.unix">
<property name="lcms.msg"
                   value="liblcms development package not installed
@@ -214,6 +219,10 @@
        <download-one-file src="${msvcr71.url}" dest="${msvcr71.dll}"
                            md5="${msvcr71.md5}" />
+ <mkdir dir="${awtdeps.dir}" />
+       <download-one-file src="${awtdeps.url}" dest="${awtdeps.tar}"
+                           md5="${awtdeps.md5}" />
+
     </target>
<macrodef name="download-one-file">
@@ -298,6 +307,14 @@
        <jar destfile="${bcprov.jar}" basedir="${bcprov.dir}/temp"
              manifest="${bcprov.dir}/temp/META-INF/MANIFEST.MF" />
        <delete dir="${bcprov.dir}/temp" />
+    </target>
+
+    <target name="-awt-tar-extract" unless="awtdeps.uptodate">
+        <echo>Extracting awt dependencies</echo>
+       <untar src="${awtdeps.tar}" dest="${awtdeps.extract.dir}"
+               compression="gzip" />
+        <echo file="${awtdeps.testfile}"
+              message="${awtdeps.tar} extracted${line.separator}" />
     </target>
<macrodef name="check-one-link">



---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]




---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to