Skip to content
This repository has been archived by the owner on Jun 8, 2021. It is now read-only.

Switch to version features instead of autodetection #248

Merged
merged 6 commits into from
Feb 13, 2016
Merged

Switch to version features instead of autodetection #248

merged 6 commits into from
Feb 13, 2016

Conversation

gkoz
Copy link
Member

@gkoz gkoz commented Feb 12, 2016

Implement the reinstatement of version features outlined in #243.

  • Installed library version detection no longer impacts conditional compilation.
  • Each crate defaults to supporting a particular minimal upstream version.
  • Higher (newer) versions need to be enabled via cargo features e.g. --features 3.16, features = ["3.16"].
  • The selected version is checked against pkg-config aborting the build if it's too high.
  • The use of pkg-config can be overridden by setting GTK_LIB_DIR env variable, which translates into -L $GTK_LIB_DIR linker argument. No version checks are done in this case.

[breaking change]

- Installed library version detection no longer impacts conditional
  compilation.
- Each crate defaults to supporting a particular minimal upstream
  version.
- Higher (newer) versions need to be enabled via cargo features e.g.
  `--features 3.16`, `features = ["3.16"]`.
- The selected version is checked against `pkg-config` aborting the
  build if it's too high.
- The use of `pkg-config` can be overridden by setting `GTK_LIB_DIR` env
  variable, which translates into `-L $GTK_LIB_DIR` linker argument. No
version checks are done in this case.

[breaking change]
Update version cfg syntax and make 3.4 symbols unconditional
@gkoz
Copy link
Member Author

gkoz commented Feb 12, 2016

Naturally this can't be tested by our CI tools. It's been preliminary tested locally but we'll only know after merging everything.

GuillaumeGomez added a commit that referenced this pull request Feb 13, 2016
Switch to version features instead of autodetection
@GuillaumeGomez GuillaumeGomez merged commit 2b0a87e into gtk-rs:master Feb 13, 2016
EPashkin pushed a commit to EPashkin/gtk that referenced this pull request Apr 18, 2016
Add myself to the authors and put license headers where they're missing
alex179ohm pushed a commit to alex179ohm/gtk that referenced this pull request Oct 21, 2019
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