Skip to content

Conversation

@jogibear9988
Copy link

  • switch to new csproj file format.
  • create a version for net472 and net8
  • automaticly load correct openssl dll
  • switch to ne solution file format (slnx)

@jogibear9988
Copy link
Author

Maybe the switch to slnx is a little bit early and should not be done yet, I need to look if building with dotnet command is already possible atm.
I'm also looking to generate a build pipline wich would automaticly create a nuget package. Would this be okay, or do you not want this?

@jogibear9988
Copy link
Author

I also added your fix in zlib to cinderblocks/zlib.net#5
So the nuget here: https://www.nuget.org/packages/zlib.net-mutliplatform/1.0.7 could be used (when it is merged)

@thomas-v2
Copy link
Owner

I can't use .Net Core, and I have to stay at Visual Studio 2019 for some reason.

@thomas-v2
Copy link
Owner

thomas-v2 commented Aug 22, 2024

I had to fix a bug in zlib.net, which is also still present in the .Net core version (maybe no one used zlib with dictionaries, or .Net core handles type casts in a different manner). So I'd like to keep it internally, also I prefer to reduce external dependencies to an absolute minimum. Download the source, and you should be ready to go.

@jogibear9988 jogibear9988 marked this pull request as draft August 22, 2024 20:52
@jogibear9988
Copy link
Author

I've seen the bug, I've created a pull with your fix at a zlib nuget package. wich merged it : cinderblocks/zlib.net#5

But no problem to keep it internal.

@jogibear9988
Copy link
Author

VS2019 should be possible ( I think, need to try), but for sure the SLNX will not work

@jogibear9988
Copy link
Author

@thomas-v2 my changes compile now also in VS2019

jogibear9988 and others added 9 commits January 22, 2025 10:04
-> switching dll names does not work very well, and openssl has -x64 on the 64bit dlls.
atm supporting full framework is not needed

add support for win arm (needs test)
and osx (needs test)
# Conflicts:
#	src/S7CommPlusGUIBrowser/Form1.cs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants