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
espanso: init at 0.6.3 #86495
espanso: init at 0.6.3 #86495
Conversation
cargoBuildFlags = [ "--locked" ]; | ||
|
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.
Builds without this just fine for me. Is there a reason this is necessary? Otherwise:
cargoBuildFlags = [ "--locked" ]; |
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.
None that I am aware of. I just noticed the aur package had this flag.
I also managed to build and run without this flag.
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 thus removed the extra line.
684037a
to
61b1921
Compare
I added a meta longDescription. |
I can confirm espanso packages : https://hub.espanso.org/ also work and can be installed like usual. |
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.
Diff LGTM. Binary displays help. Doesn't support Wayland, so I cannot attest to its functionality.
[4 built, 2 copied (0.6 MiB), 0.1 MiB DL]
https://github.com/NixOS/nixpkgs/pull/86495
1 package built:
espanso
This is working great for me. I've created a module to enable it. Once this is merged I'll open a PR for it. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
@kimat can you rebase please to fix the conflict? Also maybe update to v0.6.2. |
a16169a
to
8027148
Compare
@bbigras I just rebased, bumped to 0.6.3 and retested the app's behaviour with |
Reviewed points
Possible improvementsComments |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
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.
Added one small comment, the license does not seem accurate. Feel free to ping me once you have changed this to get it merged.
meta = with stdenv.lib; { | ||
description = "Cross-platform Text Expander written in Rust"; | ||
homepage = "https://espanso.org"; | ||
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.
license = licenses.gpl3; | |
license = licenses.gpl3Plus; |
This should probably be gpl3Plus, since the source files have the either version 3 of the License, or (at your option) any later version wording.
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.
Based on similar PR comments, it should yes. I just updated the file accordingly.
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.
Awesome, thanks a lot!
8027148
to
6f460a7
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.
LGTM
Result of nixpkgs-review pr 86495
1
1 package built:
- espanso
espanso
command runs as expected.
@numkem are you still planning to do that module PR? |
@bbigras done! |
Motivation for this change
Add espanso to nixpkgs
Took this archlinux build script as reference: https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=espanso
doCheck = false;
? https://gist.github.com/kimat/c0b253dafef640da9cd5371af031a01dTo test this without systemd : run
./result/bin/espanso
then type:espanso
anywhere and that text replaces itself.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)