[Xorp-hackers] Bugzilla #54

Pavlin Radoslavov pavlin@icir.org
Sun, 04 Sep 2005 11:21:24 -0700


> I've written a small patch to generate a nicer
> configuration file header.
> 
> /* XORP configuration file
>  * 
>  * XORP version: 1.1
>  * XORP file format: 0.1
>  * date: 2005-09-34 11:34:22
>  * user: xorp-user
>  * host: xorp-lab1
>  */
> 
> I've setup a new variable in configure.in called
> XORP_CONFIG_FILE_FORMAT_VERSION.. unfortunately it
> seems I am not able to properly bootstrap on my
> system so I cannot verify that this variable is
> set properly.
> In my tests I've just #defined it at the top of
> master_conf_tree.cc
> I'm not sure at all whether such a variable would
> be the best way to mark file format version but 
> it's there for now.
> 
> The patch is at:
> http://skalman.juniks.net/~ply/xorp/config_header.patch
> at some positions it includes changes that were
> made in my patch for #53...

Yes, in the past we have been thinking about adding more info to the
header of the XORP config files, but so far we didn't get the time
to do it ourselves.

Again, please upload the above patch to bugzilla entry #53.

Thanks,
Pavlin