D17650: Install kioslave as kioslave5 on Windows

[Ralf Habacker]

habacker added a comment.


  > I'd argue that the problem is with your distribution.
  
  It is designed in ECM to install mentioned helpers on windows (which are cross compiled packages on obs) in main executable install dir.
  
    $ rpm -q -f /usr/i686-w64-mingw32/sys-root/mingw/bin/kioslave.exe 
    mingw32-kdelibs4-4.14.60-30.27.noarch
    $ rpm -q -f /usr/i686-w64-mingw32/sys-root/mingw/bin/kioslave.exe 
    mingw32-kio-5.47.0-11.28.noarch
  
  > Everybody still shipping qt4 is on its own anyway,
  
  Unfortunally yes, for example https://build.opensuse.org/package/show/windows%3Amingw%3Awin32/mingw32-kdelibs4, but this is unrelated, we are about talking about a patch for KF5, not qt4.
  
  > seeing that it's not possible to upstream patches (and there are quite a few needed to keep it building) as the git repo is locked down.
  
  It needs someone to take over maintenance of kdelibs.
  
  > And I'm pretty sure this patch breaks at least one other framework.
  
  no, it is only used in kio internally by klauncher to start io slaves.
  
  > Additionally there might be future maintenance costs,
  
  no, it simply follows the name style other kio executables installed in normal 'bin' dir already have e.g kcookiejar5, ktelnetservices5, ktrash5.
  
  > so I see no reason to flog a dead horse and penalize everybody else for it.
  
  In recent years, of the 104 KDE4 applications for Windows (https://community.kde.org/Windows/Releases/4.10.2), only 7 are available for KF5 (https://community.kde.org/Windows, minus the stable versions of Umbrello and KMymoney that are still on KDE4).  KDE4 will probably stay as a base for a while.

REPOSITORY
  R241 KIO

REVISION DETAIL
  https://phabricator.kde.org/D17650

To: habacker, #frameworks, lbeltrame, vonreth
Cc: heikobecker, ngraham, lbeltrame, kde-frameworks-devel, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181221/0cdd193f/attachment-0001.html>