[cmake-developers] [CMake 0015158]: MemCheck: Valgrind with multiple supressions files

2014-09-16 Thread Mantis Bug Tracker
The following issue has been SUBMITTED. == http://www.cmake.org/Bug/view.php?id=15158 == Reported By:trsystran Assigned To:

[cmake-developers] vs-nsight-tegra-generator topic

2014-09-16 Thread Brad King
Hi Folks, This topic introduces support for generating VS project files for the NVIDIA Nsight Tegra Visual Studio Edition, which then builds for Android. I've merged it to 'next' for testing: Merge topic 'vs-nsight-tegra-generator' into next

Re: [cmake-developers] vs-nsight-tegra-generator topic

2014-09-16 Thread Brad King
On 09/16/2014 10:05 AM, Brad King wrote: Please take a look if you are interested in Android development. [snip] On 09/15/2014 07:04 PM, Stephen Kelly wrote: Brad King wrote: On 09/10/2014 11:49 AM, Stephen Kelly wrote: target_link_libraries(mylib cmake::android-16) Interesting idea. These

Re: [cmake-developers] vs-nsight-tegra-generator topic

2014-09-16 Thread Stephen Kelly
Brad King wrote: On 09/16/2014 10:05 AM, Brad King wrote: Please take a look if you are interested in Android development. [snip] On 09/15/2014 07:04 PM, Stephen Kelly wrote: Brad King wrote: On 09/10/2014 11:49 AM, Stephen Kelly wrote: target_link_libraries(mylib cmake::android-16)

[cmake-developers] [CMake 0015159]: MemCheck CDash: timeout reported as failed instead of timeout

2014-09-16 Thread Mantis Bug Tracker
The following issue has been SUBMITTED. == http://www.cmake.org/Bug/view.php?id=15159 == Reported By:trsystran Assigned To:

[cmake-developers] Exporting a library shared and static

2014-09-16 Thread Gereon Kremer
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Hi all, I have a project that consists of a library and an application that are developed as two separated projects (different repos, separate cmake setups, etc.). Let's call them lib and app. The app exports itself to ~/.cmake/ and creates a

[cmake-developers] [CMake 0015160]: Different timeout for test and memcheck

2014-09-16 Thread Mantis Bug Tracker
The following issue has been SUBMITTED. == http://www.cmake.org/Bug/view.php?id=15160 == Reported By:trsystran Assigned To:

[cmake-developers] [PATCH 3/6] Resolve replace @rpath placeholders

2014-09-16 Thread Adam Strzelecki
This is done by gathering LC_RPATH commands for main bundle executable and using it for @rpath lookup in dependent frameworks. To achieve this all utility functions now take path to executable rather than path to its directory. This enabled apps using @rpath to be bundled correctly, which will

Re: [cmake-developers] OS X packaging updates

2014-09-16 Thread Adam Strzelecki
I had to revert again because it causes the Qt4Deploy to fail. The topic changes the signature of gp_file_type. User projects could be calling that, so we can't change it. In this case it is the Modules/DeployQt4.cmake file that was calling it. Thanks for spotting that. I've send updated

Re: [cmake-developers] [PATCH 3/6] Resolve replace @rpath placeholders

2014-09-16 Thread Clinton Stimpson
This patch has problems. You are calling BundleUtilities::get_item_key() from GetPrerequisites::gp_resolve_item(). There are users, including myself, which sometimes use GetPrerequisites.cmake but don't use BundleUtilities.cmake. So you cannot call BundleUtilities functions from

[cmake-developers] [PATCH 3/6] Resolve replace @rpath placeholders

2014-09-16 Thread Adam Strzelecki
This is done by gathering LC_RPATH commands for main bundle executable and using it for @rpath lookup in dependent frameworks. To achieve this all utility functions now take path to executable rather than path to its directory. This enabled apps using @rpath to be bundled correctly, which will

[cmake-developers] [PATCH 4/6] Process executables first when scanning bundle

2014-09-16 Thread Adam Strzelecki
This makes rpaths populated (if any), so libraries containing @rpath will be resolved properly. --- Modules/BundleUtilities.cmake | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/Modules/BundleUtilities.cmake b/Modules/BundleUtilities.cmake

Re: [cmake-developers] [PATCH 3/6] Resolve replace @rpath placeholders

2014-09-16 Thread Adam Strzelecki
Instead, can you extract rpaths for a binary in BundleUtilities and pass that into gp_resolve_item via the existing dirs argument? Okay, fixed this in the new 3/6 + 4/6 patches, attached to previous patch post. FYI I cannot use existing dirs arguments because other replacements search paths

Re: [cmake-developers] [PATCH 3/6] Resolve replace @rpath placeholders

2014-09-16 Thread Clinton Stimpson
On Tuesday, September 16, 2014 08:48:31 PM Adam Strzelecki wrote: Instead, can you extract rpaths for a binary in BundleUtilities and pass that into gp_resolve_item via the existing dirs argument? Okay, fixed this in the new 3/6 + 4/6 patches, attached to previous patch post. FYI I

Re: [cmake-developers] [PATCH 3/6] Resolve replace @rpath placeholders

2014-09-16 Thread Adam Strzelecki
No? These functions are called by other codes and we can't just change the meaning of the arguments. You are completely right. I am changing all stuff back, once I pass rpath as optional argument we can have exepath everywhere. Thanks for your feedback. Regards, -- Adam -- Powered by

Re: [cmake-developers] [PATCH 3/6] Resolve replace @rpath placeholders

2014-09-16 Thread Adam Strzelecki
I have sent [PATCH 3/5] Resolve replace @rpath placeholders which replaces previous 3/6 and obsoletes 4/6. Since it is getting messy like checking fixing maybe stage account and topic branch would be more accurate. --Adam -- Powered by www.kitware.com Please keep messages on-topic and

Re: [cmake-developers] [PATCH 3/6] Resolve replace @rpath placeholders

2014-09-16 Thread Clinton Stimpson
On Tuesday, September 16, 2014 10:53:14 PM Adam Strzelecki wrote: I have sent [PATCH 3/5] Resolve replace @rpath placeholders which replaces previous 3/6 and obsoletes 4/6. Since it is getting messy like checking fixing maybe stage account and topic branch would be more accurate. --Adam

Re: [cmake-developers] [PATCH 3/6] Resolve replace @rpath placeholders

2014-09-16 Thread Adam Strzelecki
Yes, it would be easier to review on stage or on github. Thanks. Here it is: https://github.com/nanoant/CMake/commits/fix-bundle-rpaths I would love to get stage access though ;) Cheers, -- Adam -- Powered by www.kitware.com Please keep messages on-topic and check the CMake FAQ at:

Re: [cmake-developers] vs-nsight-tegra-generator topic

2014-09-16 Thread Mourad Boufarguine
On Tue, Sep 16, 2014 at 4:05 PM, Brad King brad.k...@kitware.com wrote: Hi Folks, This topic introduces support for generating VS project files for the NVIDIA Nsight Tegra Visual Studio Edition, which then builds for Android. I've merged it to 'next' for testing: Hi Brad, I am really

Re: [cmake-developers] [PATCH 3/6] Resolve replace @rpath placeholders

2014-09-16 Thread Clinton Stimpson
On Tuesday, September 16, 2014 11:01:33 PM Adam Strzelecki wrote: Yes, it would be easier to review on stage or on github. Thanks. Here it is: https://github.com/nanoant/CMake/commits/fix-bundle-rpaths I would love to get stage access though ;) Cheers, What is the new optional

[cmake-developers] [CMake 0015161]: FindProtobuf: Generated source not regenerated on library update

2014-09-16 Thread Mantis Bug Tracker
The following issue has been SUBMITTED. == http://public.kitware.com/Bug/view.php?id=15161 == Reported By:hansmi Assigned To: