-
-
Notifications
You must be signed in to change notification settings - Fork 862
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
AIX port #1282
AIX port #1282
Conversation
Added few things for process. Haven't been able to test it yet. |
oshi-core/src/main/java/oshi/software/os/unix/aix/AixOSProcess.java
Outdated
Show resolved
Hide resolved
oshi-core/src/main/java/oshi/software/os/unix/aix/AixOSProcess.java
Outdated
Show resolved
Hide resolved
oshi-core/src/main/java/oshi/software/os/unix/aix/AixOSProcess.java
Outdated
Show resolved
Hide resolved
oshi-core/src/main/java/oshi/software/os/unix/aix/AixOperatingSystem.java
Outdated
Show resolved
Hide resolved
Looking good. I don't see a problem with process count being different between queries, a user should expect that. I think I prefer counting numeric filenames in proc (same code as in Linux, and in fact it's working as-is) to a command line ps call. |
ohh and btw you don't have to |
Thank you! I was just about to ask what you did and then wondered, did I ever chmod it to be executable? Was that the problem? |
i don't think executable rights was the problem. What i found was that it never ran the |
oshi-core/src/main/java/oshi/software/os/unix/aix/AixOperatingSystem.java
Outdated
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## master #1282 +/- ##
============================================
- Coverage 85.93% 85.01% -0.92%
- Complexity 280 288 +8
============================================
Files 25 25
Lines 910 941 +31
Branches 124 134 +10
============================================
+ Hits 782 800 +18
- Misses 73 80 +7
- Partials 55 61 +6
Continue to review full report at Codecov.
|
Fixes #1166