Opened 9 years ago

Closed 9 years ago

#5862 closed bug (fixed)

CLI parameter --delay is ignored

Reported by: humdinger Owned by: julun
Priority: normal Milestone:
Component: Applications/Screenshot Version: R1/Development
Keywords: Cc: WPJvanderMeer@…
Blocked By: Blocking:
Has a Patch: yes Platform: All

Description

This is hrev36555.

The --delay parameter is ignored.

Attachments (1)

Screenshot-delay-wim.patch (1.2 KB) - added by Wim 9 years ago.
This patch makes sure that the the command line "--delay" switch is not ignored.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 9 years ago by anevilyak

Just curious, you are also passing --silent, right? I had a quick glance and the code appears to be handling the delay parameter, but it's not used unless you're also asking for silent operation.

comment:2 Changed 9 years ago by humdinger

You're right. With --silent it works. But it should also work without that, right? At least it did so some time ago, because my 'nifty' desklink script worked before:

#!/bin/sh

# Add Screenshot to the Deskbar tray

cd /boot/system/apps
desklink "cmd=Active window (2s):Screenshot --window --border --delay 2" "cmd=Remove replicant:desklink --remove=Screenshot" Screenshot

:)

comment:3 Changed 9 years ago by anevilyak

As far as I can tell, without the silent parameter, all it does is set the starting delay that eventually shows up in the UI. No idea if that behavior was changed recently since I've never really kept up with Screenshot development. Without --silent I'm somewhat puzzled as to what you're expecting though, did you want it to wait the delay before popping up the screenshot window, or what?

comment:4 Changed 9 years ago by humdinger

Yes. I want to get this 2 seconds delay to make my target window active, before the GUI pops up to set name, destination and save the bugger.

comment:5 Changed 9 years ago by Wim

This could be the result of the changes I made to save the Screenshot settings. I'll look into it.

Changed 9 years ago by Wim

Attachment: Screenshot-delay-wim.patch added

This patch makes sure that the the command line "--delay" switch is not ignored.

comment:6 Changed 9 years ago by Wim

Cc: WPJvanderMeer@… added

I patched the application in such a way that without specifying the delay on command line a screenshot with delay 0 is taken, even if the saved settings has a non-zero value.

comment:7 Changed 9 years ago by Wim

Is there any chance to get this patch committed before the alpha2 release?

comment:8 Changed 9 years ago by Wim

Has a Patch: set

comment:9 Changed 9 years ago by Wim

Resolution: fixed
Status: newclosed

Fixed in hrev36969.

Note: See TracTickets for help on using tickets.