harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alexei Fedotov" <alexei.fedo...@gmail.com>
Subject Re: [build-test] new ant target
Date Wed, 26 Mar 2008 10:22:48 GMT
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

Mime
View raw message