Review Request 125611: Make SharedMimeInfo an optional requirement

[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 macro, is that ok then?


- Christoph


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125611/#review86741
-----------------------------------------------------------


On Oct. 12, 2015, 5:51 p.m., Christoph Cullmann wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125611/
> -----------------------------------------------------------
> 
> (Updated Oct. 12, 2015, 5:51 p.m.)
> 
> 
> Review request for KDE Frameworks, David Faure and Kåre Särs.
> 
> 
> Repository: kcoreaddons
> 
> 
> Description
> -------
> 
> Make SharedMimeInfo an optional requirement, useful e.g. on win and mac. There is nothing like that normally and we can then avoid to install that just to have it around for the dependency check
> 
> 
> Diffs
> -----
> 
>   src/mimetypes/CMakeLists.txt 207b114 
> 
> Diff: https://git.reviewboard.kde.org/r/125611/diff/
> 
> 
> Testing
> -------
> 
> If the info is found, make install did still update the mime stuff.
> 
> 
> Thanks,
> 
> Christoph Cullmann
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151012/4158c387/attachment-0001.html>