Remove ancient RHL 6.x config in RPM spec.

[FabioPedretti]

Note: please have a detailed look at this, since I am actually not very familiar building packages for RH. Anyway I tested it on RHEL 6 (where this config was applied) and it looks everything is OK.

It looks like build6x options were intended for RHL 6.x (the Red Hat distro predating Fedora, not RHEL), but were then applied to RHEL.

Completely remove support for this ancient configuration.

Successfully built, installed and run on RHEL 6. This also remove a build warning about deprecation of PreReq.

[FabioPedretti]

Actually the original build6x option for RHL 6.x was introduced 20+ years ago: cfe4a89

While it was later wrongly reused for RHEL 6.x in 87346f1

[FabioPedretti]

Also note that part of (actually, the 2 ".old" files) cfe4a89 was already removed in d83ec9e, so the original intent to support such old distros was already broken since over 2 years.

@nkadel can you review this?

[nkadel]

RHEL 6 systems that get this RPM installed will ignore their old hostkeys in "/etc/" and generate new ones in "/etc/ssh/". This is..... begging for an unwelcome surprise as clients find mis-matched hostkeys in their $HOME/.ssh/known_hosts file and refuse to connect. I'm tempted to say "it's RHEL 6, who cares?", but banks and hospitals tend to run critical systems far, far beyond the expiration date written on the cereal box, sometimes for really good reasons to access old data with unique software. The current location is baked into ansible, chef, puppet, and systemd-init, and waagent. I would remain consistent with RHEL 6's published layout and not touch any of this for another few years.

[FabioPedretti]

Thanks for the quick reply @nkadel, but it looks like the original build6x behaviour was intended for RHL 6.x (released in 1999), while your commit 87346f1 changed this behaviour and enabled it for RHEL 6.x (released in 2010), right? Was this done by purpose, or it's an error?

Anyway, the RHL behaviour was already broken with d83ec9e removing the old PAM config and init script.

[djmdjm]

Maybe @Jakuje could comment here?

[Jakuje]

I would say go ahead and remove them. I do not think anybody will build openssh from source on RHEL6 (or RHL 6) from upstream repo/tarball these days with the upstream spec file. If they run RHEL6 (hope not RHL6), they have contact for somebody to maintain the packages for them and I hope that somebody will not blindly pick up spec file from upstream when there is perfectly good working spec file in dist git.

[FabioPedretti]

Hi, note that the patch is not removing RHEL 6 (released in 2010), but just RHL6 (released in 1999).

The patch was intended to fix building on RHEL 6, which I still used (IIRC the patch just fixed a warning on RHEL 6).

[sincerelysaucy]

If it's going to stick around, the could should be labeled for RHEL 6, not "RHL 6". No on, and I mean no one referred to Red Hat 6 as RHL 6. And I helped companies update about 20,000 hosts form Red Hat 5 to Red Hat 6.