Re: Review Request 125611: Make SharedMimeInfo an optional requirement

2015-10-12 Thread Christoph Cullmann
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125611/ --- (Updated Oct. 12, 2015, 8:01 p.m.) Status -- This change has been ma

Re: Review Request 125611: Make SharedMimeInfo an optional requirement

2015-10-12 Thread David Faure
> On Oct. 12, 2015, 7:48 p.m., David Faure wrote: > > I think you should install the xml file even if s-m-i isn't found. > > > > Otherwise, if someone installs kcoreaddons before s-m-i, he'll be missing > > the file. > > Christoph Cullmann wrote: > Ok, I will install the xml but skip the c

Re: Review Request 125611: Make SharedMimeInfo an optional requirement

2015-10-12 Thread Christoph Cullmann
> On Oct. 12, 2015, 7:48 p.m., David Faure wrote: > > I think you should install the xml file even if s-m-i isn't found. > > > > Otherwise, if someone installs kcoreaddons before s-m-i, he'll be missing > > the file. Ok, I will install the xml but skip the call to the then not known update ma

Re: Review Request 125611: Make SharedMimeInfo an optional requirement

2015-10-12 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125611/#review86741 --- I think you should install the xml file even if s-m-i isn't fo