Hi, too bad noone was up to push it finally into kde-runtime when it was ready for it, but let's pick it up again now:
Mardi, le 26 avril 2011, à 14:53, Nikhil Marathe a écrit: > Hi, > > KDE SC 4.7 soft feature freeze is close, and I would like to propose > the UPnP MediaServer KIO slave > (https://projects.kde.org/projects/playground/base/kio-upnp-ms/) be > included into the set of kio slaves shipped with kde-runtime. The > slave was created as part of GSoC 2010 - Amarok and KDE UPnP support > and it was decided that it should be merged into kde-runtime at some > point. > > A couple of reasons I believe the slave is now ready for standard release > is: 1) HUpnp (http://herqq.org/news.html) - the Qt based UPnP library used > by the slave has a stable API and ABI with the release of 1.0.0 about 3 > weeks ago. > 2) The slave has been considerably simplified and single threaded, and > stable now. > 3) The slave is independent and can be conditionally compiled and > installed if HUpnp is installed. kdelibs already contains a > FindHUpnp.cmake to find the HUpnp library. > 4) The Solid UPnP backend (enabled in 4.7, again if HUpnp is found) > automatically launches UPnP media servers in the file manager with the > slave. 4b) And that Places integration of 4) will be useless if there is no upnp-ms kio-slave, people would click on the mediaserver icon but see nothing in the list 5) It is also used by Amarok since 2.4.0 (if found) 6) It adds no further dependencies than there are with kdelibs (i.e. HUpnp as optional library, if not found the kio-slave will be simply skipped). So UPnP support in Solid is aligned with kio-upnp-ms here. > My exams get over this week and I can ensure that krazy checks pass > and the code is cleaned up some more. But then Nikhil had his job life interfering, so could not spend any time on pushing this request forward :( I did a short code review, looked fine (could not comment on HUPnP usage, but who can besides Nikhil?), and some code optimization patches I did even got merged by Nikhil meanwhile. > There is inline documentation where required, and the search and > browse API documentation exists. There is no user > manual since it is a slave. I am confident about having it ready by > hard feature freeze. I also would consider it ready for first serious usage. After all it will enable all KIO-enabled programs to access the media content on UPnP media servers out there (e.g. Xbox), at least to a certain degree. So, release-team, what would you think about an exception to still include the upnp-ms kio-slave in kde-runtime 4.7? After all it was ready, just noone replied to him besides Kevin and me both saying "Keep it coming!". It does not affect any other code, as it is an isolated kio-slave. Might not even be compiled if some distro has not yet HUpnp packages. But if there are HUpnp packages the Places integration (Dolphin/filedialog) for UPnP media servers will be useless without this kio-slave. So distros would have to install this kio-slave anyway. So it might make sense to have the kio-slave already in kde-runtime, instead of extragear, where I would like to push it otherwise, so it can be released together with the SC 4.7 officially and distros know about this additional feature/kio-slave. If you say No (understandable): What is the process to make the playground/base repo of kio-upnp-ms an extragear/base repo in times of git repos? Could not find it documented on techbase, pointers welcome. And how would that repo be marked that it should be included in the SC 4.7 release process? If you say Yes (would be welcome): Tomorrow/Today evening (wednesday, 29 june) I would see to create a branch from kde-runtime master where the commit history of the kio-upnp-ms repo is merged (been there, done that), have that reviewed by someone and then backport that merge to 4.7 branch. Okay? Cheers Friedrich -- Desktop Summit 2011 in Berlin - Registered already? - www.desktopsummit.org _______________________________________________ release-team mailing list release-team@kde.org https://mail.kde.org/mailman/listinfo/release-team