Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rakudo doesn't warn about Perl 5 $] #3207

Closed
p6rt opened this issue Aug 14, 2013 · 8 comments
Closed

Rakudo doesn't warn about Perl 5 $] #3207

p6rt opened this issue Aug 14, 2013 · 8 comments

Comments

@p6rt
Copy link

p6rt commented Aug 14, 2013

Migrated from rt.perl.org#119293 (status was 'resolved')

Searchable as RT119293$

@p6rt
Copy link
Author

p6rt commented Aug 14, 2013

From @ShimmerFairy

<lue> r​: say $]
<camelia> rakudo c0814a​: OUTPUT«===SORRY!=== Error while compiling
/tmp/bQ267__iqC␤Non-declarative sigil is missing its name␤at
/tmp/bQ267__iqC​:1␤------> say ⏏$]␤ expecting any of​:␤ argument list␤
prefix or term␤ prefix or meta-prefix␤»…

The error message should be closer to what STD says​:

<lue> std​: say $]
<camelia> std c2215f0​: OUTPUT«===SORRY!===␤Unsupported use of $]
variable; in Perl 6 please use $*PERL_VERSION at /tmp/30I9vo1peV line
1​:␤------> say $]⏏<EOL>␤Parse failed␤FAILED 00​:00 42m␤»

@p6rt
Copy link
Author

p6rt commented Feb 24, 2014

From @coke

On Wed Aug 14 14​:39​:07 2013, lue wrote​:

<lue> r​: say $]
<camelia> rakudo c0814a​: OUTPUT«===SORRY!=== Error while compiling
/tmp/bQ267__iqC␤Non-declarative sigil is missing its name␤at
/tmp/bQ267__iqC​:1␤------> say ⏏$]␤ expecting any of​:␤ argument list␤
prefix or term␤ prefix or meta-prefix␤»…

The error message should be closer to what STD says​:

<lue> std​: say $]
<camelia> std c2215f0​: OUTPUT«===SORRY!===␤Unsupported use of $]
variable; in Perl 6 please use $*PERL_VERSION at /tmp/30I9vo1peV line
1​:␤------> say $]⏏<EOL>␤Parse failed␤FAILED 00​:00 42m␤»

Fixed. Closable with tests​:

12​:09 < [Coke]> m​: say $]
12​:09 <+camelia> rakudo-moar 299d70​: OUTPUT«===SORRY!=== Error while compiling
  /tmp/_cHikCl3Xj␤Unsupported use of $] variable; in Perl 6
  please use $*PERL_VERSION␤at /tmp/_cHikCl3Xj​:1␤------> say
  ⏏$]␤ expecting any of​:␤ argument list␤ …»
--
Will "Coke" Coleda

@p6rt
Copy link
Author

p6rt commented Feb 24, 2014

The RT System itself - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented Oct 6, 2014

From @usev6

Unfortunately the error message is only "Unexpected closing bracket" now​:

13​:00 < bartolin> r​: say $]
13​:00 <+camelia> rakudo-{parrot,moar} 65819d​: OUTPUT«===SORRY!=== Error while compiling /tmp/tmpfile␤Unexpected closing bracket␤at /tmp/tmpfile​:1␤------> say $⏏]

1 similar comment
@p6rt
Copy link
Author

p6rt commented Oct 6, 2014

From @usev6

Unfortunately the error message is only "Unexpected closing bracket" now​:

13​:00 < bartolin> r​: say $]
13​:00 <+camelia> rakudo-{parrot,moar} 65819d​: OUTPUT«===SORRY!=== Error while compiling /tmp/tmpfile␤Unexpected closing bracket␤at /tmp/tmpfile​:1␤------> say $⏏]

@p6rt
Copy link
Author

p6rt commented Feb 17, 2015

From @coke

On Mon Oct 06 04​:01​:24 2014, bartolin@​gmx.de wrote​:

Unfortunately the error message is only "Unexpected closing bracket"
now​:

13​:00 < bartolin> r​: say $]
13​:00 <+camelia> rakudo-{parrot,moar} 65819d​: OUTPUT«===SORRY!===
Error while compiling /tmp/tmpfile␤Unexpected closing bracket␤at
/tmp/tmpfile​:1␤------> say $⏏]

Re-fixed; test added to S32-exceptions/misc.t with the other perl5var tests.

There was still code around to deal with this in the exception class, but the check was lost from the grammar.
--
Will "Coke" Coleda

@p6rt
Copy link
Author

p6rt commented Feb 17, 2015

From @coke

On Mon Oct 06 04​:01​:24 2014, bartolin@​gmx.de wrote​:

Unfortunately the error message is only "Unexpected closing bracket"
now​:

13​:00 < bartolin> r​: say $]
13​:00 <+camelia> rakudo-{parrot,moar} 65819d​: OUTPUT«===SORRY!===
Error while compiling /tmp/tmpfile␤Unexpected closing bracket␤at
/tmp/tmpfile​:1␤------> say $⏏]

Re-fixed; test added to S32-exceptions/misc.t with the other perl5var tests.

There was still code around to deal with this in the exception class, but the check was lost from the grammar.

--
Will "Coke" Coleda

@p6rt p6rt closed this as completed Feb 17, 2015
@p6rt
Copy link
Author

p6rt commented Feb 17, 2015

@coke - Status changed from 'open' to 'resolved'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant