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

Don't index package main #163

Open
rwstauner opened this issue Feb 14, 2012 · 5 comments
Open

Don't index package main #163

rwstauner opened this issue Feb 14, 2012 · 5 comments

Comments

@rwstauner
Copy link
Contributor

https://metacpan.org/release/DAGOLDEN/Metabase-Fact-0.020/

main is listed in the docs section
because bin/metabase-profile (incorrectly) generates a =head1 NAME \n main -.

I've sent dagolden a bug report and pull req for that,
but i don't think there's ever a reason to index package main.

Even more fun is that if you click on the doc link for main you actually get taken to another dist which has the same problem.
I sent jquelin a pull req for his as well, but you can see by this search that the problem is bigger:
https://metacpan.org/search?q=main

@monken
Copy link
Contributor

monken commented Apr 11, 2012

sco has the same problem. I don't feel too strong about introducing all those workarounds for broken dists. It makes the behaviour of the indexer even more obscure than it is today.

@oalders
Copy link
Member

oalders commented Apr 11, 2012

I don't think we want a lot of workarounds, but making an exception for main might not be a bad idea. I can see how that might come up here and there.

@rwstauner
Copy link
Contributor Author

comparing
https://metacpan.org/release/DAGOLDEN/Metabase-Fact-0.020/
to
http://search.cpan.org/~dagolden/Metabase-Fact-0.021/

under "Documentation" metacpan lists main and sco lists metabase-profile.

So that part still seems "broken" to me...

As far as indexing main I don't disagree with you on the basis that there's probably no good reason for a person to be searching for main.

@monken
Copy link
Contributor

monken commented Apr 11, 2012

You need to compare the same version :)
https://metacpan.org/release/DAGOLDEN/Metabase-Fact-0.021/

Guess dagolden fixed that.

@rwstauner
Copy link
Contributor Author

oops, sorry :-)

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

No branches or pull requests

4 participants