Code review comment for lp:~mbp/bzr/626679-strace
Revision history for this message
| Andrew Bennetts (spiv) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
Martin Pool wrote:
> Thanks.
>
> That actually pushes me a bit more towards deleting/disabling the
> tests, but leaving the facility. It's useful to have it, but it seems
> silly to have test hangs for something that's not actively used other
> than in the tests.
I think that's a reasonable compromise. Maybe leave the tests there but
skipped? It doesn't seem like something worth spending large amounts of
time on. The next person to work on the strace helpers can look at
fixing their tests if they like.