> Perhaps we should split the config into a separate package? That way there's > a lightdm-config-ubuntu and a lightdm-config-xubuntu package (the both would > obviously have to conflict with eachother).
This is overkill to create packages for only one conffile, and probably a mess to deal with on upgrades or when you install the other variant (and that doesn't look like a sane use of conflicts). > Errors in the patch: > - The exec line is missing --config > - Has tabs instead of spaces No: - "-c" is used, according to the manpage, it has the same meaning... - then fix the orig files? because they already contain a mix of tabs and spaces > I think a better solution would be to allow lightdm to parse the standard > conf by default, and then parse the vendor conf for any "override" values for > things in the standard conf. That keeps all other tools just having to look > at a single file. Yes, similar to what Didier did for gdm (set-default-session). This is much better. But given the time needed to review the tiny patch above (and the review), I'm not very inclined to start working on a more complicated one... I guess I'll forget any attempt to do this cleanly and go back to the dpkg-divert way. Feel free to close this bug report. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/799754 Title: Please let vendors easily provide their own config To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/799754/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
