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

less: 483 -> 481: Use recommended upstream version since less-483.tar.gz has disappeared #20711

Merged
merged 1 commit into from Nov 26, 2016

Conversation

sh01
Copy link
Contributor

@sh01 sh01 commented Nov 25, 2016

Motivation for this change

Cherrypick version downgrade of package "less" into release-16.09 to make it buildable once more in that version.

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

This is a fix for the current package source file
http://www.greenwoodsoftware.com/less/less-483.tar.gz
not being available anymore.

We bump the less version back to 481, and adjust the source package hash
accordingly. This is a (slight) downgrade from 483 as opposed to an
upgrade since
a) 481 is the current Recommended version by http://www.greenwoodsoftware.com/less/download.html
b) Upstream is unreliable about keeping experimental versions around.

(cherry picked from commit 0f9f74f)

….gz has disappeared

This is a fix for the current package source file
  http://www.greenwoodsoftware.com/less/less-483.tar.gz
not being available anymore.

We bump the less version back to 481, and adjust the source package hash
accordingly. This is a (slight) downgrade from 483 as opposed to an
upgrade since
  a) 481 is the current Recommended version by http://www.greenwoodsoftware.com/less/download.html
  b) Upstream is unreliable about keeping experimental versions around.

(cherry picked from commit 0f9f74f)
@mention-bot
Copy link

@sh01, thanks for your PR! By analyzing the history of the files in this pull request, we identified @matthiasbeyer, @edolstra and @mbakke to be potential reviewers.

@dezgeg dezgeg merged commit ef7cc9a into NixOS:release-16.09 Nov 26, 2016
@sh01 sh01 deleted the cp_fix_less branch November 26, 2016 02:38
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.

None yet

3 participants