Google Groups unterstützt keine neuen Usenet-Beiträge oder ‑Abos mehr. Bisherige Inhalte sind weiterhin sichtbar.

[perl #38123] [TODO] build - change genfile() interpolation syntax

0 Aufrufe
Direkt zur ersten ungelesenen Nachricht

Joshua Hoblitt

ungelesen,
01.01.2006, 21:44:3501.01.06
an bugs-bi...@rt.perl.org
# New Ticket Created by Joshua Hoblitt
# Please include the string: [perl #38123]
# in the subject line of all future correspondence about this issue.
# <URL: https://rt.perl.org/rt3/Ticket/Display.html?id=38123 >


Parrot::Configure::Step::genfile() currently interpolates values in the
format ${foo}. This is undesirable because ${ BAREWORD } is valid perl5
syntax and in make files it looks uncomfortable similar to $(foo). It
should be changed to @foo@ where whitespace is not allowed between the
@s. This format is superior as a) it clearly stands out and b) it's
compatible with autoconf.

-J

--

Joshua Hoblitt via RT

ungelesen,
11.01.2006, 06:17:1111.01.06
an perl6-i...@perl.org
> [jhob...@ifa.hawaii.edu - Sun Jan 01 18:44:33 2006]:

>
> Parrot::Configure::Step::genfile() currently interpolates values in the
> format ${foo}. This is undesirable because ${ BAREWORD } is valid perl5
> syntax and in make files it looks uncomfortable similar to $(foo). It
> should be changed to @foo@ where whitespace is not allowed between the
> @s. This format is superior as a) it clearly stands out and b) it's
> compatible with autoconf.

I've added support for the autoconf compatable @foo@ syntax to genfile()
and experimentally converted root.in (the main makefile). If this
doesn't cause any smoke failures or bug reports in the next day or so
I'll convert the rest of the processed files.

Cheers,

-J

--

Joshua Hoblitt via RT

ungelesen,
11.01.2006, 23:26:4511.01.06
an perl6-i...@perl.org
The ${foo} interoplation/substitution syntax is no more. It has been
replaced with the autoconf like @foo@ syntax and all of the *.in files
have been updated. The special quoting syntax has also been removed
from parrot.pc.in

Support implimented in r11082 & r11117.

Cheers,

-J

--

0 neue Nachrichten