Skip to content

Add command line parameter -o to specify the file name of the snapshot #2

Open
ChanningBJ wants to merge 4 commits intolinuxdeepin-packages:masterfrom
ChanningBJ:master
Open

Add command line parameter -o to specify the file name of the snapshot #2
ChanningBJ wants to merge 4 commits intolinuxdeepin-packages:masterfrom
ChanningBJ:master

Conversation

@ChanningBJ
Copy link

I'm trying to use deepin-scrot instead of scrot to capture screen in emacs, which require the screen capture program can specify the output file name from command line.
I modified deepin-scrot a little bit, adding a -o option to specify the output file name. If -o is used, it will save the snapshot to the file specified by -o when click finish button. If -o option is not used, the behaver will be the same (save to clipboard)
Please accept this pull request if you like the change and any comments are welcomed

Cheers!

@kindlychung
Copy link

Funny, I did the same thing! I should have looked at the pull requests before doing it myself, would have saved me some time.

Anyways, thank you for your effort.

I am wondering why there is no response from upstream authors.

@ChanningBJ
Copy link
Author

That's only proof how import this feature is :)
There is another pull request made a year ago for this repository, seems we can't get their response soon.

I will send a mail to these guys from deepin to see if they have time on this.

@ChanningBJ
Copy link
Author

Hi Victor,

I got reply from deepin guys, they do not use this repository anymore, a new one is created here: https://github.com/linuxdeepin/deepin-screenshot

I tried installing this deepin-screenshot a little bit, but seems it has dependency on deepin-ui , deepin-utils and maybe some more deepin packages which only works on Ubuntu 13.04 .

Here is their response mail for your reference:

deepin-scrot is very old repository, current stable repository at: https://github.com/linuxdeepin/deepin-screenshot
QML version deepin-screenshot, under developing, at: https://github.com/linuxdeepin/deepin-screenshot-v3
Please send pull request to https://github.com/linuxdeepin/deepin-screenshot , i will merge it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants