<p dir="ltr">Hi Brad,</p>
<p dir="ltr">sorry for the empty mail, my phone acted up:-)</p>
<p dir="ltr">Am 07.06.2016 16:48 schrieb "Brad King" <<a href="mailto:brad.king@kitware.com">brad.king@kitware.com</a>>:<br>
><br>
> On 06/07/2016 10:42 AM, Tobias Hunger wrote:<br>
> > On Mo, 2016-06-06 at 13:24 -0400, Brad King wrote:<br>
> >> We should be able to refactor things to share the flags computation.<br>
> >> Methods like<br>
> >><br>
> >>   cmLocalGenerator::GetTargetFlags<br>
> ><br>
> > Does this patch here make sense:<br>
> ><br>
> > <a href="https://github.com/hunger/CMake/commit/124f8295bee0c228b79a5cf38f0b2581be308118">https://github.com/hunger/CMake/commit/124f8295bee0c228b79a5cf38f0b2581be308118</a><br>
> ><br>
> > It passes the configuration to GetTargetFlags instead of forcing it to the build<br>
> > type. Since I may end up with different configurations in the project structure,<br>
> > I think this is necessary.<br>
><br>
> Yes.  Previously that method was only called in single-configuration<br>
> generators so it could look up the CMAKE_BUILD_TYPE directly.  In<br>
> multi-config generators that variable is meaningless and instead they<br>
> iterate over CMAKE_CONFIGURATION_TYPES.<br>
><br>
> When you have a few such independent refactoring changes done<br>
> we can look at integrating them immediately to avoid holding<br>
> them externally in your daemon topic for too long.</p>
<p dir="ltr">Can you take that patch from github or do you want them sent here?</p>
<p dir="ltr">Best Regards,<br>
Tobias</p>