Bug #45544 for Module-Build: Can't create manpages on vfat partition on unix

Wed Apr 29 01:30:26 2009 jcyl_baihu [...] 163.com - Ticket created

Subject:

A compilation problem may caused by Base.pm?

Hi! I was trying to install a new module Math::MatrixReal on Ubuntu 8.10, but a problem was reported: ------------------------------------------------ root@ubuntu:/***/Math-MatrixReal-1.3# ./Build Manifying blib/lib/Math/MatrixReal.pm -> blib/libdoc/Math::MatrixReal.3pm Can't write-open blib/libdoc/Math::MatrixReal.3pm: 非法的参数 at /***/perl/5.10/Module/Build/Base.pm line 2585 ------------------------------------------------ That was because a filename contains ":" was illegal(at least on Ubuntu & Win). I'd checked the source code of Base.pm. The method "manpage_separator" returns "::" which causes this problem, and "/" will be OK. Did this mean it would return an illegal filename when installing a subclass?

Wed Apr 29 14:43:11 2009 dagolden [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #45544] A compilation problem may caused by Base.pm?
Date:	Wed, 29 Apr 2009 14:42:47 -0400
To:	bug-Module-Build [...] rt.cpan.org
From:	David Golden <dagolden [...] cpan.org>

On Wed, Apr 29, 2009 at 1:30 AM, Jason Lee via RT <bug-Module-Build@rt.cpan.org> wrote: Show quoted text

> That was because a filename contains ":" was illegal(at least on > Ubuntu & Win). I'd checked the source code of Base.pm. The > method "manpage_separator" returns "::" which causes this problem, > and "/" will be OK. Did this mean it would return an illegal filename > when installing a subclass?

It works fine for me on Ubuntu 8.10. How is your filesystem formatted? (run "mount" and see what says for the partition containing the blib in your error. -- David

Wed Apr 29 14:43:12 2009 The RT System itself - Status changed from 'new' to 'open'

Thu Apr 30 06:57:47 2009 jcyl_baihu [...] 163.com - Correspondence added

From:

jcyl_baihu [...] 163.com

在 2009-04-29 14:43:11 星期三时，DAGOLDEN 写到： Show quoted text

> > It works fine for me on Ubuntu 8.10. How is your filesystem > formatted? (run "mount" and see what says for the partition > containing the blib in your error. > > -- David

vfat. Does that matter? I didn't know.....

Thu Apr 30 08:54:32 2009 dagolden [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #45544] A compilation problem may caused by Base.pm?
Date:	Thu, 30 Apr 2009 08:54:13 -0400
To:	bug-Module-Build [...] rt.cpan.org
From:	David Golden <dagolden [...] cpan.org>

On Thu, Apr 30, 2009 at 6:57 AM, Jason Lee via RT <bug-Module-Build@rt.cpan.org> wrote: Show quoted text

> vfat. Does that matter? I didn't know.....

Yes, that's the problem. Or the bug, depending on one's point of view. On a unix-like OS, most things assume typical unix filename constraints, but vfat has Windows filename constraints, even when mounted by a unix operating system. Still -- it would probably be good for M::B to try to detect this and give you a more useful error message. -- David

Thu Apr 30 12:27:01 2009 jcyl_baihu [...] 163.com - Correspondence added

在 2009-04-30 08:54:32 星期四时，DAGOLDEN 写到： Show quoted text

> Yes, that's the problem. Or the bug, depending on one's point of > view. On a unix-like OS, most things assume typical unix filename > constraints, but vfat has Windows filename constraints, even when > mounted by a unix operating system. > > Still -- it would probably be good for M::B to try to detect this and > give you a more useful error message. > > -- David

I see that. Thank you very much! I should have noticed that I still worked on a Windows filesystem. It's not a good habit, I know... I'm almost a beginner in Unix-like OS... Maybe a detection would be a good idea, if that won't introduce any other problems. Or perhaps simply return a "/" instead of "::" may have avoided this filesystem problem? I'm not quite sure if it will be a simpler solution.

Thu Apr 30 13:03:58 2009 dagolden [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #45544] A compilation problem may caused by Base.pm?
Date:	Thu, 30 Apr 2009 13:03:40 -0400
To:	bug-Module-Build [...] rt.cpan.org
From:	David Golden <dagolden [...] cpan.org>

On Thu, Apr 30, 2009 at 12:27 PM, Jason Lee via RT <bug-Module-Build@rt.cpan.org> wrote: Show quoted text

> Maybe a detection would be a good idea, if that won't introduce any > other problems. Or perhaps simply return a "/" instead of "::" may > have avoided this filesystem problem? I'm not quite sure if it will be > a simpler solution.

I've raised it on IRC on #p5p. My thought is that if :: doesn't work, then manpages shouldn't be created at all. There's no point changing the manpage name since they won't be reachable with "man" anyway. You'd have to just rely on "perldoc". -- David

Thu Apr 30 20:42:35 2009 jcyl_baihu [...] 163.com - Correspondence added

在 2009-04-30 13:03:58 星期四时，DAGOLDEN 写到： Show quoted text

> > I've raised it on IRC on #p5p. > > My thought is that if :: doesn't work, then manpages shouldn't be > created at all. There's no point changing the manpage name since they > won't be reachable with "man" anyway. You'd have to just rely on > "perldoc". > > -- David

Yep, my mistake. It didn't work with "/" unless a few changes were made. Well, thank you then!

Fri Jun 19 23:05:00 2009 dagolden [...] cpan.org - Subject changed from 'A compilation problem may caused by Base.pm?' to 'Can't create manpages on vfat partition on unix'

Wed Aug 19 15:43:50 2009 dagolden [...] cpan.org - Correspondence added

Fixed in repository trunk.

Wed Aug 19 15:43:59 2009 dagolden [...] cpan.org - Status changed from 'open' to 'patched'

Wed Aug 19 22:54:34 2009 jcyl_baihu [...] 163.com - Correspondence added

在 2009-08-19 15:43:50 星期三时，DAGOLDEN 写到： Show quoted text

> Fixed in repository trunk.

I see that. Thank you very much!

Wed Aug 19 22:54:35 2009 The RT System itself - Status changed from 'patched' to 'open'

Sat Aug 22 22:10:48 2009 dagolden [...] cpan.org - Status changed from 'open' to 'patched'

Mon Sep 07 22:46:56 2009 dagolden [...] cpan.org - Status changed from 'patched' to 'resolved'

Mon Sep 07 22:46:56 2009 dagolden [...] cpan.org - Fixed in 0.34_06 added