List Info

Thread: Re: 2.22 released!




Re: 2.22 released!
user name
2007-05-29 04:37:59
On 29/05/07, Dan Kogai <dankogaidan.co.jp> wrote:
> http://www.dan.co.jp/~dankogai/cpan/Encode-2.22.tar.gz

> and CPAN near you.

Thanks, upgraded into bleadperl as change #31305.

> ! Makefile.pl
>    Encode used to suppress man page generation.  Now it
does.
>    http://r
t.cpan.org/NoAuth/Bug.html?id=#27200

Except this chunk. Man pages are generated by installman
inside the
core. So you need to make this conditional, e.g. by using
the same
method as Digest::MD5 :

push(extra, 'MAN3PODS' => {}) if $PERL_CORE; # Pods
built by installman.

(and likewise for MAN1PODS)

http://search.cpan.org/src/GAAS/Digest-MD5-2.36/Makef
ile.PL

Re: 2.22 released!
user name
2007-05-29 05:07:27
rgs & ghostbar,


On May 29, 2007, at 18:37 , Rafael Garcia-Suarez wrote:
>> ! Makefile.pl
>>    Encode used to suppress man page generation. 
Now it does.
>>    http://r
t.cpan.org/NoAuth/Bug.html?id=#27200
>
> Except this chunk. Man pages are generated by
installman inside the
> core. So you need to make this conditional, e.g. by
using the same
> method as Digest::MD5 :
>
> push(extra, 'MAN3PODS' => {}) if $PERL_CORE; # Pods
built by  
> installman.
>
> (and likewise for MAN1PODS)

Below is a preliminary patch that does just that.  Tell me
if it  
makes both of you happy.

Dan the Encode Maintainer


--- Makefile.PL 2006/05/03 18:24:10     2.3
+++ Makefile.PL 2007/05/29 10:06:16
 -25,6
+25,8 
$ARGV and push exe_files, more_exe_files;
$ARGV   and push pmlibdirs,
"ucm";
+my man =  ();
+man
= ( MAN1PODS => {}, MAN3PODS => {} ) if $PERL_CORE;
WriteMakefile(
            NAME         => "Encode",
 -36,8
+38,7 
            SUFFIX       => 'gz',
            DIST_DEFAULT => 'all tardist',
            },
-          MAN1PODS     => {},
-          MAN3PODS     => {},
+         man,
            INC       => "-I./Encode",
            PMLIBDIRS => pmlibdirs,
            INSTALLDIRS => 'perl',


Re: 2.22 released!
user name
2007-05-29 08:52:50
On Tue, May 29, 2007 at 11:37:59AM +0200, Rafael
Garcia-Suarez wrote:
> On 29/05/07, Dan Kogai <dankogaidan.co.jp> wrote:
> >http://www.dan.co.jp/~dankogai/cpan/Encode-2.22.tar.gz

> >and CPAN near you.
> 
> Thanks, upgraded into bleadperl as change #31305.
> 
> >! Makefile.pl
> >   Encode used to suppress man page generation. 
Now it does.
> >   http://r
t.cpan.org/NoAuth/Bug.html?id=#27200
> 
> Except this chunk. Man pages are generated by
installman inside the
> core. So you need to make this conditional, e.g. by
using the same
> method as Digest::MD5 :
> 
> push(extra, 'MAN3PODS' => {}) if $PERL_CORE; # Pods
built by installman.
> 
> (and likewise for MAN1PODS)
> 
> http://search.cpan.org/src/GAAS/Digest-MD5-2.36/Makef
ile.PL

Attached patch disables the install of man pages when
building by the core, by
setting INSTALLMAN3DIR=none when calling the Makefile.PL
(maybe
INSTALLMAN1DIR should also be set to '').
I think this is a better, more generic way to disable
installing man
pages when in the core.
(I send it previously as part of having CPAN modules in
their own
direcotry)



-- 
Gerard Goossen
TTY Internet Solutions

  
[1-3]

about | contact  Other archives ( Real Estate discussion Medical topics )