Title: is always None
Type: Severity: normal
Components: Versions: Jython 2.7, Jython 2.5
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: Nosy List: alex.gronholm, pekka.klarck, zyasoft
Priority: high Keywords:

Created on 2014-10-14.10:44:27 by pekka.klarck, last changed 2015-01-08.15:05:06 by alex.gronholm.

File name Uploaded Description Edit Remove pekka.klarck, 2014-10-14.10:44:26
msg9145 (view) Author: Pekka Klärck (pekka.klarck) Date: 2014-10-14.10:44:26
Title pretty much explains the problem. After studying this a bit, I was able to implement getting pid so that it works at least on Windows 7 and Linux Mint. The resulting code is at the end of this description and it can be tested by running the attached script that contains also simple code to verify the results.

If there's interest, I can take a look at turning this into a pull request. Final solution ought to have error handling at least when accessing private fields. I would assume it's better to return None than raise an exception on error.


import os

if os.sep == '/':

    def get_pid(process):
        field = process.getClass().getDeclaredField('pid')
        return field.getLong(process)


    import ctypes
    GetProcessId = ctypes.cdll.kernel32.GetProcessId
    GetProcessId.argtypes = (ctypes.c_long,)

    def get_pid(proces):
        field = process.getClass().getDeclaredField('handle')
        handle = field.getLong(process)
        return GetProcessId(handle)
msg9147 (view) Author: Pekka Klärck (pekka.klarck) Date: 2014-10-14.12:38:59
Colleague tested this on OSX. His comments:

1) Code works fine.

2) The description isn't too clear how to test this. The correct approach is downloading and running the attached The code in the description doesn't work with normal subprocess.Popen.

3) Windows version of `get_pid` has argument `proces` (typo). The test code works on Windows, though, because there is `process` available in the global scope.
msg9148 (view) Author: Jim Baker (zyasoft) Date: 2014-10-14.17:46:54
Interesting use of ctypes. I believe the most portable way to do this is with JMX however.

Let's implement a robust scheme. As long as we have error handling, it will be better than simply returning None, which remains valid for Jython if we cannot determine the value. (Or perhaps we raise an exception - that could be reasonable too). Then once we have JEP 102 ( available on Java 9, we can use that instead in a future release of 2.7.x (where x > 0, probably greater than 1)

Personally I just prefer a patch, including tests, attached to this bug. PRs on Bitbucket are just a mess; although it's possible that PRs against the GitHub mirror work better. To be seen shortly.

Please add any tests to

msg9153 (view) Author: Pekka Klärck (pekka.klarck) Date: 2014-10-14.22:24:16
Forgot to mention that both POSIX and Windows solutions are based on answers to SO question and on linked from there. Getting pid on POSIX and handle on Windows via reflection is pretty trivial, assuming we don't need to care about security managers. Converting a handle to a pid looked pretty complicated in the examples, but after I found out Kernel32 has GetProcessId for exactly that purpose it turned out to be pretty simple too.

I'm nor certain is the above the best way to get pid but at least it is pretty promising. I don't have experience using JMX but could take a look at that too. Do you have pointers where to look?

Regardless the approach we choose, I think not getting a pid shouldn't cause an exception at least directly when the process is started. We could turn into a property and raise an exception if it is accessed, but I'm not sure is that worth the effort. Just setting it to None would be backwards compatible with earlier Jython versions.

After we are ready with the overall design, I can take a look at creating a pull request on GitHub. My experience from them both when reviewing and submitting has been fairly positive. Would it be OK for tests to run a process like `python -c "import os; print os.getpid()"` and compare the output to In other words, can be assume that devs running tests have python on PATH?
msg9158 (view) Author: Pekka Klärck (pekka.klarck) Date: 2014-10-16.15:48:53
Here is a pull request:
msg9359 (view) Author: Alex Grönholm (alex.gronholm) Date: 2015-01-08.15:05:06
Merged in changesets 593d5c73444b and 2ffae23d1e07.
Date User Action Args
2015-01-08 15:05:06alex.gronholmsetstatus: open -> closed
resolution: fixed
messages: + msg9359
nosy: + alex.gronholm
2014-10-28 23:15:40zyasoftsetpriority: high
2014-10-16 15:48:54pekka.klarcksetmessages: + msg9158
2014-10-14 22:24:17pekka.klarcksetmessages: + msg9153
2014-10-14 17:46:55zyasoftsetnosy: + zyasoft
messages: + msg9148
2014-10-14 12:38:59pekka.klarcksetmessages: + msg9147
2014-10-14 10:44:27pekka.klarckcreate