[MERGE][bug #102019] strace hanging the test suite

Martin Pool mbp at sourcefrog.net
Tue Jul 3 10:50:02 BST 2007


On 7/3/07, Vincent Ladeuil <v.ladeuil+lp at free.fr> wrote:

>     >> def strace_detailed(function, args, kwargs, follow_children=True):
>     >> ...
>     >>
>     >> and strace becomes
>     >> def strace(function, *args, **kwargs):
>     >> return strace_detailed(function, args, kwargs)
>
> Thanks for the hint, I didn't know how to do it in python.
>
>     >>
>     >> and the tests that test executing strace can do
>     >> strace_detailed calls rather than strace calls.
>
>     mbp> I agree with Robert that this style is nicer.
>
> strace_detailed looks ugly to my eyes, but here is a patch that
> implements your suggestion.

It's more explicit - it keeps separate that "this is passed to the
traced function" and "this controls the tracing".  I agree the
declaration is uglier.

>
>     mbp> I know there are cases in the test suite where we mix
>     mbp> them together and that's why I have this opinion.
>
> I can't find them or I do not understand what you are talking about ;)

I mean, that run_bzr has kwargs some of which control the running and
some of which are passed through.  Or at least it used to.

>
>     mbp> However, it seems to me that using strace -f on bzr is
>     mbp> likely to cause hangs in regular operation just as in
>     mbp> the test suite.  Therefore I don't see it as urgent to
>     mbp> support it until it actually works properly.

Yes, you're right, I'd forgotten it was only the presence of threads
that provokes the problem.

-- 
Martin



More information about the bazaar mailing list