Opened 13 years ago

Closed 6 years ago

Last modified 3 months ago

#608 defect closed wontfix (wontfix)

Twisted doesn't detect child process termination on HP-UX

Reported by: lzfnk Owned by: warner
Priority: low Milestone:
Component: core Keywords:
Cc: warner, lzfnk, Thijs Triemstra, Adi Roiban Branch:
Author:

Description


Attachments (1)

proctest.py (238 bytes) - added by lzfnk 13 years ago.

Download all attachments as: .zip

Change History (12)

Changed 13 years ago by lzfnk

Attachment: proctest.py added

comment:1 Changed 13 years ago by lzfnk

When running a subprocess via reactor.spawnProcess() under Twisted 1.2.0, the
ProcessProtocol's processEnded() method will not be called at child process
termination.  The very simple attached test program demonstrates the bug via the
twisted.internet.utils.getProcessOutput() utility function.  The test program
never terminates until the user hits ctrl-c.

comment:2 Changed 13 years ago by lzfnk

I forgot to mention, apparently HP has machines available for testing at
http://www.testdrive.compaq.com/

The problem occurs under HP-UX 11i, aka 11.11.

comment:3 Changed 11 years ago by Glyph

Component: conch

HP-UX isn't a supported platform, as no developers that I'm aware of have access to HP-UX machines and we don't have an HP-UX buildbot. None of our developers have any HP-UX experience that I know of either, so a maintainer would be a plus.

The buildbot is the major sticking point though. We can't even really accept patches to fix issues with HP-UX until we have that, since we don't have a way of verifying that they work.

It looks like the "testdrive" program is still around, and maybe somebody could create a buildbot there.

comment:4 Changed 11 years ago by jknight

Component: conchcore

I hope the above doesn't scare anybody off from trying to identify the issue and make a patch.

If the solution is sufficiently straightforward, I think such a patch would have a chance of being accepted regardless of the existence of a buildbot.

Obviously HPUX cannot be a supported platform unless there's a buildbot, since it's liable to be broken by any future changes without anybody noticing, though.

comment:5 in reply to:  3 ; Changed 9 years ago by Thijs Triemstra

Cc: Thijs Triemstra added

Replying to glyph:

It looks like the "testdrive" program is still around, and maybe somebody could create a buildbot there.

Updated url: http://www.testdrive.hp.com/faq/

comment:6 in reply to:  5 Changed 9 years ago by Thijs Triemstra

Replying to thijs:

Updated url: http://www.testdrive.hp.com/faq/

Ah, they closed this program down on Sept 30, 2008 (http://www.testdrive.hp.com/news/)

comment:7 Changed 9 years ago by Thijs Triemstra

Priority: highlow

So we can either close this ticket, because of the lack of a buildbot, or open a new ticket for setting up a HP-UX buildbot. and perhaps ask around on the mailinglist or IRC if anyone has such a system available that we can use for a buildslave. Also don't think this ticket should have a high priority, changing that.

Thoughts?

comment:8 Changed 9 years ago by Jean-Paul Calderone

Probably should be left open. Low priority is totally fine, though. I have a lead on an HP-UX machine, not sure when it will materialize, though.

comment:9 Changed 6 years ago by Itamar Turner-Trauring

Resolution: wontfix
Status: newclosed

We are unlikely to ever get a HP-UX machine to test this on, and for that matter the process code has been improved. Re-open if issue still exists.

comment:10 in reply to:  9 Changed 5 years ago by gavekort

Would anyone be willing to help if i can get access to a HP-UX machine ?

Kind regards. Gavekort.

Replying to itamar:

We are unlikely to ever get a HP-UX machine to test this on, and for that matter the process code has been improved. Re-open if issue still exists.gavekort biografgavekort skabelon

comment:11 Changed 3 months ago by Adi Roiban

Cc: Adi Roiban added

I have access (own) a couple of HP-UX systems. For now, fixing this is low priority for my, but I am happy to sponsor the fix for this ticket... and also review the work

Note: See TracTickets for help on using tickets.