Please do not use bash in org.gnome.DejaDup.Monitor.desktop autostart file

Bug #1715571 reported by Laurent Bigonville
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Déjà Dup
Fix Released
Undecided
Unassigned

Bug Description

Hi,

Would it be possible to not use bash in the /etc/xdg/autostart/org.gnome.DejaDup.Monitor.desktop file? bash is not guaranteed to be installed in debian. sh should be used instead

Revision history for this message
Laurent Bigonville (bigon) wrote :

I'm also adding the "exec" statement so we don't let a (ba)sh process running for nothing

Revision history for this message
Michael Terry (mterry) wrote :

Thank you for both fixes! I went with bash because I didn't see ulimit -v in sh(1), but it does in fact support it. Patch committed.

Changed in deja-dup:
status: New → Fix Committed
Changed in deja-dup:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Patches

Remote bug watches

Bug watches keep track of this bug in other bug trackers.