search

kythyria / payday2-model-tool

Diesel Model importer/exporter, with bones/rigging coming soon!
https://ci.appveyor.com/project/kythyria/payday2-model-tool/build/artifacts
GNU General Public License v3.0
19 stars 7 forks source link

(Hopefully) Fix Win11 Compatibility #26

Closed Sprixitite closed 1 year ago

Sprixitite commented 1 year ago

Should fix issues with the System.Windows.Forms DLL not loading on Windows 11 according to issue #24, this doesn't seem to revert linux functionality (in my limited testing of "does it compile?")

ghost commented 1 year ago

It appears I'm still unable to open the program, and I am still getting the same error D:

Unhandled exception. System.IO.FileLoadException: Could not load file or assembly 'System.Windows.Forms, Version=6.0.2.0, Culture=neutral, PublicKeyToken=b77a5c561934e089'. The located assembly's manifest definition does not match the assembly reference. (0x80131040)
File name: 'System.Windows.Forms, Version=6.0.2.0, Culture=neutral, PublicKeyToken=b77a5c561934e089'
   at PD2ModelParser.Program.Main(String[] args)
Sprixitite commented 1 year ago

I assume you already have it installed but could you please run the installer found at this link just to double check you have the .net 6 runtime installed? https://dotnet.microsoft.com/en-us/download/dotnet/6.0

Sprixitite commented 1 year ago

Another thing to note is that those compiling under Linux (without the use of wine) will now need to set the flag "--runtime win10-x64" or the program will try to resolve for the Linux version of the Windows Desktop Runtime (which obviously doesn't exist)

I'm unsure if this classifies as a regression or not however, as I'm not sure cross-compilation was possible at all before the use of these csproj changes (once again without the use of wine)

ghost commented 1 year ago

It appears the error was on my end, as I simply forgot to install .net 6 runtime :( All seems to be working great now.

Sprixitite commented 1 year ago

Unfortunate, glad I could help though.

Still asking this be pushed provided it doesn't regress windows compatibility, it (at least on Linux) only seems to be a step forward

Thanks