Hi,

[...]
> >> As a result, files remain open and the package build cannot complete 
> >> properly
> >> when working in a chroot (and waiting to umount the build directory).
> 
> Ah.  The process should "cd" to / then, too?
> 

Yes, this might help to solve the problem.

> >> Most likely it suffices to reduce the kill_delay for usage.test, but a 
> >> proper
> >> implementation would terminate the watchdog whenever the test itself has
> >> ended properly.
> > 
> > Makes sense.  I did something quick and dirty because I've had stuff hang
> > forever and this was easy to do.  I'm open to suggestions....
> 
> Maybe this?  "AG_TIMEOUT" is an adjustment for the platform speed.
> "51" is a pretty high number for that.
> 
[...] (patch snipped)

Yes, the patch looks good from my point of view. Even if AG_TIMEOUT is a high
value: I think it would be best not to rely exclusively on timeout values being
suitable for a specific machine - but as far as I understand, your patch
properly addresses this.

Best,
Michael

Attachment: pgpAHYxVTuUXB.pgp
Description: PGP signature

Reply via email to