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 >
