The ProcessUtils in lib-process-utils allows getting PID from a process using reflection. As part of the general Java 9/10/11 hackathon all setAccessible reflection use was being reviewed.
This issue is to remove the reflection use, and as a nice side effect the code will work on Windows too. However, this requires upgrading to Java 9+, so I don't really expect it to happen anytime soon. For now I wanted to preserve the little bit of work I did.
|Remote Link||This issue links to "https://github.com/jenkinsci/lib-process-utils/pull/2 (Web Link)" [ 20911 ]|
|Status||Open [ 1 ]||In Progress [ 3 ]|
|Status||In Progress [ 3 ]||In Review [ 10005 ]|
|Remote Link||This issue links to "convert to multi-release JAR (Web Link)" [ 20924 ]|
|Labels||java10_hackathon java11||java10_hackathon java11 triaged|
|Status||In Review [ 10005 ]||In Progress [ 3 ]|
|Status||In Progress [ 3 ]||Open [ 1 ]|
|Labels||java10_hackathon java11 triaged||java11|
|Assignee||Jonah Graham [ jonahgraham ]|
|Component/s||other [ 15490 ]|
|Component/s||core [ 15593 ]|
|Summary||remove reflection and support Windows in lib-process-utils for getting PID||Illegal reflective access by org.jenkinsci.utils.process.ProcessUtils to field java.lang.ProcessImpl#pid|
|Issue Type||Improvement [ 4 ]||Bug [ 1 ]|