It appears that the redhat/openssh.spec has not been updated to account for the removal of slogin from the build. This breaks rpmbuild. Specifically lines 361 and 362 still have the following: %attr(-,root,root) %{_bindir}/slogin %attr(-,root,root) %{_mandir}/man1/slogin.1* Robert Moucha Geophysics Syracuse University
On Mar 14 16:52, Robert Moucha wrote:> It appears that the redhat/openssh.spec has not been updated to account for the removal of slogin from the build. This breaks rpmbuild. > Specifically lines 361 and 362 still have the following: > > > %attr(-,root,root) %{_bindir}/slogin > > %attr(-,root,root) %{_mandir}/man1/slogin.1*IMHO the right thing to do is to revert the patch removing the symlinks. I can't think of any good reason to break backward compatibility with existing installations. The Cygwin and Fedora distros now maintain fixed spec files creating the symlinks by themselves for the time being. Corinna -- Corinna Vinschen Cygwin Maintainer Red Hat -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.mindrot.org/pipermail/openssh-unix-dev/attachments/20160314/9c92121c/attachment.bin>
On Mon, 14 Mar 2016, Corinna Vinschen wrote:> On Mar 14 16:52, Robert Moucha wrote: > > It appears that the redhat/openssh.spec has not been updated to account for the removal of slogin from the build. This breaks rpmbuild. > > Specifically lines 361 and 362 still have the following: > > > > > > %attr(-,root,root) %{_bindir}/slogin > > > > %attr(-,root,root) %{_mandir}/man1/slogin.1*thanks - I've removed them from the .spec files.> IMHO the right thing to do is to revert the patch removing the symlinks. > I can't think of any good reason to break backward compatibility with > existing installations. > > The Cygwin and Fedora distros now maintain fixed spec files creating > the symlinks by themselves for the time being.Well, I messed up by neglecting to mention the slogin removal from the 7.2 release notes but I don't plan on putting it back. Adapting is just s/slogin/ssh/ in scripts with 100% backwards-compatibility. If downstreams want to maintain a symlink in their scripts then that's naturally fine though. -d