On Sat, 01 Mar 2008, Cosimo Streppone wrote:
> Latest changes
> --------------
> 
> 2008-03-01  Win32::API v0.52     Cosimo
> 
>     - Devel::AssertOS was not properly set up in 0.51.
> 
> 2008-03-01  Win32::API v0.51     Cosimo
> 
>     - Cleaned up API.pm pod docs and clearly stated Win32::API license
>     - Fixed $$/pid tests for Cygwin
>     - Now uses Devel::AssertOS to check that we are on a Win32 or Cygwin
>       system. This should ease the work of CPAN testers.

You still have problems because you didn't exclude the inc/ subdirectory
in your distribution from the CPAN indexer, which now marks this as an
unauthorized release:

    http://search.cpan.org/~cosimo/Win32-API-0.53/

The "unauthorized" status also hides this release from e.g. the CPAN.pm
shell, and I think the module list updates.  This can be fixed by excluding
this directory in the META.yml file.

However, I think using Devel::AssertOS is really overkill for this purpose.
I just use the following code in most Win32::* modules:

    unless ($^O eq "MSWin32" || $^O eq "cygwin") {
        die "OS unsupported\n";
    }

I'm not sure why using a module would be better for this simple use case.

Cheers,
-Jan

Reply via email to