Hi, Double checking [2], it seems a harmless change. [2] https://specifications.freedesktop.org/desktop-entry-spec/latest/ar01s05.html On Sat, Oct 29, 2016 at 02:54:15PM -0400, Jeremy Bicha wrote: > On Sat, Oct 29, 2016 at 2:37 PM, Jonathon Jongsma <jjongsma@xxxxxxxxxx> wrote: > > This seems like a policy thing that different distributors could have > > different opinions about. However, I don't know if anybody else still > > ships this app, so maybe it only affects Ubuntu? So I guess I don't > > really have any objection. > > If you look through /etc/xdg/autostart/ you'll see that most files > there have NoDisplay=true set now [1] > > Is there a good reason for users to be able to easily disable the > spice-vdagent autostart? Not that I'm aware of. If nobody complains I'll be pushing it on monday. Acked-by: Victor Toso <victortoso@xxxxxxxxxx> And thanks for the patch :) - Feel free to open more bugs (and patches!) in case you have incompatibles with Ubuntu. toso > > Thanks, > Jeremy Bicha > > [1] Ubuntu 11.04 default install had 21 autostart files without > NoDisplay; this was reduced to 0 before 12.04 LTS and most upstreams > accepted the NoDisplay patches > https://blueprints.launchpad.net/ubuntu/+spec/desktop-o-startup-applications > $ ls -1 /etc/xdg/autostart/ | wc -l > $ rgrep NoDisplay /etc/xdg/autostart/ | wc -l > _______________________________________________ > Spice-devel mailing list > Spice-devel@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/spice-devel
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel