<div class="gmail_quote">On Wed, Aug 11, 2010 at 5:32 PM, Eric Noulard <span dir="ltr"><<a href="mailto:eric.noulard@gmail.com">eric.noulard@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">2010/8/11 Brad King <<a href="mailto:brad.king@kitware.com">brad.king@kitware.com</a>>:<br>
> On 08/11/2010 02:43 PM, Eric Noulard wrote:<br>
>> 2010/8/11 Brad King <<a href="mailto:brad.king@kitware.com">brad.king@kitware.com</a>>:<br>
>>> I rebased/squashed the commits into a single change and merged<br>
>>> the topic to next:<br>
>>><br>
>>>  <a href="http://cmake.org/gitweb?p=cmake.git;a=log;h=299f3246" target="_blank">http://cmake.org/gitweb?p=cmake.git;a=log;h=299f3246</a><br>
>><br>
>> Just nice, thanks.<br>
><br>
> FYI, I had to merge in a couple of warning fixes:<br>
><br>
>  <a href="http://cmake.org/gitweb?p=cmake.git;a=log;h=632c2659" target="_blank">http://cmake.org/gitweb?p=cmake.git;a=log;h=632c2659</a><br>
>  <a href="http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=31a313d4" target="_blank">http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=31a313d4</a><br>
<br>
</div>Noted, thank you for fixing my mistakes :-)<br>
<br>
However,<br>
<br>
I usually take care of warnings (some of them may be harmful)<br>
so I'm pretty I didn't have those.<br>
<br>
Did you get those warnings with MSVC or did any version  of gcc<br>
provide the same warnings ?<br></blockquote><div><br></div><div>The warnings Brad fixed showed up on the Continuous dashboard for CMake here:</div><div><a href="http://www.cdash.org/CDash/index.php?project=CMake&date=2010-08-11#Continuous">http://www.cdash.org/CDash/index.php?project=CMake&date=2010-08-11#Continuous</a></div>
<div><br></div><div>The flags used on that machine (older gcc running on Mac OSX 10.3 believe it or not) are:</div><span class="Apple-style-span" style="font-family: Arial, Helvetica, sans-serif; font-size: 14px; "><pre>CMAKE_CXX_FLAGS:STRING=-g -O0 -Wall -W -Wshadow -Wunused-variable -Wunused-parameter -Wunused-function -Wunused -Wno-system-headers -Wno-deprecated -Woverloaded-virtual
CMAKE_C_FLAGS:STRING=-g -O0 -Wall -W
</pre></span><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
Another gitish question:<br>
<br>
When you want to develop something that is only in next (and not in master)<br>
is it acceptable to branch from next and then merge to next<br>
or is there a better git workflow ?<br>
<div class="im"><br></div></blockquote><div><br></div><div>We branch from the commit that was merged into next, not next itself, to continue work that was begun based on master but is not merged back into master yet.</div>
<div><br></div><div>So... find the commit that Brad made that fixed the warnings, which I think is this one:</div><div><a href="http://cmake.org/gitweb?p=cmake.git;a=commit;h=31a313d47043923bc722554175912e75a03a13f6">http://cmake.org/gitweb?p=cmake.git;a=commit;h=31a313d47043923bc722554175912e75a03a13f6</a></div>
<div><br></div><div>And then begin a new branch from there:</div><div>git checkout -b CPack-APIredesign 31a313d4</div><div><br></div><div>If you already have a local branch with that name, I'm sure there's a way to just pull those commits into that pre-existing branch, too. But you'll have to wait for Brad to reply for that one.</div>
<div><br></div><div><br></div><div>Cheers,</div><div>David</div><div><br></div></div>