Merge lp://staging/~kamstrup/bzr-gtk/bzr-notify-autostart-delay into lp://staging/bzr-gtk/gtk2

Proposed by Mikkel Kamstrup Erlandsen
Status: Merged
Approved by: Jelmer Vernooij
Approved revision: 740
Merged at revision: 740
Proposed branch: lp://staging/~kamstrup/bzr-gtk/bzr-notify-autostart-delay
Merge into: lp://staging/bzr-gtk/gtk2
Diff against target: 9 lines (+2/-0)
1 file modified
bzr-notify.desktop (+2/-0)
To merge this branch: bzr merge lp://staging/~kamstrup/bzr-gtk/bzr-notify-autostart-delay
Reviewer Review Type Date Requested Status
Jelmer Vernooij (community) Approve
Review via email: mp+77291@code.staging.launchpad.net

Description of the change

See attached bug and commit message

To post a comment you must log in.
Revision history for this message
Jelmer Vernooij (jelmer) wrote :

Thanks for the MP.

This seems reasonable. 30 seconds is a bit arbitrary, it would be nicer to just be able to say "take your time to start this" rather than a specific delay.

Revision history for this message
Jelmer Vernooij (jelmer) :
review: Approve
Revision history for this message
Mikkel Kamstrup Erlandsen (kamstrup) wrote :

For general info - the 30s was chosen because that's what many other
files in /etc/xdg/autostart defines. So not *completely* arbitrary :-)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes:
to status/vote changes: