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 #20703

Merged
merged 1 commit into from Nov 25, 2016

Conversation

sh01
Copy link
Contributor

@sh01 sh01 commented Nov 25, 2016

Motivation for this change

Change less version to 481 since the current source http://www.greenwoodsoftware.com/less/less-483.tar.gz has become unavailable, making it impossible to build this.

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.

@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.

….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.
Copy link
Contributor

@matthiasbeyer matthiasbeyer left a comment

Choose a reason for hiding this comment

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

Reverting 473e5a4 would be much easier, but as you already provided the patch ... I'm okay with that!

@Mic92 Mic92 merged commit a3f852f into NixOS:master Nov 25, 2016
@Mic92
Copy link
Member

Mic92 commented Nov 25, 2016

Thanks!

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

4 participants