Dancer2 icon indicating copy to clipboard operation
Dancer2 copied to clipboard

Dancer2 pod hidden by dancer2 on case-insensitive filesystems (eg. OS X)

Open y opened this issue 9 years ago • 8 comments

On case-insensitive filesystems, like OS X's default- HFS+, perldoc Dancer2 results in the pod for dancer2 being displayed. Using Test::Portability::Files would help find other occurrences.

y avatar Mar 20 '15 06:03 y

I'm not sure how to fix it. The problem is that these two exist, no matter what.

There is the command line application and the top class name.

xsawyerx avatar Mar 22 '15 16:03 xsawyerx

Can you rename dancer2 to e.g. dancemaster, dancertwo, dancerctl, ...?

hvoers avatar Mar 22 '15 18:03 hvoers

Maybe dancer2ctl. I would prefer to get more input before doing this. Also, Dancer2 is just a conduit to the rest of the documentation and we can put that documentation in dancer2 as well.

xsawyerx avatar Mar 23 '15 09:03 xsawyerx

Solving this to Dancer2 doesn't solve it for a lot of other tools that might have the same problem. Shouldn't perldoc be patched in a way to include some -a option, like the unix man page?

Also, perldoc might be intelligent to find first modules if the name starts with uppercase.

I know, we do not control perldoc, but I think it is more important to try to "solve" or "diminish" the problem in a global way than to run from the problem.

This said, nothing against changing the command line tool name.

ambs avatar Mar 23 '15 14:03 ambs

@ambs is right that renaming is probably not enough, since that will work only for people that first manually uninstall the old version then install the new version. cpanm does not delete cruft from older versions of a distribution when they are updated, though it does have an option to uninstall a distribution.

Reference: https://github.com/miyagawa/cpanminus/issues/344

y avatar Mar 24 '15 16:03 y

As I see it, renaming the dancer2 script isn't a fix, it's a bodgy workaround for a broken system. Fixing the broken system would be far better, or if a workaround for such a broken system is required, it should be in perldoc so it'd work for everyone, not in Dancer2.

bigpresh avatar Mar 25 '15 09:03 bigpresh

Maybe for now, until this is fixed in a global level, we could provide add the links from Dancer2.pm to dancer2.pm. That's all Dancer2.pm really has that's interesting.

We could alternatively move everything into Dancer2::Manual and link both simply there.

xsawyerx avatar Mar 25 '15 10:03 xsawyerx

It's funny we didn't get such a ticket opened on Dancer 1 or even in Carton.

If no one suggests anything, I will simply make sure that dancer2 has explanation at the top giving the details that Dancer2 has pointing to Dancer2::Manual.

Dancer2.pm really does not have anything interesting or important. It currently only lists the import and one single short example. That can and should be removed.

xsawyerx avatar Apr 19 '15 16:04 xsawyerx