harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Geir Magnusson Jr." <g...@pobox.com>
Subject Re: svn commit: r452826 - in /incubator/harmony/enhanced/classlib/trunk: depends/libs/build/ depends/libs/windows.x86/ make/depends.properties make/depends.xml
Date Wed, 04 Oct 2006 15:30:42 GMT


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." <geir@pobox.com> 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, hindessm@apache.org 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: harmony-dev-unsubscribe@incubator.apache.org
>>> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>>>
>> ---------------------------------------------------------------------
>> Terms of use : http://incubator.apache.org/harmony/mailing.html
>> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
>> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>>
> 
> 
> 
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> 

---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org


Mime
View raw message