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?
See the full comment at https://github.com/SSSD/sssd/pull/10#issuecomment-244935329