-
-
Notifications
You must be signed in to change notification settings - Fork 15.4k
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
gnunet-gtk: init at 0.12.0 #77828
gnunet-gtk: init at 0.12.0 #77828
Conversation
|
||
meta = with stdenv.lib; { | ||
description = "GNUnet GTK User Interface"; | ||
homepage = https://github.com/ajf8/transmission-remote-gtk; |
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.
homepage = https://git.gnunet.org/gnunet-gtk.git
meta = with stdenv.lib; { | ||
description = "GNUnet GTK User Interface"; | ||
homepage = https://github.com/ajf8/transmission-remote-gtk; | ||
license = licenses.gpl3; |
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.
should be gpl3Plus
homepage = https://github.com/ajf8/transmission-remote-gtk; | ||
license = licenses.gpl3; | ||
maintainers = with maintainers;[ pstn ]; | ||
platforms = platforms.linux; |
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.
should be platforms.gnu ++ platforms.linux (same as gnunet)
Ah, I realize I should have used the "suggest changes" option for GitHub instead. Oh well. |
aefc017
to
27ebb52
Compare
Rebased my branch on master, resolved the issues and rebuilt the package. |
repushed just for signature |
ping @jonringer for review before feature freeze. |
|
||
meta = with stdenv.lib; { | ||
description = "GNUnet GTK User Interface"; | ||
homepage = https://git.gnunet.org/gnunet-gtk.git; |
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.
rfc 45
homepage = https://git.gnunet.org/gnunet-gtk.git; | |
homepage = "https://git.gnunet.org/gnunet-gtk.git"; |
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.
Good catch. Fixed.
55de181
to
d93f339
Compare
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.
installation seems to be missing some localization:
$ strace ./results//gnunet-gtk/bin/gnunet-fs-gtk
....
openat(AT_FDCWD, "/nix/store/jjcw4g52zjzw4nvqmrl2nfpqzvdlr58i-gnunet-gtk-0.12.0/share/gnunet//GNUnet GTK/locale//en_US.UTF-8/LC_MESSAGES/gnunet-gtk.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/nix/store/jjcw4g52zjzw4nvqmrl2nfpqzvdlr58i-gnunet-gtk-0.12.0/share/gnunet//GNUnet GTK/locale//en_US.utf8/LC_MESSAGES/gnunet-gtk.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/nix/store/jjcw4g52zjzw4nvqmrl2nfpqzvdlr58i-gnunet-gtk-0.12.0/share/gnunet//GNUnet GTK/locale//en_US/LC_MESSAGES/gnunet-gtk.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/nix/store/jjcw4g52zjzw4nvqmrl2nfpqzvdlr58i-gnunet-gtk-0.12.0/share/gnunet//GNUnet GTK/locale//en.UTF-8/LC_MESSAGES/gnunet-gtk.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/nix/store/jjcw4g52zjzw4nvqmrl2nfpqzvdlr58i-gnunet-gtk-0.12.0/share/gnunet//GNUnet GTK/locale//en.utf8/LC_MESSAGES/gnunet-gtk.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/nix/store/jjcw4g52zjzw4nvqmrl2nfpqzvdlr58i-gnunet-gtk-0.12.0/share/gnunet//GNUnet GTK/locale//en/LC_MESSAGES/gnunet-gtk.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
--- SIGSEGV {si_signo=SIGSEGV, si_code=SEGV_MAPERR, si_addr=0x48} ---
+++ killed by SIGSEGV (core dumped) +++
Segmentation fault (core dumped)
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.
it just seems to be that one command, and I can't think of a way to fix it.
I could reproduce the weird locale search path but not the segmentation fault. Curious. |
I worked on top of #77785 for a newer gnunet version, so please only merge after that PR.
Motivation for this change
Fixes #76754 and I want it.
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)