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

=SEE-ALSO pod comment can no longer be parsed (=SEE-ALSO foo) #6626

Closed
p6rt opened this issue Oct 21, 2017 · 6 comments
Closed

=SEE-ALSO pod comment can no longer be parsed (=SEE-ALSO foo) #6626

p6rt opened this issue Oct 21, 2017 · 6 comments
Labels
regression Issue did not exist previously

Comments

@p6rt
Copy link

p6rt commented Oct 21, 2017

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

Searchable as RT132339$

@p6rt
Copy link
Author

p6rt commented Oct 21, 2017

From @AlexDaniel

Code​:
=SEE-ALSO foo

¦«2017.09,963a0f0657^»​:

¦«963a0f0,HEAD(312cac7)»​:
===SORRY!=== Error while compiling /tmp/8cNEnQJfmq
Preceding context expects a term, but found infix = instead
at /tmp/8cNEnQJfmq​:1
------> =SEE⏏-ALSO foo «exit code = 1»

Bisected to rakudo/rakudo@963a0f0

So it is probably related to || fixes in :ratchet mode.

Maybe worth mentioning that lowercase “see-also” seems to be ok.

This was toasted from a real module​: https://github.com/skids/perl6xproto/blob/6750e05da9b70b180eeaebaeb9e15d35d5856cd3/lib/X/Protocol.pm6#L185

@p6rt
Copy link
Author

p6rt commented Oct 21, 2017

From @zoffixznet

On Sat, 21 Oct 2017 09​:06​:24 -0700, alex.jakimenko@​gmail.com wrote​:

Code​:
=SEE-ALSO foo

¦«2017.09,963a0f0657^»​:

¦«963a0f0,HEAD(312cac7)»​:
===SORRY!=== Error while compiling /tmp/8cNEnQJfmq
Preceding context expects a term, but found infix = instead
at /tmp/8cNEnQJfmq​:1
------> =SEE⏏-ALSO foo «exit code = 1»

Bisected to
rakudo/rakudo@963a0f0

So it is probably related to || fixes in :ratchet mode.

Maybe worth mentioning that lowercase “see-also” seems to be ok.

This was toasted from a real module​:
https://github.com/skids/perl6xproto/blob/6750e05da9b70b180eeaebaeb9e15d35d5856cd3/lib/X/Protocol.pm6#L185

Thank you for the report. This is now fixed.

Fix​: rakudo/rakudo@ba718f4581a95e1
Test​: Raku/roast@2ede0d1fc3f2e0536

@p6rt
Copy link
Author

p6rt commented Oct 21, 2017

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

@p6rt
Copy link
Author

p6rt commented Oct 21, 2017

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

@p6rt p6rt closed this as completed Oct 21, 2017
@p6rt
Copy link
Author

p6rt commented Oct 21, 2017

From @AlexDaniel

Just for the record, there was another module affected by this​: https://github.com/nkh/P6-Text-Template/blob/5e1752c08ddb064e366be862fd500c4de0c4b833/lib/Text/Template.pm#L142

On 2017-10-21 10​:44​:51, cpan@​zoffix.com wrote​:

On Sat, 21 Oct 2017 09​:06​:24 -0700, alex.jakimenko@​gmail.com wrote​:

Code​:
=SEE-ALSO foo

¦«2017.09,963a0f0657^»​:

¦«963a0f0,HEAD(312cac7)»​:
===SORRY!=== Error while compiling /tmp/8cNEnQJfmq
Preceding context expects a term, but found infix = instead
at /tmp/8cNEnQJfmq​:1
------> =SEE⏏-ALSO foo «exit code = 1»

Bisected to
rakudo/rakudo@963a0f0

So it is probably related to || fixes in :ratchet mode.

Maybe worth mentioning that lowercase “see-also” seems to be ok.

This was toasted from a real module​:
https://github.com/skids/perl6xproto/blob/6750e05da9b70b180eeaebaeb9e15d35d5856cd3/lib/X/Protocol.pm6#L185

Thank you for the report. This is now fixed.

Fix​: rakudo/rakudo@ba718f4581a95e1
Test​: Raku/roast@2ede0d1fc3f2e0536

@p6rt
Copy link
Author

p6rt commented Oct 21, 2017

From @AlexDaniel

And also​:
https://github.com/tadzik/perl6-Acme-Meow/blob/35286b14447d22516fbf68767c4d7e18b7396971/lib/Acme/Meow.pm#L65-L69
https://github.com/skids/perl6-Control-Bail/blob/3e6ab6c3bab9176a44476bd0883ba99205306400/lib/Control/Bail.pm6#L81
https://github.com/tadzik/Grammar-BNF/blob/14f51ca5882f00f0acee643c9845ab37559b5119/lib/Grammar/ABNF.pm#L435

On 2017-10-21 11​:07​:33, alex.jakimenko@​gmail.com wrote​:

Just for the record, there was another module affected by this​:
https://github.com/nkh/P6-Text-
Template/blob/5e1752c08ddb064e366be862fd500c4de0c4b833/lib/Text/Template.pm#L142

On 2017-10-21 10​:44​:51, cpan@​zoffix.com wrote​:

On Sat, 21 Oct 2017 09​:06​:24 -0700, alex.jakimenko@​gmail.com wrote​:

Code​:
=SEE-ALSO foo

¦«2017.09,963a0f0657^»​:

¦«963a0f0,HEAD(312cac7)»​:
===SORRY!=== Error while compiling /tmp/8cNEnQJfmq
Preceding context expects a term, but found infix = instead
at /tmp/8cNEnQJfmq​:1
------> =SEE⏏-ALSO foo «exit code = 1»

Bisected to

rakudo/rakudo@963a0f0

So it is probably related to || fixes in :ratchet mode.

Maybe worth mentioning that lowercase “see-also” seems to be ok.

This was toasted from a real module​:

https://github.com/skids/perl6xproto/blob/6750e05da9b70b180eeaebaeb9e15d35d5856cd3/lib/X/Protocol.pm6#L185

Thank you for the report. This is now fixed.

Fix​: rakudo/rakudo@ba718f4581a95e1
Test​: Raku/roast@2ede0d1fc3f2e0536

@p6rt p6rt added the regression Issue did not exist previously label Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
regression Issue did not exist previously
Projects
None yet
Development

No branches or pull requests

1 participant