Mailing-List: contact dev-help@harmony.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@harmony.apache.org
Received-SPF: pass (athena.apache.org: domain of alexei.fedotov@gmail.com
 designates 209.85.162.181 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=beta;
        h=message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references;
        b=aT0NZ4Z0FsQ+QmRBBrogT/6wSZ8LT/lPRmcVJE1aEEz+xVmOovKcIayjgFi+C9QnMO1YbIbags3kF4J3due5q/wDoh9EGCW6he0YSpc/AqZ7EMVKGWSP+YMEozrR+AGpsY8MEfu9QVqGkR5cRr3hEP0M45Y/iffy29x6ArGwTBE=
Message-ID: <c30d70960803260322i4969abcfka2a2ee6ff85e657f@mail.gmail.com>
Date: Wed, 26 Mar 2008 13:22:48 +0300
From: "Alexei Fedotov" <alexei.fedotov@gmail.com>
To: dev@harmony.apache.org
Subject: Re: [build-test] new ant target
In-Reply-To: <6e47b64f0803252204r17b50533ydfcfc7c42928be0d@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
References: <2c9597b90803210525q170aebecq47540bf831c530bb@mail.gmail.com>
	 <6e47b64f0803211112j5c3ad986q94812801998a5f66@mail.gmail.com>
	 <6e47b64f0803250138k7553d0c9uc2988b68986cdfe8@mail.gmail.com>
	 <2c9597b90803250611r383ea68etc9de4e0640275eef@mail.gmail.com>
	 <6e47b64f0803252204r17b50533ydfcfc7c42928be0d@mail.gmail.com>

Stepan,
I like your idea about flexibility. It satisfies both use cases.

With best regards, Alexei

On Wed, Mar 26, 2008 at 8:04 AM, Stepan Mishura
<stepan.mishura@gmail.com> wrote:
> On 3/25/08, Alexei Zakharov <alexei.zakharov@gmail.com> wrote:
>  > Thanks Stepan. By the way, I also noticed that by default all test
>  > sources are downloaded from
>  >
>  > http://svn.apache.org/repos/....
>  >
>
>  HTTP is used intentionally - it was assumed that on a test system
>  there is no need in committing updates for test suites.
>
>  But I think you are right - the infra should be more flexible and let
>  committer do her/his work without additional (i.e. switch --relocate)
>  steps. I think that the infra should follow federated build's way. So
>  if the infra is checked out using HTTPS then tests sources are also
>  checked out via HTTPS.
>
>  Thanks,
>  Stepan.
>
>
>
>  > This is a bit inconvenient if you are a committer and trying to commit
>  > changes in tests. Because svn refuses to commit anything if the
>  > repository URL doesn't start with HTTPS. So yesterday I had to
>  > recheckout all sources in order to commit a trivial change to struts
>  > tests.
>  >
>  > Thanks,
>  > Alexei
>  >
>  > 2008/3/25, Stepan Mishura <stepan.mishura@gmail.com>:
>  > > On 3/22/08, Stepan Mishura <stepan.mishura@gmail.com> wrote:
>  > >  > On 3/21/08, Alexei Zakharov <alexei.zakharov@gmail.com> wrote:
>  > >  > > Hi Stepan,
>  > >  > >
>  > >  > > Today I was trying to update the buildtest workspace and found that
>  > >  > > you've made several changes to it. I tried to read README.txt but this
>  > >  > > was not very helpful since the file itself looks outdated. Then from
>  > >  > > svn logs I understand that you've added new 'install' target to main
>  > >  > > build.xml . So what is the correct sequence now
>  > >  > >
>  > >  > > ant install
>  > >  > > ant -Dtest.suites=struts_test setup
>  > >  > > ant -Dtest.suites=struts_test run
>  > >  > >
>  > >  >
>  > >  > Yes, this is correct sequence now, but install should has
>  > >  > -Dtest.suites=struts_test param too.
>  > >  >
>  > >  > I'm sorry that I didn't warned about the change and not updated
>  > >  > README. Currently I'm working on infra's improvements to make work
>  > >  > flow more simple/transparent and resolve several issues mentioned on
>  > >  > the dev-list. Please be patient to me if I broke something or the doc
>  > >  > is inconsistent.
>  > >
>  > >
>  > > BTW, if you find something broken please let me know.
>  > >  Also in this case you may use BTI's svn revision < 639550 - it should work.
>  > >
>  > >  Thanks,
>  > >
>  > > Stepan.
>  > >
>  > >
>  > >  > After I complete improvements I'm going to align the
>  > >  > README and the page on the web-site.
>  > >  >
>  > >  > Thanks,
>  > >  > Stepan.
>  > >  >
>  > >  > > ?
>  > >  > >
>  > >  > > Regards,
>  > >  > > Alexei
>  >
>


-- 
With best regards,
Alexei