On Thu, Apr 28, 2011 at 3:42 AM, Tom Bachmann <ness...@googlemail.com> wrote: >> >> This is the first step to create a fully automatic webservice. It will >> require some time and resources, but with this current implementation >> of sympy-bot, it is already useful and all of you can send patches to >> it to improve it. >> > > This sounds really good. > >> 2) run "./setup.py test" >> > > That should be easy I suppose. The following should do it, but I have > no time to test right now: > > diff --git a/bin/sympy-next.py b/bin/sympy-next.py > index 111d793..9ee556c 100755 > --- a/bin/sympy-next.py > +++ b/bin/sympy-next.py > @@ -52,7 +52,7 @@ def read_branchfile(f): > > def run_tests(): > logit("Running unit tests.") > - out = subprocess.Popen(["./bin/test"], > stdout=subprocess.PIPE).stdout > + out = subprocess.Popen(["./setup.py", "test"], > stdout=subprocess.PIPE).stdout > > report = [] > def my_join(file): > @@ -69,10 +69,15 @@ def my_split(iter): > buf = '' > for c in iter: > buf += c > + # TODO make this nicer > if buf.endswith('sympy/'): > r = buf[:-6] > buf = '' > yield r > + if buf.endswith('doc/'): > + r = buf[:-4] > + buf = '' > + yield r > for line in my_split(my_join(out)): > good = None > if line.endswith('[OK]\n'):
Thanks for the patch. I need to go to bed now. If you could send a pull request and test it, that'd be great. > >> 3) if the merge fails, still report it, and why it failed. Currently >> sympy-next fails to generate the report, so sympy-bot submits an empty >> pastebin report as well. >> > > I cannot reproduce that. > With the following branches file: > > https://github.com/sympy/sympy.git master > https://github.com/goodok/sympy.git 821_Max > > I get the following log: [...] Yes, but for some reason, the log in the "out" directory is empty, or none. I use the following code in sympy-bot to submit the report: print "> Uploading test results" s = cmd("other/pastebin.py %s/out/2* -n sympy-bot --public -e N" % \ tmpdir, capture=True) url = s.split()[-1] print "> Uploaded at: %s" % url so the file .../out/2* must exist. It'd be nice to make sympy-next generate a simple file as a report, not a filename as a date, since it's hard to read it from sympy-bot. Ondrej Ondrej -- You received this message because you are subscribed to the Google Groups "sympy" group. To post to this group, send email to sympy@googlegroups.com. To unsubscribe from this group, send email to sympy+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/sympy?hl=en.