Skip to content
This repository was archived by the owner on Mar 19, 2021. It is now read-only.

Conversation

mariogrip
Copy link
Member

This kills the scope api part, and also does some cleanups

Copy link
Member

@dobey dobey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This breaks API/ABI, so I think we need to bump the soname, rename the binary package for the lib, and will need to rebuild anything which depends on it (thumbnailer seems to be the only thing other than scopes stuff at least).

@mariogrip
Copy link
Member Author

This breaks API/ABI, so I think we need to bump the soname, rename the binary package for the lib, and will need to rebuild anything which depends on it (thumbnailer seems to be the only thing other than scopes stuff at least).

Ok bumped the soname, but why would we need to rename the binary package?

I'll also remove anything that depends on it

@mariogrip
Copy link
Member Author

One thing that we might do is up the soname on the debian packaging also.

@dobey
Copy link
Member

dobey commented Apr 24, 2019

Ok bumped the soname, but why would we need to rename the binary package?

You only bumped the micro number in the API version string. I think you need to bump the UNITY_API_VERSION variable in src/CMakeLists.txt to be 1 instead. And the binary package, as per Debian Policy, includes the SO version in the name (libunity-api0 in this case), so when the binary version changes, the binary package name for the library, has to change with it.

@mariogrip
Copy link
Member Author

Ok bumped the soname, but why would we need to rename the binary package?

You only bumped the micro number in the API version string. I think you need to bump the UNITY_API_VERSION variable in src/CMakeLists.txt to be 1 instead. And the binary package, as per Debian Policy, includes the SO version in the name (libunity-api0 in this case), so when the binary version changes, the binary package name for the library, has to change with it.

Ah i see, we did change a cpp file (version.cpp), I thought it was only headers. will update the soname

@mariogrip
Copy link
Member Author

There i think we should be ok

@dobey dobey merged commit 98cb672 into xenial_-_edge Apr 26, 2019
@dobey dobey deleted the xenial_-_edge_-_killscope branch April 26, 2019 22:12
UniversalSuperBox pushed a commit that referenced this pull request Nov 24, 2019
This kills the scope api part, and also does some cleanups.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants