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
grimshot: init at 2020-05-08 #87831
grimshot: init at 2020-05-08 #87831
Conversation
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 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).
# after sway-1.5 this may be switched to sway-unwrapped.src | ||
src = fetchFromGitHub { | ||
owner = "swaywm"; | ||
repo = "sway"; | ||
# latest change to contrib/grimshot* | ||
rev = "b1d08db5f5112ab562f89564825e3e791b0682c4"; | ||
sha256 = "1j9wk2m79grhjxyx8fzgcza0virlkansl2fvlvjszk526i0igri6"; | ||
}; |
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.
Are there major issues with grimshot from Sway 1.4?
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.
there were a bunch of improvements 2 weeks ago
a man page was also added
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.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i suppose i could switch to downloading just the 2 files i'm using
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.
done
i'm not familiar with |
this still fails in failure output
i'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
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)101551232
Stuff
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.grimshot
with a link to it in all-packages.nix?having a separate description for this should improve discoverability