On Tue, Sep 6, 2016 at 2:30 PM, jhrozek <[email protected]> wrote: > jhrozek commented on a pull request > > """ > On Tue, Sep 06, 2016 at 05:16:00AM -0700, fidencio wrote: >> The distributions that would break with this patch are: >> - RHEL/CentOS 5 and older > > I don't think we care about RHEL-5 with master and I'm not sure sssd > master even builds there. > >> - Debian Wheezy (from 2013) and older > > OK, this one I think we care about for the basic functionality: > 7.0 Wheezy May 4th 2013 April 26th 2016 (full) / May 2018 (LTS) > But LTS means bug fixes and for those I think sssd-1-13 would be OK. > >> - Ubuntu 12.04 LTS and older > > 12.04 is often used (for example travis-ci still offers only this > distro) and ends its life in 2017. > > On one hand, it's unlikely that users of LTS distributions will run > master, they will probably only run sssd-1-13 or some PPAs, on the other > hand, I don't see too much value except for cleaner code. > > So all in all I suggest we nack this patchset for now and push it when > Ubuntu 12.04 goes EOL. > > Does that sound like a reasonable compromise?
Works for me. > > """ > > See the full comment at > https://github.com/SSSD/sssd/pull/10#issuecomment-244935329 > > _______________________________________________ > sssd-devel mailing list > [email protected] > https://lists.fedorahosted.org/admin/lists/[email protected] > _______________________________________________ sssd-devel mailing list [email protected] https://lists.fedorahosted.org/admin/lists/[email protected]
