[Dune-devel] stamp-regenerate-config-h ?

[Oliver Sander]

Am 24.11.2014 um 12:06 schrieb Markus Blatt:
> On Fri, Nov 21, 2014 at 04:16:42PM +0100, Oliver Sander wrote:
>> Ping?
>>
>> Am 17.11.2014 um 07:24 schrieb Oliver Sander:
>>> Thanks,
>>> but why do I need such a button?  Why can't I simply remove config.h and
>>> then call cmake to have it regenerated?  Markus?
> 
> Because it is a two step procedure:
> - build-dir/config.h.cmake is generated from all config.h.cmake in the
>   source directory of this module and all modules that we depend on.
> - Then config.h is generated from that one.
> 
> The button is there for developers to test new changes in upstream
> modules. A better alternative would be to explicitly state the
> dependencies between the config.h.cmake files.

So if this is a pro tool, why is everybody getting such file from duneproject?
Wouldn't it be less confusing to not have the file by default?  Dune-savvy
people would create it with a simple 'touch' whenever they need it.
Cheers,
Oliver


> 
> Markus
> 
>>>
>>>
>>> Am 14.11.2014 um 22:59 schrieb Dominic Kempf:
>>>> Hey,
>>>>
>>>> I had a talk with Steffen and did a bit of research.
>>>>
>>>> The idea behind the file is to have a "button" to force regeneration of
>>>> config.h. In DuneMacros.cmake in line 837 a dependency of config.h.cmake on
>>>> this file is declared. By touching it, you for cmake to regenerate it.
>>>> Whether or not this is a usual or necessary design, I cannot judge.
>>>>
>>>> Best,
>>>> Dominic
>>>>
>>>> On Fri, Nov 14, 2014 at 9:52 PM, Christoph Grüninger <
>>>> christoph.grueninger at iws.uni-stuttgart.de> wrote:
>>>>
>>>>> Hi Oliver,
>>>>> I had a quick look and it does not seem to be used. The creation of
>>>>> file was added in 3c418763c3525ad33ce3c7557055964232c265c8 by
>>>>> Markus but still, there seems to be no other use of it. I'd drop the
>>>>> file, but let's wait what Markus says.
>>>>>
>>>>> Bye
>>>>> Christoph
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> Dune-devel mailing list
>>>>> Dune-devel at dune-project.org
>>>>> http://lists.dune-project.org/mailman/listinfo/dune-devel
>>>>>
>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> Dune-devel mailing list
>>>> Dune-devel at dune-project.org
>>>> http://lists.dune-project.org/mailman/listinfo/dune-devel
>>>>
>>>
>>>
>>>
>>>
>>> _______________________________________________
>>> Dune-devel mailing list
>>> Dune-devel at dune-project.org
>>> http://lists.dune-project.org/mailman/listinfo/dune-devel
>>>
>>
>>
> 
> 
> 
>> _______________________________________________
>> Dune-devel mailing list
>> Dune-devel at dune-project.org
>> http://lists.dune-project.org/mailman/listinfo/dune-devel
> 
> 
> 
> 
> _______________________________________________
> Dune-devel mailing list
> Dune-devel at dune-project.org
> http://lists.dune-project.org/mailman/listinfo/dune-devel
> 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: OpenPGP digital signature
URL: <https://lists.dune-project.org/pipermail/dune-devel/attachments/20141125/8f04d92c/attachment.sig>