Re: FindInotify devel/kf5/extra-cmake-modules

2020-05-22 Thread Antoine Jacoutot
On Fri, May 22, 2020 at 03:46:52PM +0200, Rafael Sadowski wrote: > On Fri May 22, 2020 at 08:57:21AM +0100, Stuart Henderson wrote: > > IIRC it's installed there on purpose, exactly so that it is not > > automatically picked up. > > > > Fair enough, here another try. Just help cmake if inotify

Re: FindInotify devel/kf5/extra-cmake-modules

2020-05-22 Thread Rafael Sadowski
On Fri May 22, 2020 at 08:57:21AM +0100, Stuart Henderson wrote: > IIRC it's installed there on purpose, exactly so that it is not > automatically picked up. > Fair enough, here another try. Just help cmake if inotify is required "find_package(inotify REQUIRED)". This make porting consumer less

Re: FindInotify devel/kf5/extra-cmake-modules

2020-05-22 Thread Stuart Henderson
On 2020/05/22 12:10, Antoine Jacoutot wrote: > On Fri, May 22, 2020 at 08:57:21AM +0100, Stuart Henderson wrote: > > IIRC it's installed there on purpose, exactly so that it is not > > automatically picked up. > > Indeed. > But should we change the rule now and install it under the default path?

Re: FindInotify devel/kf5/extra-cmake-modules

2020-05-22 Thread Antoine Jacoutot
On Fri, May 22, 2020 at 08:57:21AM +0100, Stuart Henderson wrote: > IIRC it's installed there on purpose, exactly so that it is not > automatically picked up. Indeed. But should we change the rule now and install it under the default path? > On 22 May 2020 07:28:55 Rafael Sadowski wrote: > >

Re: FindInotify devel/kf5/extra-cmake-modules

2020-05-22 Thread Stuart Henderson
IIRC it's installed there on purpose, exactly so that it is not automatically picked up. On 22 May 2020 07:28:55 Rafael Sadowski wrote: Simple patch to fix FindInotify. CMake needs some help to find "sys/inotify.h" and libinotify because we do not install it under LOCALBASE/{include,lib}.

FindInotify devel/kf5/extra-cmake-modules

2020-05-22 Thread Rafael Sadowski
Simple patch to fix FindInotify. CMake needs some help to find "sys/inotify.h" and libinotify because we do not install it under LOCALBASE/{include,lib}. OK? Index: Makefile === RCS file: