[Bug 1225229] Review Request: light-locker - A simple session-locker for lightdm

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1225229



--- Comment #2 from Raphael Groner <projects.rg@xxxxxxxx> ---
Spec URL: https://raphgro.fedorapeople.org/review/locker/light-locker.spec
SRPM URL:
https://raphgro.fedorapeople.org/review/locker/light-locker-1.6.0-2.fc22.src.rpm

rawhide build: http://koji.fedoraproject.org/koji/taskinfo?taskID=10218012
epel7   build: http://koji.fedoraproject.org/koji/taskinfo?taskID=10218073

* Fri Jun 26 2015 Raphael Groner <> - 1.6.0-2
- fix license, enhance build options, add gsettings schema scriplets

(In reply to Rex Dieter from comment #1)
…
> 3. SHOULD drop
> desktop-file-validate %{buildroot}%{_sysconfdir}/xdg/autostart/*.desktop
> no need to validate autostart, only stuff under /usr/share/applications/

Well, I do that intentionally. There's an issue with NotShowIn and a patch
provided, see also the upstream issue.

…
> As a practical matter, it worries my a little that it seems to conditionally
> autostart based on DE used (regardless if lightdm is active or not?).  this
> feels wrong to me, but I don't have any suggestions on how to improve that
> offhand.

Not sure what you mean. The DE should provide an on/off switch. What DE could
be problematic in that matter?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]