Re: Fwd: [Module::Build] Does ./Build work on Windows?

[<and...@fr...>]

>>>>> On Mon, 30 Jan 2006 15:55:36 -0500, David Golden <da...@hy...> said:

  > CPAN.pm 1.83_59 does this to set up to run Build.PL:
  >  my($perl) = $self->perl or die "Couldn\'t find executable perl\n";
  >  $system = "$perl Build.PL $CPAN::Config->{mbuildpl_arg}";

  > Then it does this to set up the subsequent call to it:

  >  $system = "./Build $CPAN::Config->{mbuild_arg}";

  >  $system = "./Build test";

  >  $system = "./Build clean";

  > Every single one of these could be replaced with something like the first:

  >  $system = "$perl ./Build $CPAN::Config->{mbuild_arg}";

  >  $system = "$perl ./Build test";

  >  $system = "$perl ./Build clean";

  > That's a minute patch that should make CPAN.pm run ./Build on any
  > system that has Perl and can run Build.PL in the first place.

David Golden, listen, the part of CPAN.pm is done in 1.83_60. Ticket
closed on the CPAN.pm side. I may have introduced bugs, and I hope I
will get feedback to fix them.

  > I don't understand why creating/destroying a .bat file is
  > preferable.

Because it is the *goal* of Module::Build to make life easier for the
users.

  > Am I missing something?

I'd say the obvious.

  > Only install is more complex:

  >  my($mbuild_install_build_command) =
  >             $CPAN::Config->{'mbuild_install_build_command'} ||
  >             "./Build";
  >  $system = join(" ",
  >                 $mbuild_install_build_command,
  >                 "install",
  >                 $CPAN::Config->{mbuild_install_arg},
  >  );

  > Even that could be "$perl ./Build" -- and the docs could say that
  > mbuild_install_arg should be "sudo perl ./Build".

  > Unless I'm missing something, I'd really rather see a simple (and
  > backwards compatible) patch to CPAN.pm than another crufty,
  > OS-specific behavior added to Module::Build.

Oh yeah, you're apparently missing what kind of problem Module::Build
is trying to solve.

I suggest this thread to be closed now. I for one will not argue on
the matter any more.

Bugreports welcome, but please open new threads for them.

Thanks,
-- 
andreas