Web lists-archives.com

Re: kio-stash is in KDE Review




El divendres, 9 de juny de 2017, a les 11:45:14 CEST, Arnav Dhamija va 
escriure:
> > Do you mean i have to manually start a process for the tests to succeed?
> 
> If installed correctly with kdeinit restarted and the Stash Daemon is
> running in kded5, it is not required. However, I cannot recall what
> steps are needed for this to happen as I think it requires a system
> restart or logout to do so.
> 
> Hence, I have created a fallback testdaemon program which creates a
> daemon process for the Stash Daemon to run. This is launched
> automatically by the test suite if the test suite cannot find the
> Stash Daemon running in kded. I just patched it so that it finds the
> path to the testdaemon program correctly (it had the wrong path when
> you tested it). It should pass the test cases now with no manual
> invocation of the testdaemon process.

Doesn't seem to work for me.

https://paste.kde.org/ptnn5hhkt

Cheers,
  Albert

> 
> On Fri, Jun 9, 2017 at 1:47 AM, Albert Astals Cid <aacid@xxxxxxx> wrote:
> > El dijous, 8 de juny de 2017, a les 16:25:02 CEST, Arnav Dhamija va 
escriure:
> >> Yes, I will file a sysadmin ticket for the same. Until I do so, could
> >> you send me the output of the test suite in a screenshot or as plain
> >> text?
> >> 
> >> The ioslave works by storing a list of virtual files and folders in a
> >> process running in the KDED, communicating with it over D-Bus. Hence,
> >> some of the tests won't work unless a) the testdaemon process is
> >> started
> > 
> > Do you mean i have to manually start a process for the tests to succeed?
> > 
> > If that is the case, that's not acceptable.
> > 
> > Cheers,
> > 
> >   Albert
> >> 
> >> or b) the Stash KIO slave watcher is running in KDED. Usually
> >> it requires a restart of KDED and a kdeinit after installing the
> >> ioslave to have the Stash daemon running in KDED.
> >> 
> >> On Thu, Jun 8, 2017 at 3:42 AM, Albert Astals Cid <aacid@xxxxxxx> wrote:
> >> > El divendres, 26 de maig de 2017, a les 22:55:11 CEST, Arnav Dhamija va
> >> > 
> >> > escriure:
> >> >> Hi!
> >> >> 
> >> >> I had made the kio-stash ioslave last year as part of my GSoC 2016
> >> >> Project. Now that most of the changes which I wished to make in
> >> >> Dolphin relevant to the ioslave are part of master, I wish to ship the
> >> >> ioslave soon.
> >> >> 
> >> >> The project is now in KDE review and is awaiting feedback. The link
> >> >> for the same can be found here:
> >> >> https://phabricator.kde.org/R247:0f4dc333ac8469b36d1b3a1cedd0a6672ac90
> >> >> 7b8
> >> >> 
> >> >> The source of the project can be found here:
> >> >> https://cgit.kde.org/kio-stash.git/
> >> > 
> >> > Can you please get it covered by the CI?
> >> > 
> >> > I have some of the tests failing but don't know if it is because my
> >> > setup
> >> > is a bit broken at this moment, so wanted to check in CI if the tests
> >> > were file but I have not been able to find it there.
> >> > 
> >> > Cheers,
> >> > 
> >> >   Ablert
> >> >> 
> >> >> Thanks.