[Windows] Environment variables in Bakefiles.bkgen do not work anymore

[Francesco Montorsi]

On Windows I used to put in Bakefiles.bkgen lines like:

    <add-flags>-I %WXWIN%/build/bakefiles/wxpresets</add-flags>

However it looks that with the SVN trunk version this doesn't work anymore:

[1/9] generating msvs2008prj from usbpicprog.bkl
E:\upp_wx_thread\upp_wx\build\bakefile\usbpicprog.bkl (line unknown, install libxml2 to show it): error: can't find file 'presets/wx.bkl' in E:\upp_wx_thread\upp_wx\build\bakefile:%WXWIN%\build\bakefiles\wxpresets:E:\Bakefile\rules:E:\Bakefile\rules\modules:E:\Bakefile\output:E:\Bakefile

so it looks like %WXWIN% is not expanded... I remember it was in previous versions however.

[Vaclav Slavik]

Replying to frm:

I remember it was in previous versions however.

It never was -- not intentionally.

[Francesco Montorsi]

It would be nice to have this officially supported, then. Would you accept a patch adding this functionality?

[anonymous]

Replying to frm:

It would be nice to have this officially supported, then. Would you accept a patch adding this functionality?

If it was added in a way consistent with the way variables are used in Bakefile itself (i.e. definitely not some platform-specific syntax, but the syntax used in Bakefile, i.e. envvar()) and documented, then why not.

[Francesco Montorsi]

I've created and tested the patch.

Note that I couldn't reuse expression evaluation functions from mk.py because otherwise the envvar() function defined in utils.py would be called, instead of the envvar() function defined in bakefile_gen.py...

[Francesco Montorsi]

Adds envvar() function evaluation to bakefile_gen

[Francesco Montorsi]

The patch does not include changes in the docs... if the patch is ok I can make an additional patch for docs, too.