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
foot: init at 1.4.4 #95596
foot: init at 1.4.4 #95596
Conversation
@GrahamcOfBorg build foot fuzzel |
Weird, don't know why it didn't work out for me. Am I supposed to be able to run this on Xorg? |
No, it is wayland only! |
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.
No, it is wayland only!
Then I can't help further. The build logs looks good and the tests seem to be running fine so this LGTM. However, I have some personal style nitpicks :)
{ stdenv, fetchgit, fcft, freetype, pixman, libxkbcommon, fontconfig, wayland | ||
, meson, ninja, ncurses, scdoc, tllist, wayland-protocols, pkg-config | ||
}: |
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.
{ stdenv, fetchgit, fcft, freetype, pixman, libxkbcommon, fontconfig, wayland | |
, meson, ninja, ncurses, scdoc, tllist, wayland-protocols, pkg-config | |
}: | |
{ stdenv | |
, fetchgit | |
, pkg-config | |
, fcft | |
, freetype | |
, pixman | |
, libxkbcommon | |
, fontconfig | |
, wayland | |
, meson | |
, ninja | |
, ncurses | |
, scdoc | |
, tllist | |
, wayland-protocols | |
}: |
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.
For my taste these just convert horizontal space into vertical space without a real benefit to readability. I reduced horizontal space somewhat and split the inputs by category more obviously.
|
||
src = fetchgit { | ||
url = "https://codeberg.org/dnkl/foot.git"; | ||
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.
rev = "${version}"; | |
rev = version; |
Result of 3 packages built:- fcft - foot - fuzzel |
Motivation for this change
add
foot
a simple and fast terminal emulator for wayland based offcft
font rendering.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)