-
Notifications
You must be signed in to change notification settings - Fork 922
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
IOP workshop fixes and build improvements #1562
Conversation
note: testing .NET5.0 build, will be removed |
This pull request introduces 1 alert when merging 373e332 into ed434e9 - view on LGTM.com new alerts:
|
This pull request introduces 1 alert when merging 5212cfc into ed434e9 - view on LGTM.com new alerts:
|
This pull request introduces 1 alert when merging fb74d65 into ed434e9 - view on LGTM.com new alerts:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
as a sidenote the .NET5.0 build is passing, but the .NET 4.7.2 build is missing some functionality (ImportSubjectPublicKeyInfo) to be able to fully support the ECC and RSA cert functions. The .NET5 and net472 build steps have been removed for this PR. |
-fixes for complex types: - a structure which contains an enum from namespace 0 can not be created. fixes #1561. - a structure with multidimensional array may lead to a endless loop in `ComplexTypeSystem.Load` - a malformed namespace Uri which contains a space character prevents creation of type system. - select endpoint doesn't pick the right scheme if the default endpoint must be picked -prep for net472 or greater and net5.0 or greater build, only build #ifdef
-prep for net472 or greater and net5.0 or greater build, only build #ifdef
-fixes for complex types:
ComplexTypeSystem.Load