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
elinks: bump to newest version, thereby making it usable again #90593
Conversation
preConfigure = '' | ||
patchShebangs ./autogen.sh | ||
./autogen.sh | ||
''; | ||
|
||
meta = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Currently ofborg fails to evaluate this, as maintainers
isn't in scope.
meta = { | |
meta = with stdenv.lib; { | |
description = "Full-featured text-mode web browser (package based on the fork felinks)"; | |
homepage = "https://github.com/rkd77/felinks"; | |
license = licenses.gpl2; | |
platforms = platforms.linux; | |
maintainers = with maintainers; [ iblech ]; | |
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, thank you, I added the maintainers field at the last moment and didn't notice that it wasn't in scope. I also tried platforms.unix
, I cannot test on macOS, but Hydra can, let's see!
aabd52e
to
0a7ac21
Compare
Conflicts with master atm. |
Thank you @Ma27, I just rebased to latest master and re-tested (alas only on Linux, not on macOS). I believe Borg will test darwin. |
@GrahamcOfBorg build elinks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
elinks builds and works fine but it seems you accidentally included an unrelated commit 1a48e59 . Please remove it.
Oh, thanks for noticing! I just removed the stray commit. :-) |
Motivation for this change
elinks is a text-mode browser from the old times. I vividly remember that it was my main browser at some point. Alas, the current nixpkgs version of elinks is not really usable, as it fails with SSL errors on most sites. You can reproduce this yourself with
or with most any site really.
This pull request packages the newest version, rendering it usable again. Since development of the official elinks repository seems to have paused, this pull request follows Debian in packaging felinks, a fork of elinks. We also adopt Debian's patches which make the build more reproducible and which implement defaults more adapted to today's age (for instance, to honor the caching behavior prescribed by web servers).
Since the nixpkgs package seemed to be unmaintained, I added myself as maintainer.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)