Skip to content
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

Add version numbers to native libraries #1347

Closed
wants to merge 3 commits into from

Conversation

mattleibow
Copy link
Contributor

@mattleibow mattleibow commented Jun 23, 2020

Description of Change

Add a "suffix" to library names:

libSkiaSharp_<milestone>_<skiasharp>.<ext>

For example for 1.68.3:

libSkiaSharp_68_3.dll

And for 2.80.0:

libSkiaSharp_80_0.dll

This will avoid the cases when older versions are used with newer apps and causing crashes.

The reason for the 2 numbers only is the way SkiaSharp is versioned:

<major-change-indicator>.<skia-version>.<skiasharp-version>.<bug-or-packaging-fix>

So...

  • Going from 1.* to 2.* is nothing special - except we may have re-written the library (which we haven't actually done because we like backwards compatibility)
  • Going from *.1 to *.2 means that Google's code is new - and may have behavioral changes (which usually doesn't happen - except for bug fixes)
  • Going from *.*.1 to *.*.2 means that we did a new API or changed some interop code (and thus ABI breaking)
  • Going form *.*.*.1 to *.*.*.2 means that there may have been a docs/packaging/managed-only change (and the native library is still the same version)

@mattleibow mattleibow changed the base branch from dev/experimental/svg-dom to dev/library-loading June 23, 2020 13:36
@mattleibow
Copy link
Contributor Author

Closing this in favor of adding a version API to the actual library in #1342

@mattleibow mattleibow closed this Jun 24, 2020
@mattleibow mattleibow deleted the dev/library-versioning branch June 25, 2020 00:20
@mattleibow mattleibow added this to the v2.80.0 milestone Jul 9, 2020
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.

1 participant