-
Notifications
You must be signed in to change notification settings - Fork 36
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 API version to everything that gets installed #10
Comments
I noticed this problem looking at my installed headers. WPE WebKit installs into /usr/include/wpe-0.1. WPEBackend installs into /usr/include/wpe. One solution would be to put them both in wpe-0.1 and expect the API versions to be synced. But this means it will be almost impossible to ever change the WPEBackend API. It would surely be better for WPEBackend to not share an API version with WPE WebKit. This means we should either:
The later solution seems nicer to me. |
Also, I notice this:
which is maybe OK, but means you can't make any ABI changes without bumping the major version. If that's intended, then OK. |
Well, currently WPE WebKit uses /usr/include/wpe-0.1/WPE, but I remove the last WPE part in this bug and then we wind up with this conflict. My suggestion would be to use /usr/include/wpe-webkit-0.1/ instead. |
We should add the API version to:
etc.
The text was updated successfully, but these errors were encountered: