Re: [cmake-developers] Which binaries should be required in FindLATEX?

2015-01-12 Thread Brad King
On 1/10/2015 4:03 PM, Christoph Grüninger wrote: The link does not work and I couldn't find the commit in next. Can you double check whether you pushed it? Oops, thanks. I pushed it to our staging repo but never merged to 'next'. Done now, along with the below. Additionally I attached

Re: [cmake-developers] Which binaries should be required in FindLATEX?

2015-01-10 Thread Christoph Grüninger
Hi Brad, concerning FindLATEX again: Applied: FindLATEX: Add components handling http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=07a3f9ad The link does not work and I couldn't find the commit in next. Can you double check whether you pushed it? with minor documentation tweaks and one

Re: [cmake-developers] Which binaries should be required in FindLATEX?

2014-12-23 Thread Brad King
On 12/22/2014 6:29 PM, Christoph Grüninger wrote: please find attached a new patch following your latest suggestions. Thanks. I'd like to have lower case components but I dislike mixed-cased variable names. The latter feeling is stronger. Mixed-case variable names are quite common in CMake

Re: [cmake-developers] Which binaries should be required in FindLATEX?

2014-12-23 Thread Christoph Grüninger
Hi Brad, next try and as always better than the last attempt. Bye Christoph From ba775bd58b70ad2581fc05f30d8c5e42e1c94096 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Christoph=20Gr=C3=BCninger?= gruen...@dune-project.org Date: Tue, 23 Dec 2014 11:33:32 +0100 Subject: [PATCH] FindLATEX: Add

Re: [cmake-developers] Which binaries should be required in FindLATEX?

2014-12-23 Thread Brad King
On 12/23/2014 05:35 AM, Christoph Grüninger wrote: next try and as always better than the last attempt. Applied: FindLATEX: Add components handling http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=07a3f9ad with minor documentation tweaks and one logic fix: -# set(LATEX_MAKEINDEX_FOUND

Re: [cmake-developers] Which binaries should be required in FindLATEX?

2014-12-22 Thread Brad King
On 12/22/2014 02:44 AM, Christoph Grüninger wrote: please find attached a patch that introduces components handling for optional LaTeX executable. Thanks for working on this. Maybe we should deprecate the old variables name? The names should not be changed at all. They become cache entries

Re: [cmake-developers] Which binaries should be required in FindLATEX?

2014-12-22 Thread Christoph Grüninger
Hi Brad, please find attached a new patch following your latest suggestions. Once you have that explicit mapping then you can optionally use lower-case component names. I have no strong preference on that though. I'd like to have lower case components but I dislike mixed-cased variable

Re: [cmake-developers] Which binaries should be required in FindLATEX?

2014-12-21 Thread Christoph Grüninger
Hi Brad, please find attached a patch that introduces components handling for optional LaTeX executable. FPHSA's components handling feature is pretty cool. I first wrote the components handling myself, but the built-in version is more elegant. I am the first one to use this feature in an official

Re: [cmake-developers] Which binaries should be required in FindLATEX?

2014-12-05 Thread Brad King
Hi Christoph, Thanks for working on this. On 12/05/2014 02:35 AM, Christoph Grüninger wrote: For users it would be great to simply have all these binaries. I don't have all the converters installed but can still build PDF books. The varying tools do not all come in the same package on Linux