(maint) removed warning CA1416

by disabling the parent process detection in non-windows.

Author: nils-a

src/JavaVersionSwitcher/Adapters/ShellAdapter.cs

@@ -1,5 +1,6 @@
 using System;
 using System.Diagnostics;
+using System.Runtime.InteropServices;
 using JavaVersionSwitcher.Logging;
 
 namespace JavaVersionSwitcher.Adapters
@@ -14,8 +15,14 @@ public ShellAdapter(ILogger logger)
             _logger = logger;
         }
 
+        
         public ShellType GetShellType()
         {
+            if (!RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
+            {
+                return ShellType.Unknown;
+            }
+            
             try
             {
                 var proc = GetParentProcess(Process.GetCurrentProcess());
@@ -45,11 +52,20 @@ public ShellType GetShellType()
             }
         }
 
-        private static Process GetParentProcess(Process process) {
+        private static Process GetParentProcess(Process process) 
+        {
             return FindPidFromIndexedProcessName(FindIndexedProcessName(process.Id));
         }
 
-        private static string FindIndexedProcessName(int pid) {
+#pragma warning disable CA1416
+        private static string FindIndexedProcessName(int pid) 
+        {
+            if (!RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
+            {
+                throw new NotImplementedException(
+                    "Accessing parent process is currently only available on windows.");
+            }
+    
             var processName = Process.GetProcessById(pid).ProcessName;
             var processesByName = Process.GetProcessesByName(processName);
             string processIndexedName = null;
@@ -66,8 +82,15 @@ private static string FindIndexedProcessName(int pid) {
         }
 
         private static Process FindPidFromIndexedProcessName(string indexedProcessName) {
+            if (!RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
+            {
+                throw new NotImplementedException(
+                    "Accessing parent process is currently only available on windows.");
+            }
+
             var parentId = new PerformanceCounter("Process", "Creating Process ID", indexedProcessName);
             return Process.GetProcessById((int) parentId.NextValue());
         }
+#pragma warning restore CA1416
     }
 }

src/JavaVersionSwitcher/JavaVersionSwitcher.csproj

@@ -5,7 +5,7 @@
     <TargetFrameworks>netcoreapp3.1;net5.0</TargetFrameworks>
     <PackAsTool>true</PackAsTool>
   </PropertyGroup>
-
+  
   <ItemGroup>
     <None Include="$(MSBuildProjectDirectory)/../../README.md" PackagePath="" Pack="true" />
     <None Include="$(MSBuildProjectDirectory)/../../res/logo/logo-128.png" PackagePath="" Pack="true" />
@@ -37,9 +37,9 @@
 
   <ItemGroup>
     <PackageReference Include="JetBrains.Annotations" Version="2021.1.0" />
+    <PackageReference Include="Microsoft.Windows.Compatibility" Version="5.0.2" />
     <PackageReference Include="SimpleInjector" Version="5.3.1" />
     <PackageReference Include="Spectre.Console" Version="0.40.0" />
-    <PackageReference Include="System.Diagnostics.PerformanceCounter" Version="5.0.1" />
   </ItemGroup>
 
 </Project>