On Thu, Sep 8, 2011 at 6:10 PM, Gustavo Niemeyer <email address hidden>wrote:
> Review: Approve
>
> Looks great, thanks Jim. Only a couple of minor comments, and it can go in
> IMO.
>
> Then, for the driver! I hope we can see a waterfall working this week
> still!
>
Please see the corresponding butler branch in review
>
> [4]
>
> === added file 'pystache-LICENSE'
>
> Please rename this to LICENSE and put it within the pystache dir itself.
>
Renamed accordingly
>
> [5]
>
> +OUTPUT_STATUS_RE =
> re.compile(r"(?P<name>.*?)(?P<status>\.out.FAILED|\.out)")
>
> This regex makes things more complex than it helps. E.g.:
>
> FAILED_SUFFIX = ".out.FAILED"
> OK_SUFFIX = ".out"
> (...)
> if name.endswith(FAILED_SUFFIX):
> test_name = name[:-len(FAILED_SUFFIX)]
>
> etc.
>
>
Done
On Thu, Sep 8, 2011 at 6:10 PM, Gustavo Niemeyer <email address hidden>wrote:
> Review: Approve
>
> Looks great, thanks Jim. Only a couple of minor comments, and it can go in
> IMO.
>
> Then, for the driver! I hope we can see a waterfall working this week
> still!
>
Please see the corresponding butler branch in review
>
> [4]
>
> === added file 'pystache-LICENSE'
>
> Please rename this to LICENSE and put it within the pystache dir itself.
>
Renamed accordingly
> r"(?P<name> .*?)(?P< status> \.out.FAILED| \.out)" ) FAILED_ SUFFIX) : len(FAILED_ SUFFIX) ]
> [5]
>
> +OUTPUT_STATUS_RE =
> re.compile(
>
> This regex makes things more complex than it helps. E.g.:
>
> FAILED_SUFFIX = ".out.FAILED"
> OK_SUFFIX = ".out"
> (...)
> if name.endswith(
> test_name = name[:-
>
> etc.
>
>
Done