<div class="gmail_quote">On Thu, Feb 9, 2012 at 3:45 PM, Alexander Neundorf <span dir="ltr"><<a href="mailto:neundorf@kde.org">neundorf@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On Monday 30 January 2012, Stephen Kelly wrote:<br>
> David Cole wrote:<br>
> > On Sun, Jan 29, 2012 at 11:44 AM, Stephen Kelly<br>
> ><br>
> > <<a href="mailto:steveire@gmail.com">steveire@gmail.com</a>> wrote:<br>
> >> David Cole wrote:<br>
> >>> OK... nearly complete now. Please review, then reply and tell me if<br>
> >>> you object to any of the 7 commits in this topic branch.<br>
> >><br>
> >> No objections. They all seem fine.<br>
> ><br>
> > Great, thanks.<br>
> ><br>
> >>> Steve, I've<br>
> >>> preserved your authorship for most of these commits, but have<br>
> >>> significantly re-written some of them.<br>
> >><br>
> >> The 3 following 073bc421620e25fef6389b0d8b71cfad8ca79786 (CMake:<br>
> >> Eliminate cmMakefile::IncludeDirectories) seem to have been<br>
> >> substantially re-written. I'm not sure what is the appropriate way to<br>
> >> deal with commits like that, but whatever you choose for the author<br>
> >> line is fine.<br>
> ><br>
> > So I guess the question is: if somebody looks up the author at some<br>
> > point in the future and comes asking a question, do you want to be the<br>
> > one they go to, or would you rather have them come to me? ;-) If you<br>
> > want to send them to me, I'll change the author line, but you<br>
> > certainly deserve the credit for getting the ball rolling on this<br>
> > topic.<br>
><br>
> Ok, let's leave it as-is and I'll take responsibility (blame/credit) :)<br>
><br>
> <snip><br>
><br>
> >>> - double-check with Alex about the changes in cmake::FindPackage --<br>
> >>> Alex?<br>
><br>
> Still pending.<br>
<br>
</div></div>Sorry that it took so long, I was really busy the last few weeks.<br>
<br>
I cloned this repo: git clone <a href="https://github.com/dlrdave/CMake.git" target="_blank">https://github.com/dlrdave/CMake.git</a><br>
<br>
Both the Eclipse generator and cmake --find-package work fine.<br></blockquote><div><br></div><div><br></div><div>Excellent : thanks very much for trying it out. I'll try to get this first draft merged to 'next' tonight or tomorrow.</div>
<div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
Just to make completely sure: the purpose of this target property is *not*<br>
that targets linking against a library with this property set should reuse<br>
these include dirs, right ?<br>
<br>
Nevertheless, did you already think about how the interface for such a feature<br>
could look like (to make sure it fits with this new target property) ?<br>
Maybe something like this ?<br>
set_target_properties(... PROPERTIES<br>
                          INCLUDE_DIRECTORIES a/ b/ c/<br>
                          PUBLIC_BUILD_INCLUDE_DIRECTORIES d/ e/ f/<br>
                          PUBLIC_INSTALL_INCLUDE_DIRECTORIES g/ h/ i/ )<br>
<br>
It would have to be possible to separate between include dirs to be used at<br>
build time of the library, and include dirs to be used once the library has<br>
been installed (similar to the build and install RPATH)<br>
<br>
...just to make sure the syntax for the new INCLUDE_DIRECTORIES property is ok<br>
and doesn't make problems later on.<br>
<br>
Alex<br>
<div class="HOEnZb"><div class="h5">--<br>
<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Please keep messages on-topic and check the CMake FAQ at: <a href="http://www.cmake.org/Wiki/CMake_FAQ" target="_blank">http://www.cmake.org/Wiki/CMake_FAQ</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers" target="_blank">http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers</a><br>
</div></div></blockquote></div><br>