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

glpk: 4.56 -> 4.61 #22227

Merged
merged 1 commit into from Feb 1, 2017
Merged

glpk: 4.56 -> 4.61 #22227

merged 1 commit into from Feb 1, 2017

Conversation

lsix
Copy link
Member

@lsix lsix commented Jan 28, 2017

See https://lists.gnu.org/archive/html/info-gnu/2017-01/msg00011.html
for release announcement

Motivation for this change
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.

@kevincox
Copy link
Contributor

kevincox commented Jan 28, 2017

This appears to break octave on travis due to a linker error. Did you have different results when running locally?

@lsix
Copy link
Member Author

lsix commented Jan 29, 2017

I did build it fine. I’ll relaunch the build to be sure.

@lsix
Copy link
Member Author

lsix commented Jan 29, 2017

I can run:

$ nix-shell -p nox --command 'nox-review pr 22227'
=== Reviewing PR 22227 : glpk: 4.56 -> 4.61
==> Fetching base (master)
[…]
/nix/store/xi00s2zvcaydj4xsq2v507gyw92v01sk-octave-4.2.0
/nix/store/m17sd5wxd7fyvy8wli5x00dj0spisclj-pure-glpk-0.5
/nix/store/l3zr2kwawc1qdbh46ljc8mcn2b174mv3-glpk-4.61
Result in /run/user/1000/nox-review-8ssmd2_a
total 0
lrwxrwxrwx 1 lsix lsix 56 29 janv. 17:30 result -> /nix/store/xi00s2zvcaydj4xsq2v507gyw92v01sk-octave-4.2.0
lrwxrwxrwx 1 lsix lsix 57 29 janv. 17:30 result-2 -> /nix/store/m17sd5wxd7fyvy8wli5x00dj0spisclj-pure-glpk-0.5
lrwxrwxrwx 1 lsix lsix 53 29 janv. 17:30 result-3 -> /nix/store/l3zr2kwawc1qdbh46ljc8mcn2b174mv3-glpk-4.61
$ /nix/store/xi00s2zvcaydj4xsq2v507gyw92v01sk-octave-4.2.0/bin/octave --version
octave: X11 DISPLAY environment variable not set
octave: disabling GUI features
GNU Octave, version 4.2.0
Copyright (C) 2016 John W. Eaton and others.
This is free software; see the source code for copying conditions.
There is ABSOLUTELY NO WARRANTY; not even for MERCHANTABILITY or
FITNESS FOR A PARTICULAR PURPOSE.

Octave was configured for "x86_64-pc-linux-gnu".

Additional information about Octave is available at http://www.octave.org.

Please contribute if you find this software useful.
For more information, visit http://www.octave.org/get-involved.html

Read http://www.octave.org/bugs.html to learn how to submit bug reports.

@LnL7 LnL7 merged commit 7a1dd5f into NixOS:master Feb 1, 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

4 participants