harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alexei Zakharov" <alexei.zakha...@gmail.com>
Subject Re: [build-test] new ant target
Date Wed, 26 Mar 2008 12:23:05 GMT
Yeah, forgot to say this in my previous message: +1 for Stepan's solution.

Regards,
Alexei

2008/3/26, Alexei Fedotov <alexei.fedotov@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
>

Mime
View raw message