search

buildingSMART / IDS-Audit-tool

Official buildingSMART tool to audit the validity of a .ids file (according to the full IDSxml standard).
MIT License
19 stars 8 forks source link

Multi-target .net dependencies explicitly per target, and add net8 support #23

Closed andyward closed 8 months ago

andyward commented 11 months ago

Rationale: net6 target was depending on Logging.Abstractions v7.0.0 which creates issues in environments like azure functions when there are major version discrepancies.

Because ids-lib under net6 was referencing 7.0.0 logging abstractions while other libraries in a net6 app required 6.0.0 this was creating issues for us (e.g. xbim Geometry is targeting net6 and using 6.0.0 logging). While 7.0.0 is interface compatible with 6 some cases exist where it cause issues resolving an appropriate version, so I've split the ids-lib .net framework targeting explicitly for netstandard2, net6, and I've added explicit net7 and net8 targets.

Worth noting many orgs may not want 7.0 dependencies as its not long-term supported, unlike net6 & net8.

Upgrade testlib and codegen to net8 just to flush out any issues.

TODO: Assembly versions need updating for nuget publishing.

andyward commented 11 months ago

@CBenghi If you get a chance can you review please? Since XIDS took dependency on ids-lib is causing an issue for us as we have a conflict with xbim.geometry v6

andyward commented 10 months ago

@CBenghi any chance you can take a look at this? Just needs the build no incrementing...