OSX/CI: kio placed files erroneously due to missing required backslash in path

2014-12-22 Thread Marko Käning
Hi, on OSX it looks like some app places files directly in "~/Library", as it misses to insert a backslash in between the standard location "~/Library/Application Support” and the 3 files (user-places.xbel*) to be generated in there: --- $ ls -l1 Library/Application\ Supportuser-places.xbel* -r

Re: Review Request 121447: Return inode/directory when isDir returns true (kfileitem)

2014-12-22 Thread David Faure
> On Dec. 11, 2014, 2:27 p.m., Mark Gaiser wrote: > > src/core/kfileitem.cpp, line 255 > > > > > > --> add here > > > > } else { > > // Fix for IO slaves that don't set UDS_MIME_TYPE for a folder

Re: KPackage framework

2014-12-22 Thread David Edmundson
No objections from me ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 121447: Return inode/directory when isDir returns true (kfileitem)

2014-12-22 Thread Àlex Fiestas
> On des. 11, 2014, 2:27 p.m., Mark Gaiser wrote: > > src/core/kfileitem.cpp, line 255 > > > > > > --> add here > > > > } else { > > // Fix for IO slaves that don't set UDS_MIME_TYPE for a folder

Re: KPackage framework

2014-12-22 Thread Marco Martin
On Monday 10 November 2014, Marco Martin wrote: > Hi all, > since at akademy there seemed the interest in it, > I have been working on some classes i extracted from libplasma to be on > their own, those related to Plasma::Package, since several applications > have the interest of having scriptable

Re: New framework to review: KPackage

2014-12-22 Thread Marco Martin
On Wednesday 10 December 2014, David Edmundson wrote: > ​The binary is called kpackagetool. Given the complications we've had with > frameworks co-installability does it make sense to call it kpackagetool5? > > The class name in kpackagetool/kpackagetool.cpp should probably be renamed > > Documen

Re: Review Request 121447: Return inode/directory when isDir returns true (kfileitem)

2014-12-22 Thread Mark Gaiser
> On dec 11, 2014, 2:27 p.m., Mark Gaiser wrote: > > src/core/kfileitem.cpp, line 255 > > > > > > --> add here > > > > } else { > > // Fix for IO slaves that don't set UDS_MIME_TYPE for a folder.

Re: Review Request 121581: Plotter in kdeclarative

2014-12-22 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121581/ --- (Updated Dec. 22, 2014, 12:27 p.m.) Review request for KDE Frameworks and

Re: KPackage framework

2014-12-22 Thread David Faure
On Monday 22 December 2014 10:21:21 Marco Martin wrote: > On Saturday 20 December 2014, David Faure wrote: > > > in the branch mart/kpluginmetadata there is an experiment in porting > > > away > > > from kservice, tough if i go all the way it should have a complete own > > > copy of the plugin inde

Re: KPackage framework

2014-12-22 Thread Marco Martin
On Saturday 20 December 2014, David Faure wrote: > > in the branch mart/kpluginmetadata there is an experiment in porting away > > from kservice, tough if i go all the way it should have a complete own > > copy of the plugin indexing (and indexing tool) since if it doesn't > > depend from kservice

Re: Review Request 121447: Return inode/directory when isDir returns true (kfileitem)

2014-12-22 Thread David Faure
> On Dec. 11, 2014, 2:27 p.m., Mark Gaiser wrote: > > src/core/kfileitem.cpp, line 255 > > > > > > --> add here > > > > } else { > > // Fix for IO slaves that don't set UDS_MIME_TYPE for a folder