Some more projects in need of respecting KF5_INSTALL_TARGETS_DEFAULT_ARGS

2015-01-06 Thread Marko Käning
Hi Christoph,

I’ve found that these projects do not make use of 
KF5_INSTALL_TARGETS_DEFAULT_ARGS
at the moment:

 - systemsettings
 - muon
 - rocs
 - libkdegames
 - kiten
 - cantor
 - kolourpaint
 - libkmahjongg

See for details below in order of appearance.

I figure this is only the beginning of it and more projects might turn up in 
the future.

Thanks for taking care of these.

Regards,
Marko




P.S.: I’ve tested locally to prepend "KF5_” to INSTALL_TARGETS_DEFAULT_ARGS
  for project systemsettings only, which worked fine and made the project
  install successfully again here on my OSX/CI system.




P.P.S.: I realised by now that there are no changes on the production branch,
which probably means that there was some change in ECM or wherever
provoking these issues...




---

systemsettings:

CMake Error at core/CMakeLists.txt:37 (install):
  install TARGETS given no LIBRARY DESTINATION for shared library target
  "systemsettingsview”.
---

muon:

CMake Error at libmuon/notifiers/CMakeLists.txt:7 (install):
  install TARGETS given no LIBRARY DESTINATION for shared library target
  "MuonNotifiers".

CMake Error at libmuon/CMakeLists.txt:63 (install):
  install TARGETS given no LIBRARY DESTINATION for shared library target
  "muonprivate”.
---

rocs:

CMake Error at libgraphtheory/CMakeLists.txt:106 (install):
  install TARGETS given no LIBRARY DESTINATION for shared library target
  "rocsgraphtheory”.
---

libkdegames:

CMake Error at CMakeLists.txt:163 (install):
  install TARGETS given no LIBRARY DESTINATION for shared library target
  "KF5KDEGames".

CMake Error at CMakeLists.txt:222 (install):
  install TARGETS given no LIBRARY DESTINATION for shared library target
  "KF5KDEGamesPrivate”.
---

kiten:

CMake Error at lib/CMakeLists.txt:37 (install):
  install TARGETS given no LIBRARY DESTINATION for shared library target
  "kiten".
---

cantor:

CMake Error at src/lib/CMakeLists.txt:75 (install):
  install TARGETS given no LIBRARY DESTINATION for shared library target
  "cantorlibs”.

CMake Error at src/CMakeLists.txt:25 (install):
  install TARGETS given no LIBRARY DESTINATION for shared library target
  "cantor_config".
---

kolourpaint:

CMake Error at CMakeLists.txt:579 (install):
  install TARGETS given no LIBRARY DESTINATION for shared library target
  "kolourpaint_lgpl".
---

libkmahjongg:

CMake Error at CMakeLists.txt:64 (install):
  install TARGETS given no LIBRARY DESTINATION for shared library target
  "KF5KMahjongglib".

CMake Error at CMakeLists.txt:67 (install):
  install TARGETS given no LIBRARY DESTINATION for shared library target
  "KF5KMahjongglib".
---

___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Some more projects in need of respecting KF5_INSTALL_TARGETS_DEFAULT_ARGS

2015-01-06 Thread Jeremy Whiting
Yes, ecm changed recently.

On Tue, Jan 6, 2015 at 3:55 PM, Marko Käning  wrote:

> Hi Christoph,
>
> I’ve found that these projects do not make use of
> KF5_INSTALL_TARGETS_DEFAULT_ARGS
> at the moment:
>
>  - systemsettings
>  - muon
>  - rocs
>  - libkdegames
>  - kiten
>  - cantor
>  - kolourpaint
>  - libkmahjongg
>
> See for details below in order of appearance.
>
> I figure this is only the beginning of it and more projects might turn up
> in the future.
>
> Thanks for taking care of these.
>
> Regards,
> Marko
>
>
>
>
> P.S.: I’ve tested locally to prepend "KF5_” to INSTALL_TARGETS_DEFAULT_ARGS
>   for project systemsettings only, which worked fine and made the
> project
>   install successfully again here on my OSX/CI system.
>
>
>
>
> P.P.S.: I realised by now that there are no changes on the production
> branch,
> which probably means that there was some change in ECM or wherever
> provoking these issues...
>
>
>
>
> ---
>
> systemsettings:
>
> CMake Error at core/CMakeLists.txt:37 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "systemsettingsview”.
> ---
>
> muon:
>
> CMake Error at libmuon/notifiers/CMakeLists.txt:7 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "MuonNotifiers".
>
> CMake Error at libmuon/CMakeLists.txt:63 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "muonprivate”.
> ---
>
> rocs:
>
> CMake Error at libgraphtheory/CMakeLists.txt:106 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "rocsgraphtheory”.
> ---
>
> libkdegames:
>
> CMake Error at CMakeLists.txt:163 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "KF5KDEGames".
>
> CMake Error at CMakeLists.txt:222 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "KF5KDEGamesPrivate”.
> ---
>
> kiten:
>
> CMake Error at lib/CMakeLists.txt:37 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "kiten".
> ---
>
> cantor:
>
> CMake Error at src/lib/CMakeLists.txt:75 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "cantorlibs”.
>
> CMake Error at src/CMakeLists.txt:25 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "cantor_config".
> ---
>
> kolourpaint:
>
> CMake Error at CMakeLists.txt:579 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "kolourpaint_lgpl".
> ---
>
> libkmahjongg:
>
> CMake Error at CMakeLists.txt:64 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "KF5KMahjongglib".
>
> CMake Error at CMakeLists.txt:67 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "KF5KMahjongglib".
> ---
>
> ___
> Kde-frameworks-devel mailing list
> Kde-frameworks-devel@kde.org
> https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
>
___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Some more projects in need of respecting KF5_INSTALL_TARGETS_DEFAULT_ARGS

2015-01-06 Thread Kevin Funk
On Tuesday 06 January 2015 23:55:48 Marko Käning wrote:
> Hi Christoph,
> 
> I’ve found that these projects do not make use of
> KF5_INSTALL_TARGETS_DEFAULT_ARGS at the moment:
> 
>  - systemsettings
>  - muon
>  - rocs
>  - libkdegames
>  - kiten
>  - cantor
>  - kolourpaint
>  - libkmahjongg
> 
> See for details below in order of appearance.
> 
> I figure this is only the beginning of it and more projects might turn up in
> the future.

Is KF5_INSTALL_TARGETS_DEFAULT_ARGS even correct for non-frameworks projects?
KF5_INSTALL_TARGETS_DEFAULT_ARGS's INCLUDES location points to $SOMETHING/KF5.

KDEInstallDirs.cmake also has a KDE_INSTALL_TARGETS_DEFAULT_ARGS, which seems 
more appropriate.

@Alex, please elaborate.

> 
> Thanks for taking care of these.
> 
> Regards,
> Marko
> 
> 
> 
> 
> P.S.: I’ve tested locally to prepend "KF5_” to INSTALL_TARGETS_DEFAULT_ARGS
>   for project systemsettings only, which worked fine and made the
> project install successfully again here on my OSX/CI system.
> 
> 
> 
> 
> P.P.S.: I realised by now that there are no changes on the production
> branch, which probably means that there was some change in ECM or wherever
> provoking these issues...
> 
> 
> 
> 
> ---
> 
> systemsettings:
> 
> CMake Error at core/CMakeLists.txt:37 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "systemsettingsview”.
> ---
> 
> muon:
> 
> CMake Error at libmuon/notifiers/CMakeLists.txt:7 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "MuonNotifiers".
> 
> CMake Error at libmuon/CMakeLists.txt:63 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "muonprivate”.
> ---
> 
> rocs:
> 
> CMake Error at libgraphtheory/CMakeLists.txt:106 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "rocsgraphtheory”.
> ---
> 
> libkdegames:
> 
> CMake Error at CMakeLists.txt:163 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "KF5KDEGames".
> 
> CMake Error at CMakeLists.txt:222 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "KF5KDEGamesPrivate”.
> ---
> 
> kiten:
> 
> CMake Error at lib/CMakeLists.txt:37 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "kiten".
> ---
> 
> cantor:
> 
> CMake Error at src/lib/CMakeLists.txt:75 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "cantorlibs”.
> 
> CMake Error at src/CMakeLists.txt:25 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "cantor_config".
> ---
> 
> kolourpaint:
> 
> CMake Error at CMakeLists.txt:579 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "kolourpaint_lgpl".
> ---
> 
> libkmahjongg:
> 
> CMake Error at CMakeLists.txt:64 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "KF5KMahjongglib".
> 
> CMake Error at CMakeLists.txt:67 (install):
>   install TARGETS given no LIBRARY DESTINATION for shared library target
>   "KF5KMahjongglib".
> ---
> 
> ___
> Kde-frameworks-devel mailing list
> Kde-frameworks-devel@kde.org
> https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

-- 
Kevin Funk | kf...@kde.org | http://kfunk.org
___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Some more projects in need of respecting KF5_INSTALL_TARGETS_DEFAULT_ARGS

2015-01-06 Thread Christoph Cullmann
> On Tuesday 06 January 2015 23:55:48 Marko Käning wrote:
> > Hi Christoph,
> > 
> > I’ve found that these projects do not make use of
> > KF5_INSTALL_TARGETS_DEFAULT_ARGS at the moment:
> > 
> >  - systemsettings
> >  - muon
> >  - rocs
> >  - libkdegames
> >  - kiten
> >  - cantor
> >  - kolourpaint
> >  - libkmahjongg
> > 
> > See for details below in order of appearance.
> > 
> > I figure this is only the beginning of it and more projects might turn up
> > in
> > the future.
> 
> Is KF5_INSTALL_TARGETS_DEFAULT_ARGS even correct for non-frameworks projects?
> KF5_INSTALL_TARGETS_DEFAULT_ARGS's INCLUDES location points to
> $SOMETHING/KF5.
> 
> KDEInstallDirs.cmake also has a KDE_INSTALL_TARGETS_DEFAULT_ARGS, which seems
> more appropriate.
> 
> @Alex, please elaborate.
Yeah, for non-framework repos, I am not sure of that, true.

Greetings
Christoph

> 
> > 
> > Thanks for taking care of these.
> > 
> > Regards,
> > Marko
> > 
> > 
> > 
> > 
> > P.S.: I’ve tested locally to prepend "KF5_” to INSTALL_TARGETS_DEFAULT_ARGS
> >   for project systemsettings only, which worked fine and made the
> > project install successfully again here on my OSX/CI system.
> > 
> > 
> > 
> > 
> > P.P.S.: I realised by now that there are no changes on the production
> > branch, which probably means that there was some change in ECM or wherever
> > provoking these issues...
> > 
> > 
> > 
> > 
> > ---
> > 
> > systemsettings:
> > 
> > CMake Error at core/CMakeLists.txt:37 (install):
> >   install TARGETS given no LIBRARY DESTINATION for shared library target
> >   "systemsettingsview”.
> > ---
> > 
> > muon:
> > 
> > CMake Error at libmuon/notifiers/CMakeLists.txt:7 (install):
> >   install TARGETS given no LIBRARY DESTINATION for shared library target
> >   "MuonNotifiers".
> > 
> > CMake Error at libmuon/CMakeLists.txt:63 (install):
> >   install TARGETS given no LIBRARY DESTINATION for shared library target
> >   "muonprivate”.
> > ---
> > 
> > rocs:
> > 
> > CMake Error at libgraphtheory/CMakeLists.txt:106 (install):
> >   install TARGETS given no LIBRARY DESTINATION for shared library target
> >   "rocsgraphtheory”.
> > ---
> > 
> > libkdegames:
> > 
> > CMake Error at CMakeLists.txt:163 (install):
> >   install TARGETS given no LIBRARY DESTINATION for shared library target
> >   "KF5KDEGames".
> > 
> > CMake Error at CMakeLists.txt:222 (install):
> >   install TARGETS given no LIBRARY DESTINATION for shared library target
> >   "KF5KDEGamesPrivate”.
> > ---
> > 
> > kiten:
> > 
> > CMake Error at lib/CMakeLists.txt:37 (install):
> >   install TARGETS given no LIBRARY DESTINATION for shared library target
> >   "kiten".
> > ---
> > 
> > cantor:
> > 
> > CMake Error at src/lib/CMakeLists.txt:75 (install):
> >   install TARGETS given no LIBRARY DESTINATION for shared library target
> >   "cantorlibs”.
> > 
> > CMake Error at src/CMakeLists.txt:25 (install):
> >   install TARGETS given no LIBRARY DESTINATION for shared library target
> >   "cantor_config".
> > ---
> > 
> > kolourpaint:
> > 
> > CMake Error at CMakeLists.txt:579 (install):
> >   install TARGETS given no LIBRARY DESTINATION for shared library target
> >   "kolourpaint_lgpl".
> > ---
> > 
> > libkmahjongg:
> > 
> > CMake Error at CMakeLists.txt:64 (install):
> >   install TARGETS given no LIBRARY DESTINATION for shared library target
> >   "KF5KMahjongglib".
> > 
> > CMake Error at CMakeLists.txt:67 (install):
> >   install TARGETS given no LIBRARY DESTINATION for shared library target
> >   "KF5KMahjongglib".
> > ---
> > 
> > ___
> > Kde-frameworks-devel mailing list
> > Kde-frameworks-devel@kde.org
> > https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
> 
> --
> Kevin Funk | kf...@kde.org | http://kfunk.org
> 

-- 
- Dr.-Ing. Christoph Cullmann -
AbsInt Angewandte Informatik GmbH  Email: cullm...@absint.com
Science Park 1 Tel:   +49-681-38360-22
66123 Saarbrücken  Fax:   +49-681-38360-20
GERMANYWWW:   http://www.AbsInt.com

Geschäftsführung: Dr.-Ing. Christian Ferdinand
Eingetragen im Handelsregister des Amtsgerichts Saarbrücken, HRB 11234
___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Some more projects in need of respecting KF5_INSTALL_TARGETS_DEFAULT_ARGS

2015-01-07 Thread David Faure
On Wednesday 07 January 2015 08:38:27 Kevin Funk wrote:
> On Tuesday 06 January 2015 23:55:48 Marko Käning wrote:
> > Hi Christoph,
> > 
> > I’ve found that these projects do not make use of
> > 
> > KF5_INSTALL_TARGETS_DEFAULT_ARGS at the moment:
> >  - systemsettings
> >  - muon
> >  - rocs
> >  - libkdegames
> >  - kiten
> >  - cantor
> >  - kolourpaint
> >  - libkmahjongg
> > 
> > See for details below in order of appearance.
> > 
> > I figure this is only the beginning of it and more projects might turn up
> > in the future.
> 
> Is KF5_INSTALL_TARGETS_DEFAULT_ARGS even correct for non-frameworks
> projects? KF5_INSTALL_TARGETS_DEFAULT_ARGS's INCLUDES location points to
> $SOMETHING/KF5.
> 
> KDEInstallDirs.cmake also has a KDE_INSTALL_TARGETS_DEFAULT_ARGS, which
> seems more appropriate.

I think you're fully correct.

-- 
David Faure, fa...@kde.org, http://www.davidfaure.fr
Working on KDE Frameworks 5

___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Some more projects in need of respecting KF5_INSTALL_TARGETS_DEFAULT_ARGS

2015-01-07 Thread Alex Merry
On Wednesday 07 January 2015 08:38:27 Kevin Funk wrote:
> On Tuesday 06 January 2015 23:55:48 Marko Käning wrote:
> > Hi Christoph,
> > 
> > I’ve found that these projects do not make use of
> > 
> > KF5_INSTALL_TARGETS_DEFAULT_ARGS at the moment:
> >  - systemsettings
> >  - muon
> >  - rocs
> >  - libkdegames
> >  - kiten
> >  - cantor
> >  - kolourpaint
> >  - libkmahjongg
> > 
> > See for details below in order of appearance.
> > 
> > I figure this is only the beginning of it and more projects might turn up
> > in the future.
> 
> Is KF5_INSTALL_TARGETS_DEFAULT_ARGS even correct for non-frameworks
> projects? KF5_INSTALL_TARGETS_DEFAULT_ARGS's INCLUDES location points to
> $SOMETHING/KF5.
> 
> KDEInstallDirs.cmake also has a KDE_INSTALL_TARGETS_DEFAULT_ARGS, which
> seems more appropriate.
> 
> @Alex, please elaborate.

Nope, any variable from KDEInstallDirs.cmake with KF5 in the name is only for 
the use of Frameworks. They all have a non-KF5 equivalent - in this case, 
KDE_INSTALL_TARGETS_DEFAULT_ARGS, as you noticed.

I think KF5_INSTALL_TARGETS_DEFAULT_ARGS should actually be 
KDE_INSTALL_TARGETS_DEFAULT_ARGS_KF5 for consistency with other variables, now 
I look again.

Alex
___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Some more projects in need of respecting KF5_INSTALL_TARGETS_DEFAULT_ARGS

2015-01-07 Thread Alex Merry
On Tuesday 06 January 2015 23:55:48 Marko Käning wrote:
> P.P.S.: I realised by now that there are no changes on the production
> branch, which probably means that there was some change in ECM or wherever
> provoking these issues...

I'm somewhat puzzled, as INSTALL_TARGET_DEFAULT_ARGS should still be set (to 
the same value as KDE_INSTALL_TARGET_DEFAULT_ARGS) unless you have set 
KDE_INSTALL_DIRS_NO_DEPRECATED to some TRUE-ish value. If not, that's a bug.

Alex
___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Some more projects in need of respecting KF5_INSTALL_TARGETS_DEFAULT_ARGS

2015-01-07 Thread Jeremy Whiting
I hit the same issues reported by Marko here on OS X also, and have no
KDE_INSTALL_DIRS_NO_DEPRECATED set either. Maybe the logic in ecm is broken
somehow?

On Wed, Jan 7, 2015 at 11:48 AM, Alex Merry  wrote:

> On Tuesday 06 January 2015 23:55:48 Marko Käning wrote:
> > P.P.S.: I realised by now that there are no changes on the production
> > branch, which probably means that there was some change in ECM or
> wherever
> > provoking these issues...
>
> I'm somewhat puzzled, as INSTALL_TARGET_DEFAULT_ARGS should still be set
> (to
> the same value as KDE_INSTALL_TARGET_DEFAULT_ARGS) unless you have set
> KDE_INSTALL_DIRS_NO_DEPRECATED to some TRUE-ish value. If not, that's a
> bug.
>
> Alex
> ___
> Kde-frameworks-devel mailing list
> Kde-frameworks-devel@kde.org
> https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
>
___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Some more projects in need of respecting KF5_INSTALL_TARGETS_DEFAULT_ARGS

2015-01-07 Thread Marko Käning
Hi Alex,
Hi Ben,

Happy New Year, for a start! :)

On 07 Jan 2015, at 19:48 , Alex Merry  wrote:

> On Tuesday 06 January 2015 23:55:48 Marko Käning wrote:
>> P.P.S.: I realised by now that there are no changes on the production
>> branch, which probably means that there was some change in ECM or wherever
>> provoking these issues...
> 
> I'm somewhat puzzled, as INSTALL_TARGET_DEFAULT_ARGS should still be set (to 
> the same value as KDE_INSTALL_TARGET_DEFAULT_ARGS) unless you have set 
> KDE_INSTALL_DIRS_NO_DEPRECATED to some TRUE-ish value. If not, that's a bug.


Hmmm...

Well, off-list I already had contacted Ben because of this issue. See this:


Begin forwarded message:
> On 07 Jan 2015, at 09:12 , David Faure  wrote:
>>> KDEInstallDirs.cmake also has a KDE_INSTALL_TARGETS_DEFAULT_ARGS, which
>>> seems more appropriate.
>> 
>> I think you're fully correct.
> 
> Thus I tried KDE_INSTALL_TARGETS_DEFAULT_ARGS for the non-KF5 “systemsettings”
> project but it lead to the same error as the original 
> INSTALL_TARGETS_DEFAULT_ARGS
> variable in my initial post.
> 
> Is this perhaps specific to the CI system?



So, I really wonder what’s going on here, as it doesn’t hit all projects! 
kstars fails,
but marble - for instance - still builds fine without the need to mess with 
these vars!

I can't find KDE_INSTALL_DIRS_NO_DEPRECATED in CMakeCache.txt of kstars at all, 
so I assume
that it is not set.

Does this mean it IS a bug for kstars and all the other projects after 
all?

Marko
___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Some more projects in need of respecting KF5_INSTALL_TARGETS_DEFAULT_ARGS

2015-01-07 Thread Christoph Cullmann
Hi,

I see why KDE_INSTALL_TARGETS_DEFAULT_ARGS and INSTALL_TARGETS_DEFAULT_ARGS 
fail on Mac,
typo:

# on the Mac support an extra install directory for application bundles
if(APPLE)
set(KDE_INSTALL_TARGETS_DEFAULT_ARGS  ${INSTALL_TARGETS_DEFAULT_ARGS}
  BUNDLE DESTINATION 
"${BUNDLE_INSTALL_DIR}" )
set(KF5_INSTALL_TARGETS_DEFAULT_ARGS  ${KF5_INSTALL_TARGETS_DEFAULT_ARGS}
  BUNDLE DESTINATION 
"${BUNDLE_INSTALL_DIR}" )
endif(APPLE)

should be

# on the Mac support an extra install directory for application bundles
if(APPLE)
set(KDE_INSTALL_TARGETS_DEFAULT_ARGS  ${KDE_INSTALL_TARGETS_DEFAULT_ARGS}
  BUNDLE DESTINATION 
"${BUNDLE_INSTALL_DIR}" )
set(KF5_INSTALL_TARGETS_DEFAULT_ARGS  ${KF5_INSTALL_TARGETS_DEFAULT_ARGS}
  BUNDLE DESTINATION 
"${BUNDLE_INSTALL_DIR}" )
endif(APPLE)

will patch that ;=)

- Ursprüngliche Mail -
> Hi Alex,
> Hi Ben,
> 
> Happy New Year, for a start! :)
> 
> On 07 Jan 2015, at 19:48 , Alex Merry  wrote:
> 
> > On Tuesday 06 January 2015 23:55:48 Marko Käning wrote:
> >> P.P.S.: I realised by now that there are no changes on the production
> >> branch, which probably means that there was some change in ECM or wherever
> >> provoking these issues...
> > 
> > I'm somewhat puzzled, as INSTALL_TARGET_DEFAULT_ARGS should still be set
> > (to
> > the same value as KDE_INSTALL_TARGET_DEFAULT_ARGS) unless you have set
> > KDE_INSTALL_DIRS_NO_DEPRECATED to some TRUE-ish value. If not, that's a
> > bug.
> 
> 
> Hmmm...
> 
> Well, off-list I already had contacted Ben because of this issue. See this:
> 
> 
> Begin forwarded message:
> > On 07 Jan 2015, at 09:12 , David Faure  wrote:
> >>> KDEInstallDirs.cmake also has a KDE_INSTALL_TARGETS_DEFAULT_ARGS, which
> >>> seems more appropriate.
> >> 
> >> I think you're fully correct.
> > 
> > Thus I tried KDE_INSTALL_TARGETS_DEFAULT_ARGS for the non-KF5
> > “systemsettings”
> > project but it lead to the same error as the original
> > INSTALL_TARGETS_DEFAULT_ARGS
> > variable in my initial post.
> > 
> > Is this perhaps specific to the CI system?
> 
> 
> 
> So, I really wonder what’s going on here, as it doesn’t hit all projects!
> kstars fails,
> but marble - for instance - still builds fine without the need to mess with
> these vars!
> 
> I can't find KDE_INSTALL_DIRS_NO_DEPRECATED in CMakeCache.txt of kstars at
> all, so I assume
> that it is not set.
> 
>   Does this mean it IS a bug for kstars and all the other projects after 
> all?
> 
> Marko
> ___
> Kde-frameworks-devel mailing list
> Kde-frameworks-devel@kde.org
> https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
> 

-- 
- Dr.-Ing. Christoph Cullmann -
AbsInt Angewandte Informatik GmbH  Email: cullm...@absint.com
Science Park 1 Tel:   +49-681-38360-22
66123 Saarbrücken  Fax:   +49-681-38360-20
GERMANYWWW:   http://www.AbsInt.com

Geschäftsführung: Dr.-Ing. Christian Ferdinand
Eingetragen im Handelsregister des Amtsgerichts Saarbrücken, HRB 11234
___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Some more projects in need of respecting KF5_INSTALL_TARGETS_DEFAULT_ARGS

2015-01-07 Thread Marko Käning

On 07 Jan 2015, at 20:15 , Christoph Cullmann  wrote:
> will patch that ;=)

Thanks for patching this so promptly!

:-)
___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Some more projects in need of respecting KF5_INSTALL_TARGETS_DEFAULT_ARGS

2015-01-07 Thread Alex Merry
On Wednesday 07 January 2015 20:15:54 Christoph Cullmann wrote:
> Hi,
> 
> I see why KDE_INSTALL_TARGETS_DEFAULT_ARGS and INSTALL_TARGETS_DEFAULT_ARGS
> fail on Mac, typo:
> 
> # on the Mac support an extra install directory for application bundles
> if(APPLE)
> set(KDE_INSTALL_TARGETS_DEFAULT_ARGS  ${INSTALL_TARGETS_DEFAULT_ARGS}
>   BUNDLE DESTINATION
> "${BUNDLE_INSTALL_DIR}" ) set(KF5_INSTALL_TARGETS_DEFAULT_ARGS 
> ${KF5_INSTALL_TARGETS_DEFAULT_ARGS} BUNDLE DESTINATION
> "${BUNDLE_INSTALL_DIR}" ) endif(APPLE)
> 
> should be
> 
> # on the Mac support an extra install directory for application bundles
> if(APPLE)
> set(KDE_INSTALL_TARGETS_DEFAULT_ARGS 
> ${KDE_INSTALL_TARGETS_DEFAULT_ARGS} BUNDLE DESTINATION
> "${BUNDLE_INSTALL_DIR}" ) set(KF5_INSTALL_TARGETS_DEFAULT_ARGS 
> ${KF5_INSTALL_TARGETS_DEFAULT_ARGS} BUNDLE DESTINATION
> "${BUNDLE_INSTALL_DIR}" ) endif(APPLE)

Oops. Well spotted! That wouldn't be caught by the unit tests, either, which 
just check the variable is set.

Alex
___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel