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

awesome: specify version #29510

Merged
merged 1 commit into from Sep 17, 2017
Merged

awesome: specify version #29510

merged 1 commit into from Sep 17, 2017

Conversation

yrashk
Copy link
Contributor

@yrashk yrashk commented Sep 17, 2017

By default, awesome will use "devel" as a version name
(or git describe). This has led to awesome always
showing "devel" for its version.

Some extensions depend on version information to figure
out what features they can use.

This change overrides the version for the build from the
derivations' version attribute.

Motivation for this change

See above. The first occurrence of an extension that depended on
awesome's version detection: deficient/calendar#3

Things done
  • Built on platform(s)
    • NixOS
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

cc @ndowens

By default, awesome will use "devel" as a version name
(or `git describe`). This has led to awesome always
showing "devel" for its version.

Some extensions depend on version information to figure
out what features they can use.

This change overrides the version for the build from the
derivations' `version` attribute.
yrashk added a commit to yrashk/nix-config that referenced this pull request Sep 17, 2017
This breaks some extensions

Solution: override the derivation to specify it

A pull request to nixpkgs has been sent NixOS/nixpkgs#29510
@fpletz fpletz added this to the 17.09 milestone Sep 17, 2017
@fpletz fpletz merged commit e29f6be into NixOS:master Sep 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants