Skip Menu |

This queue is for tickets about the ExtUtils-MakeMaker CPAN distribution.

Report information
The Basics
Id: 17041
Status: resolved
Worked: 5 min
Priority: 0/
Queue: ExtUtils-MakeMaker

People
Owner: Nobody in particular
Requestors: adamk [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Wishlist
Broken in: 6.30_01
Fixed in: 6.75_03



Subject: Sort manification and copying events for perceptive cleanliness
Currently a number of lists are written based on hashes, so they are written to the make file is pseudo-random order, and thus run in pseudo-random order. This makes things more "cluttered" on the eyes and harder to follow when watching things being installed. Where possible, we should sort lists being written to the Makefile (where the order of these lists does not actually matter), so things run in sorted order and are easier on the eyes. So please add a sort to the following locations I've traced down... ExtUtils::MM_Any::manifypods_target(/usr/share/perl/5.8/ExtUtils/MM_Any.pm:358): 358: foreach my $name (keys %{$self->{MAN1PODS}}, keys %{$self->{MAN3PODS}}) { ExtUtils::MM_Any::manifypods_target(/usr/share/perl/5.8/ExtUtils/MM_Any.pm:362): 362: foreach my $name (keys %{$self->{MAN3PODS}}) { Possibly also to ExtUtils::MM_Any::split_command, or even better just to the following places that call it... 373: push @man_cmds, $self->split_command(<<CMD, %$pods); 374: \$(NOECHO) \$(POD2MAN) --section=\$(MAN${section}EXT) --perm_rw=\$(PERM_R
On Fri Jan 13 02:54:31 2006, ADAMK wrote: Show quoted text
> Currently a number of lists are written based on hashes, so they are > written to the make file is pseudo-random order, and thus run in > pseudo-random order. > > This makes things more "cluttered" on the eyes and harder to follow > when > watching things being installed. > > Where possible, we should sort lists being written to the Makefile > (where the order of these lists does not actually matter), so things > run > in sorted order and are easier on the eyes. > > So please add a sort to the following locations I've traced down... > > ExtUtils::MM_Any::manifypods_target(/usr/share/perl/5.8/ExtUtils/MM_Any.pm:358): > 358: foreach my $name (keys %{$self->{MAN1PODS}}, keys > %{$self->{MAN3PODS}}) { > > ExtUtils::MM_Any::manifypods_target(/usr/share/perl/5.8/ExtUtils/MM_Any.pm:362): > 362: foreach my $name (keys %{$self->{MAN3PODS}}) { > > Possibly also to ExtUtils::MM_Any::split_command, or even better just > to > the following places that call it... > > 373: push @man_cmds, $self->split_command(<<CMD, %$pods); > 374: \$(NOECHO) \$(POD2MAN) --section=\$(MAN${section}EXT) > --perm_rw=\$(PERM_R
Many thanks, this has now been patched in the EUMM source repository and will be in a future stable release.
Stable release 6.76 is now on CPAN.