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
gitkraken: init at 2.1.0 #23459
gitkraken: init at 2.1.0 #23459
Conversation
lib/maintainers.nix
Outdated
@@ -353,6 +353,7 @@ | |||
notthemessiah = "Brian Cohen <brian.cohen.88@gmail.com>"; | |||
np = "Nicolas Pouillard <np.nix@nicolaspouillard.fr>"; | |||
nslqqq = "Nikita Mikhailov <nslqqq@gmail.com>"; | |||
nwdd = "Guillermo NWDD <nwdd+nixos@no.team>"; |
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.
Is that a real email address?
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.
Please add your entry to maintainers.nix in a separate commit.
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.
@bjornfor It's real, feel free to drop me a line.
@@ -0,0 +1,69 @@ | |||
{stdenv, lib, libXcomposite, libgnome_keyring, makeWrapper, udev, curl, fetchurl, libXfixes, atk, gtk2, libXrender, pango, gnome2, cairo, freetype, fontconfig, libX11, libXi, libXext, libXcursor, glib, libXScrnSaver, libxkbfile, libXtst, nss, nspr, cups, alsaLib, expat , gdk_pixbuf, libXdamage, libXrandr, dbus}: |
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.
Please wrap overlong lines using the following style:
{ stdenv, .......
, foo, .....
}:
EDIT: semicolon -> colon typo
phases = [ "unpackPhase" "installPhase" ]; | ||
buildInputs = [ makeWrapper ]; | ||
|
||
installPhase = '' |
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.
Please indent the string in installPhase:
installPhase = ''
mkdir -p "$out/opt/gitkraken"
...
'';
ln -s "$out/opt/gitkraken/gitkraken" "$out/bin/gitkraken" | ||
''; | ||
|
||
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.
If you write meta = with stdenv.lib; {
you can drop all the stdenv.lib
things inside meta.
description = "The downright luxurious and most popular Git client for Windows, Mac & Linux"; | ||
license = licenses.unfreeRedistributable; | ||
platforms = platforms.linux; | ||
maintainers = [ ]; |
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.
Can you be a maintainer?
meta = with stdenv.lib; { | ||
homepage = https://www.gitkraken.com/; | ||
description = "The downright luxurious and most popular Git client for Windows, Mac & Linux"; | ||
license = licenses.unfreeRedistributable; |
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.
I don't think the license allows us to redistribute.
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's actually the other way round. This would allow us to distribute the binaries.
But their EULA states distribution is prohibited, so this should be licenses.unfree
.
That is what I meant to say :)
…On Mon, Mar 6, 2017 at 9:35 PM Franz Pletz ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In pkgs/applications/version-management/gitkraken/default.nix
<#23459 (comment)>:
> + mkdir -p "$out/opt/gitkraken"
+ cp -r ./* "$out/opt/gitkraken"
+ fixupPhase
+ patchelf --set-interpreter $(cat $NIX_CC/nix-support/dynamic-linker) \
+ --set-rpath "$libPath:$out/opt/gitkraken" "$out/opt/gitkraken/gitkraken"
+ wrapProgram $out/opt/gitkraken/gitkraken \
+ --prefix LD_PRELOAD : "${stdenv.lib.makeLibraryPath [ curl ]}/libcurl.so.4" \
+ --prefix LD_PRELOAD : "${stdenv.lib.makeLibraryPath [ libgnome_keyring ]}/libgnome-keyring.so.0"
+ mkdir "$out/bin"
+ ln -s "$out/opt/gitkraken/gitkraken" "$out/bin/gitkraken"
+ '';
+
+ meta = with stdenv.lib; {
+ homepage = https://www.gitkraken.com/;
+ description = "The downright luxurious and most popular Git client for Windows, Mac & Linux";
+ license = licenses.unfreeRedistributable;
It's actually the other way round. This would allow us to distribute the
binaries.
But their EULA <https://www.gitkraken.com/eula> states distribution is
prohibited, so this should be licenses.unfree.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#23459 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAErrH16vCbbKixFr2R1Vyn2qfzg_i74ks5rjMJigaJpZM4MSwsW>
.
|
Motivation for this change
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)