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.
