-
Notifications
You must be signed in to change notification settings - Fork 293
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
Disambiguate PERL6LIB in search with categories #3200
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check for pre-existence of categories to avoid creating new ones.
@@ -1509,7 +1509,7 @@ Perl modules is C<PERL5LIB>. | |||
$ PERL5LIB="/some/module/lib" perl program.pl | |||
|
|||
In Raku this is similar, one merely needs to change a number! As you | |||
probably guessed, you just need to use X<C<PERL6LIB>|PERL6LIB>: | |||
probably guessed, you just need to use X<C<PERL6LIB>|p5 nutshell>: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Um, this looks weird. This is going to be indexed as what's to the right of the |; it might be better to use PERL6LIB,p5 nutshell
. That creates a new category, so use whatever category is already been used there.
@@ -16,7 +16,7 @@ page|https://github.com/rakudo/rakudo/wiki/dev-env-vars#moarvm>. | |||
|
|||
=head2 Module loading | |||
|
|||
X<|RAKUDOLIB>X<|PERL6LIB> | |||
X<|RAKUDOLIB, environment variable>X<|PERL6LIB, environment variable> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ditto here. Use only if there's already a category under that name.
Updated files pass 'make xtest' |
Closing as it's useless without search categories established. |
A fix for #3199
I didn't check it is fine locally, please do before merge.