Re: [CMake] Re: FindBoost.cmake: please test

2008-01-21 Thread Filipe Sousa
Brad King wrote: > Andreas Pakulat wrote: > > I've just implemented the request in 1645 in CMake HEAD. If you call > > find_package(Boost 1.34.1) I guess this would try to find, at least, boost version 1.34.1 but there is no way to tell that I want the exact version. Some support for operator

Re: [CMake] Re: FindBoost.cmake: please test

2008-01-21 Thread Andreas Pakulat
On 21.01.08 08:53:58, Brad King wrote: > Andreas Pakulat wrote: > > On 20.01.08 17:11:17, Rodolfo Lima wrote: > >> Andreas Pakulat escreveu: > >> > >>> Naah, _boost_XXX is internal and not exposed. I rather have a > >>> Boost_MINIMUM_VERSION. That would also allow to use REQUIRED and a more > >>> v

[CMake] Re: FindBoost.cmake: please test

2008-01-21 Thread Rodolfo Schulz de Lima
Brad King escreveu: I've just implemented the request in 1645 in CMake HEAD. If you call Thank you very much Brad, I've just seen it while updating from cvs. I'll give it a try with my own modules here. Regards, rod ___ CMake mailing list CMake@

Re: [CMake] Re: FindBoost.cmake: please test

2008-01-21 Thread Brad King
Andreas Pakulat wrote: > On 20.01.08 17:11:17, Rodolfo Lima wrote: >> Andreas Pakulat escreveu: >> >>> Naah, _boost_XXX is internal and not exposed. I rather have a >>> Boost_MINIMUM_VERSION. That would also allow to use REQUIRED and a more >>> verbose message saying wether the version is wrong or

Re: [CMake] Re: FindBoost.cmake: please test

2008-01-20 Thread Andreas Pakulat
On 20.01.08 17:11:17, Rodolfo Lima wrote: > Andreas Pakulat escreveu: > > > Naah, _boost_XXX is internal and not exposed. I rather have a > > Boost_MINIMUM_VERSION. That would also allow to use REQUIRED and a more > > verbose message saying wether the version is wrong or a needed lib is > > not fo

Re: [CMake] Re: FindBoost.cmake: please test

2008-01-20 Thread Andreas Pakulat
On 20.01.08 19:45:03, Hendrik Sattler wrote: > Am Sonntag 20 Januar 2008 schrieb Andreas Pakulat: > > On 20.01.08 15:38:44, Rodolfo Lima wrote: > > > Andreas Pakulat escreveu: > > > > The reason I did that was to simplify the module code. > > > > > > I see, but this variable is only used in a if cl

[CMake] Re: FindBoost.cmake: please test

2008-01-20 Thread Rodolfo Lima
Andreas Pakulat escreveu: > Naah, _boost_XXX is internal and not exposed. I rather have a > Boost_MINIMUM_VERSION. That would also allow to use REQUIRED and a more > verbose message saying wether the version is wrong or a needed lib is > not found. Speaking of which, find_package could have a way

Re: [CMake] Re: FindBoost.cmake: please test

2008-01-20 Thread Hendrik Sattler
Am Sonntag 20 Januar 2008 schrieb Andreas Pakulat: > On 20.01.08 15:38:44, Rodolfo Lima wrote: > > Andreas Pakulat escreveu: > > > The reason I did that was to simplify the module code. > > > > I see, but this variable is only used in a if clause, so instead of > > writing > > > > IF( Boost_USE_NON

Re: [CMake] Re: FindBoost.cmake: please test

2008-01-20 Thread Andreas Pakulat
On 20.01.08 19:15:33, Andreas Schneider wrote: > Rodolfo Lima wrote: > > Again, it would be good if we could choose which version to detect. If I > > have 1.33.0 and 1.34.1, I want my application to use 1.34.1 and beyond. > > You can simply overwrite the variable _boost_TEST_VERSIONS. Naah, _boos

Re: [CMake] Re: FindBoost.cmake: please test

2008-01-20 Thread Andreas Pakulat
On 20.01.08 15:38:44, Rodolfo Lima wrote: > Andreas Pakulat escreveu: > > The reason I did that was to simplify the module code. > > I see, but this variable is only used in a if clause, so instead of writing > > IF( Boost_USE_NONMULTITHREADED ) > SET (_boost_MULTITHREADED "") > ENDIF( Boost

Re: [CMake] Re: FindBoost.cmake: please test

2008-01-20 Thread Andreas Schneider
Rodolfo Lima wrote: > Again, it would be good if we could choose which version to detect. If I > have 1.33.0 and 1.34.1, I want my application to use 1.34.1 and beyond. You can simply overwrite the variable _boost_TEST_VERSIONS. -- andreas -- http://www.cynapses.org/ - cybernetic synaps

[CMake] Re: FindBoost.cmake: please test

2008-01-20 Thread Rodolfo Lima
Andreas Pakulat escreveu: > Luckily we don't need that complex cmake code as the Boost_VERSION is a > plain number and can be compared to another plain number in a much > easier way :) That's true :) You only have to convert the version string to this number and compare, so it can easily be done.

[CMake] Re: FindBoost.cmake: please test

2008-01-20 Thread Rodolfo Lima
Andreas Pakulat escreveu: > The reason I did that was to simplify the module code. I see, but this variable is only used in a if clause, so instead of writing IF( Boost_USE_NONMULTITHREADED ) SET (_boost_MULTITHREADED "") ENDIF( Boost_USE_NONMULTITHREADED ) you could write IF(NOT Boost_USE

Re: [CMake] Re: FindBoost.cmake: please test

2008-01-20 Thread Andreas Pakulat
On 20.01.08 14:10:53, Rodolfo Lima wrote: > Again, it would be good if we could choose which version to detect. If I > have 1.33.0 and 1.34.1, I want my application to use 1.34.1 and beyond. > There's this script written by James Bigler that would help, it's > attached to this message. You could do

Re: [CMake] Re: FindBoost.cmake: please test

2008-01-20 Thread Andreas Pakulat
On 20.01.08 14:05:17, Rodolfo Lima wrote: > Andreas Pakulat escreveu: > > > If you encounter any errors please reply to this mail with the cmake > > error message and if you want to invest the time a patch to fix the > > problem :) > > Works for me in a gentoo system. I'd only change the semantic

[CMake] Re: FindBoost.cmake: please test

2008-01-20 Thread Rodolfo Lima
Again, it would be good if we could choose which version to detect. If I have 1.33.0 and 1.34.1, I want my application to use 1.34.1 and beyond. There's this script written by James Bigler that would help, it's attached to this message. You could do: if(Boost_MIN_VERSION) include(CompareVe

[CMake] Re: FindBoost.cmake: please test

2008-01-20 Thread Rodolfo Lima
Andreas Pakulat escreveu: > If you encounter any errors please reply to this mail with the cmake > error message and if you want to invest the time a patch to fix the > problem :) Works for me in a gentoo system. I'd only change the semantics of Boost_USE_NONMULTITHREADED. I prefer setting Boost_