Conversation
There was a problem hiding this comment.
I once looked into this but seem to have forgotten about it or dropped it for some reason. My plan was to add a package sway-contrib with all scripts from the contrib directory. I just pushed the latest version I could find here: primeos@2b079e5
Could you have a look at this and maybe finish it (IIRC I never tested it and some dependencies are most likely still missing)? Feel free to use --reset-author on my existing commit ;)
Edit: Ideally all scripts should work with nix-shell --pure (forgot to mention that :o).
There was a problem hiding this comment.
Are there major issues with grimshot from Sway 1.4?
There was a problem hiding this comment.
there were a bunch of improvements 2 weeks ago
a man page was also added
There was a problem hiding this comment.
That's a bit unfortunate, seems like there are too many commits to cherry-pick them into sway-unwrapped. But ideally we'd still use 1.4 then as long as it works.
There was a problem hiding this comment.
i'd really miss the "manually select window to screenshot" function
and i think the man page is a really nice thing
ofc, if 1.5 gets released we can switch to that
There was a problem hiding this comment.
i suppose i could switch to downloading just the 2 files i'm using
|
i'm not familiar with |
|
this still fails in failure outputi'm not sure if this is avoidable though |
|
I've just opened #87979 to reach a consensus regarding the packaging. |
Motivation for this change
screenshot tool for sway from sway
this seems like a better solution than copying over snippets to my sway config
Though a different solution was suggested and implemented there, this closes #88577
Things done
sandboxinnix.confon non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"./result/bin/)nix path-info -Sbefore and after)101551232Stuff
ping @primeos since i only saw #76787 now
it seems like the dependencies of the other
contrib/stuff are somewhat differentperhaps this could be added as
sway.contrib.grimshotwith a link to it in all-packages.nix?having a separate description for this should improve discoverability