Re: svn commit: r570339 - /harmony/enhanced/buildtest/branches/2.0/adaptors/classlib-test/adaptor.xml

[Tim Ellison <t....@gmail.com>]

Stepan,

I don't see the full context of the change made here, so please correct
me if I'm wrong about what you did...

Remember that we cannot invoke "fetch-depends" as the default action of
running a build / test script etc. since the licenses for our
dependencies include things other than ALv2.

Retrieving these alternatively licensed pieces must require "an explicit
action by the user".

Regards,
Tim


smishura@apache.org wrote:
> Author: smishura
> Date: Tue Aug 28 00:40:05 2007
> New Revision: 570339
> 
> URL: http://svn.apache.org/viewvc?rev=570339&view=rev
> Log:
> Classlib tests: add 'fetch-depends' target before running tests
> 
> Modified:
>     harmony/enhanced/buildtest/branches/2.0/adaptors/classlib-test/adaptor.xml
> 
> Modified: harmony/enhanced/buildtest/branches/2.0/adaptors/classlib-test/adaptor.xml
> URL: http://svn.apache.org/viewvc/harmony/enhanced/buildtest/branches/2.0/adaptors/classlib-test/adaptor.xml?rev=570339&r1=570338&r2=570339&view=diff
> ==============================================================================
> --- harmony/enhanced/buildtest/branches/2.0/adaptors/classlib-test/adaptor.xml (original)
> +++ harmony/enhanced/buildtest/branches/2.0/adaptors/classlib-test/adaptor.xml Tue Aug 28 00:40:05 2007
> @@ -44,7 +44,7 @@
>          </tstamp>
>  
>          <exec-ant dir="${classlib-test.trunk.dir}"
> -                  targets="test"
> +                  targets="fetch-depends test"
>                    failonerror="false"
>                    failproperty="classlib-test.run.failed">
>              <jvmarg line="-Xmx800M"/>
> 
> 
> 

Re: svn commit: r570339 - /harmony/enhanced/buildtest/branches/2.0/adaptors/classlib-test/adaptor.xml

[Stepan Mishura <st...@gmail.com>]

On 8/29/07, Tim Ellison wrote:
> Stepan,
>
> I don't see the full context of the change made here, so please correct
> me if I'm wrong about what you did...
>
> Remember that we cannot invoke "fetch-depends" as the default action of
> running a build / test script etc. since the licenses for our
> dependencies include things other than ALv2.
>
> Retrieving these alternatively licensed pieces must require "an explicit
> action by the user".
>

Tim,

It depends how to interpret 'product' in the policy[1]. When I read
the policy my understanding was that we couldn't treat BTI infra as a
product - no infra releases, bundles, NOTICE file and so on.

When you raised the question about the commit I reread the policy and
thought that may be I were wrong - I didn't find exact 'product's
definition so we can defined any peace of code developed by Apache as
a product. In this case we should add an option to BTI infra and
update all adaptors.

[1] http://people.apache.org/~cliffs/3party.html

Thanks,
Stepan.

> Regards,
> Tim
>
>
> smishura@apache.org wrote:
> > Author: smishura
> > Date: Tue Aug 28 00:40:05 2007
> > New Revision: 570339
> >
> > URL: http://svn.apache.org/viewvc?rev=570339&view=rev
> > Log:
> > Classlib tests: add 'fetch-depends' target before running tests
> >
> > Modified:
> >     harmony/enhanced/buildtest/branches/2.0/adaptors/classlib-test/adaptor.xml
> >
> > Modified: harmony/enhanced/buildtest/branches/2.0/adaptors/classlib-test/adaptor.xml
> > URL: http://svn.apache.org/viewvc/harmony/enhanced/buildtest/branches/2.0/adaptors/classlib-test/adaptor.xml?rev=570339&r1=570338&r2=570339&view=diff
> > ==============================================================================
> > --- harmony/enhanced/buildtest/branches/2.0/adaptors/classlib-test/adaptor.xml (original)
> > +++ harmony/enhanced/buildtest/branches/2.0/adaptors/classlib-test/adaptor.xml Tue Aug 28 00:40:05 2007
> > @@ -44,7 +44,7 @@
> >          </tstamp>
> >
> >          <exec-ant dir="${classlib-test.trunk.dir}"
> > -                  targets="test"
> > +                  targets="fetch-depends test"
> >                    failonerror="false"
> >                    failproperty="classlib-test.run.failed">
> >              <jvmarg line="-Xmx800M"/>
> >