-
-
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
fcitx-engines.rime: init at 0.3.2 #29460
Conversation
libyamlcpp, gmock }: | ||
|
||
stdenv.mkDerivation rec { | ||
name = "librime"; |
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.
The version
needs to be part of name
.
name = "librime"; | ||
version = "1.2.9"; | ||
|
||
src = fetchurl { |
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.
You can use fetchFromGitHub
here:
{
src = fetchFromGitHub {
owner = "rime";
repo = "librime";
rev = "rime-${version}";
sha256 = "...";
};
}
ef091c6
to
9e74ee1
Compare
Thanks for the review! |
99d0dd2
to
6d9b6e9
Compare
Hello, what's the status? |
pkgs/data/misc/brise/default.nix
Outdated
# may be released under different licenses | ||
license = licenses.gpl3; | ||
platforms = platforms.all; | ||
maintainers = [ maintainers.mingchuan ]; |
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.
While you wait, you should probably update this to match your new username (and same in the other files and lib/maintainers.nix). :)
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.
Hmm, sifmelcara
is my GitHub handle, and mingchuan
is the name I used in git commit messages.
However, I can change my name in lib/maintainers.nix so that people can easily ping me on GitHub.
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.
Oh right, my bad. Yeah at the top of lib/maintainers.nix it says github handle is preferred, thanks. ✨
88b4956
to
c3417c7
Compare
I've simplified build expressions and enabled parallel building. Thank you for your contribution! |
Motivation for this change
Add fcitx support for Rime input method and Rime's dependencies.
About the
brise
package: This package provides data for many variants of Chinese input methods. Since it will try to download data from GitHub (if data do not already exists), we need to fetch those dialect's data manually.Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)