Web lists-archives.com

Re: kio-stash is in KDE Review




> 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.

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:0f4dc333ac8469b36d1b3a1cedd0a6672ac907b8
>> >>
>> >> 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.
>
>



-- 
arnav dhamija