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 <[EMAIL PROTECTED]> wrote: > On 3/25/08, Alexei Zakharov <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]>: > > > On 3/22/08, Stepan Mishura <[EMAIL PROTECTED]> wrote: > > > > On 3/21/08, Alexei Zakharov <[EMAIL PROTECTED]> 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
