Re: Review Request 129143: Fix loading plasmoids by absolute path, fixes configmodeltest

2016-10-11 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129143/ --- (Updated Oct. 11, 2016, 1:33 p.m.) Status -- This change has been ma

Re: Review Request 129143: Fix loading plasmoids by absolute path, fixes configmodeltest

2016-10-11 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129143/#review99935 --- Ship it! Ship It! - Marco Martin On Oct. 10, 2016, 9:48

Re: Review Request 129143: Fix loading plasmoids by absolute path, fixes configmodeltest

2016-10-11 Thread Marco Martin
> On Oct. 11, 2016, 8:52 a.m., Kai Uwe Broulik wrote: > > I have a FolderView applet in my panel (pointing to Downloads folder) and > > it is blank with this patch now. ok, so please don't push this yet - Marco --- This is an automatic

Re: Review Request 129143: Fix loading plasmoids by absolute path, fixes configmodeltest

2016-10-11 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129143/#review99923 --- I have a FolderView applet in my panel (pointing to Downloads

Re: Review Request 129143: Fix loading plasmoids by absolute path, fixes configmodeltest

2016-10-11 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129143/#review99921 --- Ship it! this version seems to work well - Marco Martin

Re: Review Request 129143: Fix loading plasmoids by absolute path, fixes configmodeltest

2016-10-10 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129143/ --- (Updated Oct. 10, 2016, 11:48 p.m.) Review request for KDE Frameworks and

Re: Review Request 129143: Fix loading plasmoids by absolute path, fixes configmodeltest

2016-10-10 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129143/#review99917 --- I confirm Kai's findings, to me it breaks loading of the whol

Re: Review Request 129143: Fix loading plasmoids by absolute path, fixes configmodeltest

2016-10-10 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129143/#review99914 --- Ship it! Ship It! - Marco Martin On Oct. 10, 2016, 6:44

Review Request 129143: Fix loading plasmoids by absolute path, fixes configmodeltest

2016-10-10 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129143/ --- Review request for KDE Frameworks and Plasma. Repository: plasma-framewor