1.32 Windows 11 support (1 Viewer)

framug

Super Moderator
  • Team MediaPortal
  • January 31, 2005
    5,921
    1,992
    South of France
    Home Country
    France France
    Even if MP is working on Windows 11, both, SetupTV, TvService, MP config and MP client doesn't give the correct Windows 11 OS version running, in logs.
    The provided OSInfo class fix that.

    Before :

    Code:
    [2023-07-29 10:55:38,781] [Log    ] [SetupTv  ] [INFO ] - ---- SetupTv v1.31.100.0 is starting up on Windows 10 Professional x64 [10.0.22621.0]

    After :

    Code:
    [2023-07-29 10:57:40,161] [Log    ] [SetupTv  ] [INFO ] - ---- SetupTv v1.31.100.0 is starting up on Windows 11 Professional x64 [10.0.22621.0]
     

    Attachments

    • OSInfo.rar
      7.7 KB

    ajs

    Development Group
  • Team MediaPortal
  • February 29, 2008
    15,669
    10,628
    Kyiv
    Home Country
    Ukraine Ukraine

    framug

    Super Moderator
  • Team MediaPortal
  • January 31, 2005
    5,921
    1,992
    South of France
    Home Country
    France France
    • Thread starter
    • Moderator
    • #3
    And then ?
    Where is Windows 11 ?
    Nowhere !

    That's why, according to this :

    Before :

    C#:
          if (
            !IsOSAsReported(OSMajorVersion, OSMinorVersion, OSBuildVersion, (byte) OSProductType, (short) OSServicePackMajor))
            osName = "Compatibilty Mode: " + osName;

    After :

    C#:
          if (!IsOSAsReported(OSMajorVersion, OSMinorVersion, OSBuildVersion, (byte) OSProductType, (short) OSServicePackMajor))
          {
            osName = "Compatibilty Mode: " + osName;
          }
    
          if (osName == "Windows 10" && OSBuildVersion >= 22000)
          {
            osName = "Windows 11";
          }
     

    ajs

    Development Group
  • Team MediaPortal
  • February 29, 2008
    15,669
    10,628
    Kyiv
    Home Country
    Ukraine Ukraine
    If you make changes, then do them correctly, we have two 2 functions that give information about the operating system, you made changes in one, and then a crutch. And we will get that one function will return Windows 10, the second 11, on the same Operating system. In my opinion, this is not correct.
    Well, if you change it, then you must immediately add the current versions of the Server, the information is available, you just need to make changes.
     

    framug

    Super Moderator
  • Team MediaPortal
  • January 31, 2005
    5,921
    1,992
    South of France
    Home Country
    France France
    • Thread starter
    • Moderator
    • #10
    If you make changes, then do them correctly, we have two 2 functions that give information about the operating system, you made changes in one, and then a crutch. And we will get that one function will return Windows 10, the second 11, on the same Operating system. In my opinion, this is not correct.
    Well, if you change it, then you must immediately add the current versions of the Server, the information is available, you just need to make changes.
    The OSProductType has not been changed by Msoft, it continue to be 10, at least for win 11.
    And the method GetOSName() is never used.

    The used method in MP is GetOSNameString(), for displaying OS informations.
    That's why, changes are there...
     

    Users who are viewing this thread

    Top Bottom