[PATCH MANIFEST lib/ExtUtils/MM_Cygwin.t] Add tests for ExtUtils::MM_Cygwin

2001-11-25 Thread chromatic
Here's tests for ExtUtils::MM_Cygwin. They're skipped on all platforms where $^O does not match /cygwin/i. The tests out to pass just about everywhere anyway. It would be good for someone with Cygwin to test them, though. -- c --- ~MANIFEST Sun Nov 25 19:50:46 2001 +++ MANIFESTSun Nov 2

Re: [PATCH MANIFEST lib/ExtUtils/MM_Cygwin.t] Add tests for ExtUtils::MM_Cygwin

2001-11-25 Thread Michael G Schwern
On Sun, Nov 25, 2001 at 08:05:28PM -0700, chromatic wrote: > +BEGIN { > + if ($^O =~ /cygwin/i) { > + plan tests => 17; > + $ENV{'ExtUtils/MM_Unix.pm'} = 1; $ENV{'ExtUtils/MM_Unix.pm'} ?? Maybe this supposed to be %INC? > +# test cflags, with the fake package below

Re: [PATCH MANIFEST lib/ExtUtils/MM_Cygwin.t] Add tests for ExtUtils::MM_Cygwin

2001-11-25 Thread chromatic
On Sunday 25 November 2001 20:47, Michael G Schwern wrote: > $ENV{'ExtUtils/MM_Unix.pm'} ?? Maybe this supposed to be %INC? Indeed it is. Never patch while cursing the heavy hand of George Lucas. > You should be playing with MM objects instead. ExtUtils::MM_Cygwin > will handle setting @MM::

Re: [PATCH MANIFEST lib/ExtUtils/MM_Cygwin.t] Add tests for ExtUtils::MM_Cygwin

2001-11-25 Thread Michael G Schwern
On Sun, Nov 25, 2001 at 09:10:32PM -0700, chromatic wrote: > On Sunday 25 November 2001 20:47, Michael G Schwern wrote: > > > $ENV{'ExtUtils/MM_Unix.pm'} ?? Maybe this supposed to be %INC? > > Indeed it is. Never patch while cursing the heavy hand of George Lucas. Hard to code while beating t

[REPATCH MANIFEST lib/ExtUtils/MM_Cygwin.t] use more 'Schwernishness';

2001-11-25 Thread chromatic
On Sunday 25 November 2001 21:38, Michael G Schwern wrote: > > Indeed it is. Never patch while cursing the heavy hand of George Lucas. > Hard to code while beating the hell out of your television, I know the > feeling. Television? He's playing congas two doors down. Noisy old goat... > +# te

Re: [REPATCH MANIFEST lib/ExtUtils/MM_Cygwin.t] use more 'Schwernishness';

2001-11-25 Thread Michael G Schwern
On Sun, Nov 25, 2001 at 09:42:00PM -0700, chromatic wrote: > > And I'm curious why it's necessary to still override bits of > > ExtUtils::MM_Unix? > > Porta-paranoia. cflags() and canonpath() are delegated to it. This may be > why File::Spec was commingled. Now wait a second, this means you'r

Re: [REPATCH MANIFEST lib/ExtUtils/MM_Cygwin.t] use more 'Schwernishness';

2001-11-25 Thread chromatic
On Sunday 25 November 2001 22:31, Michael G Schwern wrote: > > Porta-paranoia. cflags() and canonpath() are delegated to it. This may > > be why File::Spec was commingled. > Now wait a second, this means you're not actually testing what > ExtUtils::MM_Cygwin->catfile() and cflags() and perl_sc

Re: [REPATCH MANIFEST lib/ExtUtils/MM_Cygwin.t] use more 'Schwernishness';

2001-11-25 Thread Michael G Schwern
On Sun, Nov 25, 2001 at 10:31:15PM -0700, chromatic wrote: > On Sunday 25 November 2001 22:31, Michael G Schwern wrote: > > > > Porta-paranoia. cflags() and canonpath() are delegated to it. This may > > > be why File::Spec was commingled. > > > Now wait a second, this means you're not actually