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
deepin-terminal: Init at 2.3.3 #24768
Conversation
1dfc86b
to
5358cd1
Compare
patchPhase = '' | ||
substituteInPlace project_path.c --replace __FILE__ \"$out/share/deepin-terminal/\" | ||
''; | ||
buildInputs = [ unzip gtk3 pkgconfig bash vala cmake vte gee wnck gettext libsecret json_glib ]; |
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.
bash
should be redundant; it's already in the standard environment.
meta = { | ||
description = "The default terminal emulation for Deepin"; | ||
longDescription = '' | ||
Deepin terminal, it sharpens your focus in the world of command line! |
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.
Would you mind indenting this as in
longDescription = ''
lorem ipsum
foo bar
'';
for the sake of consistency.
longDescription = '' | ||
Deepin terminal, it sharpens your focus in the world of command line! | ||
Deepin Terminal is an advanced terminal emulator with workspace, multiple windows, remote management, quake mode and other features. | ||
It sharpens your focus in the world of command line! |
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.
This phrase is repeated.
Looks pretty good to me, added a few minor points for you to consider. |
src = fetchFromGitHub { | ||
owner = "linuxdeepin"; | ||
repo = "deepin-terminal"; | ||
rev = "${version}"; |
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.
Strictly speaking, you can do rev = version
here.
@joachifm thanks your help. I have adjusted the PR |
@joachifm Can we merge this? By the way, how can I mirror the cache.nixos.org ? I have seen someone say using |
The |
Motivation for this change
Add the deepin-terminal which is a friendly terminal emulator .
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/
)