[gtkada] Patch: Repair the support for directory variables.

[Björn Persson]

Emmanuel Briot <briot at adacore.com> wrote:
> >>    Exampledir := external("EXAMPLEDIR",
> >> "share/examples/gtkada/testgtk"); for artifacts (Exampledir) use
> >> (....)
> >> 
> > 
> > Of course i tried the obvious. It made the project file syntactically
> > invalid. GPRbuild requires a string literal, and so does the BNF in the
> > GPRbuild User's Guide. Well it was GPRbuild 2015 that I tried it with,
> > but the GPRbuild and GPR Companion Tools User’s Guide 2016 still
> > requires a literal.
> 
> Which part requires a literal string in my example ? (I agree that there are
> a number of cases in which gpr files must have literal strings, and sometimes
> even for good reasons :-)
> But in my example, I am using a literal string for the default value of "external",
> 
> Would you share (if you still have it) the experiment you performed ?

I have now found the time to write a reproducer, and also checked that
it fails with GPRbuild 2016 too. In the attachment, the difference
between literal.gpr and variable.gpr is the change that you suggested.
Please run it and see what happens.

Björn Persson
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gpr_artifacts_test.tar.gz
Type: application/gzip
Size: 594 bytes
Desc: not available
URL: </pipermail/gtkada/attachments/20160919/562876e9/attachment.gz>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signatur
URL: </pipermail/gtkada/attachments/20160919/562876e9/attachment.sig>