Re: [perl #51944] [DOCS] Cygwin Readme

[Will Coleda]

On Thu, Jul 17, 2008 at 8:33 AM, Reini Urban <[EMAIL PROTECTED]> wrote:
> 2008/7/16 Will Coleda <[EMAIL PROTECTED]>:

>> This looks good; I'd ask you double check which deps are required vs.
>> optional: I'm pretty sure we don't -require- any of those packages.
>> Ditto for their correspond build deps. (e.g. libgdbm-devel)
>
> Note, there are more cygwin-side updates to come to this README,
> which I will commit after my parrot-0.6.4 release (in 1-2 days).
>
> In which pod should we document the non-windows package requirements?
> Most of those packages have similar names anyway for every unix/posix
> platform.
>
>> parrot is recursively listed as a build requirement
>
> oops.
>
>> For the optional items, perl's Moose should be on the list (for
>> smartlink), and Test::Perl::Critic/Perl::Critic for the critic tests.
>> (Honestly we should have this documented somewhere else and just refer
>> to it from here.)
>
> Good. Where?

How about docs/dependencies.pod (new file)? This can house the
optional systems we probe for, and a list of the non-core perl modules
we're using (and what they're used for.)

Thanks for the work on cygwin!

-- 
Will "Coke" Coleda

Re: [perl #51944] [DOCS] Cygwin Readme

[Reini Urban]

2008/7/16 Will Coleda <[EMAIL PROTECTED]>:
> On Wed, Jul 16, 2008 at 2:31 AM, Reini Urban via RT
> <[EMAIL PROTECTED]> wrote:
>> 2008/7/15 Reini Urban <[EMAIL PROTECTED]>:
>>> Will Coleda via RT schrieb:
 On Tue May 13 05:21:32 2008, rurban wrote:
> 2008/5/13 Andrew Whitworth via RT  [EMAIL PROTECTED]>:
>>
>> is this ticket (#51944) resolved? I don't see any outstanding todo
>
> items
>>
>>  here that need to be considered further, and the submitted patch
>
> has
>>
>>  already been applied. Can we close this, or is this a placeholder
>
> for us
>>
>>  to further improve cygwin documentation?
>
> The list os build preq's is required.
>
> Well, the tip for smoke to do cpan Test::TAP::HTMLMatrix
> could be added. But this should be added in the general README for all
> platforms.
>
> Then platform specific is only that pg.t fails due to missing loadlib
> exceptions
> and that cygwin perl-5.10.0 fails to send the smoke report. perl-5.8.8
> works ok.
>
> I'll send that as extra README_cygwin.pod patch.

 ENOPATCH. =-)

 Can someone with cygwin access go through the README once more so we can
 get this ticket closed? Thanks.
>>>
>>> Ok, since there's now almost an official parrot package, the updates are
>>> easy.
>>> Patch attached.
>>>
>>> The parrot-0.6.4-1 packages are in the works.
>>
>> I found a better link for the SDL link.
>> Yesterday the http://cygwinports.dotsrc.org/ site was down, but now it
>> is up again.
>>
>> Please use the attached revised patch instead.
>
> This looks good; I'd ask you double check which deps are required vs.
> optional: I'm pretty sure we don't -require- any of those packages.
> Ditto for their correspond build deps. (e.g. libgdbm-devel)

Note, there are more cygwin-side updates to come to this README,
which I will commit after my parrot-0.6.4 release (in 1-2 days).

In which pod should we document the non-windows package requirements?
Most of those packages have similar names anyway for every unix/posix
platform.

> parrot is recursively listed as a build requirement

oops.

> For the optional items, perl's Moose should be on the list (for
> smartlink), and Test::Perl::Critic/Perl::Critic for the critic tests.
> (Honestly we should have this documented somewhere else and just refer
> to it from here.)

Good. Where?

> Here are some notes about the build/tests..
>
> ** I get this warning on configure:
>
> Determining CPU architecture and OS...done.
> Determining JIT capability... 10 [main] test 1944 
> _cygtls::handle_exceptions
> : Error while dumping state (probably corrupted stack)
> 10 [main] test 4596 _cygtls::handle_exceptions: Error while dumping state 
> (
> probably corrupted stack)
> ..yes.

There's a cygwin jit patch outstanding.
[perl #56824] [PATCH] Configure - fix SEGV in JIT has_exec_protect
test on Cygwin

> Generating CPU specific stuff.done.
> Verifying that the compiler supports function pointer castsyes.
>
> ** Get a similar error running t/src/extend.t
>
> ** Bug in t/doc/pod.t - few warnings of the type:
>
> Error: /cygdrive/d/sandbox/parrot/pbc_to_exe is unreadable: No such file or 
> dire
> ctory

Yes, this needs to be fixed, but it's not so urgent.
Just the test is wrong.

> ... But all tests pass.

More urgent for me is fixing the .include search path.
[perl #56996] [TODO] remove non FHS-compliant searchpaths

-- 
Reini Urban
http://phpwiki.org/ http://murbreak.at/

Re: [perl #51944] [DOCS] Cygwin Readme

[Will Coleda]

On Wed, Jul 16, 2008 at 2:31 AM, Reini Urban via RT
<[EMAIL PROTECTED]> wrote:
> 2008/7/15 Reini Urban <[EMAIL PROTECTED]>:
>> Will Coleda via RT schrieb:
>>>
>>> On Tue May 13 05:21:32 2008, rurban wrote:

 2008/5/13 Andrew Whitworth via RT >>> [EMAIL PROTECTED]>:
>
> is this ticket (#51944) resolved? I don't see any outstanding todo

 items
>
>  here that need to be considered further, and the submitted patch

 has
>
>  already been applied. Can we close this, or is this a placeholder

 for us
>
>  to further improve cygwin documentation?

 The list os build preq's is required.

 Well, the tip for smoke to do cpan Test::TAP::HTMLMatrix
 could be added. But this should be added in the general README for all
 platforms.

 Then platform specific is only that pg.t fails due to missing loadlib
 exceptions
 and that cygwin perl-5.10.0 fails to send the smoke report. perl-5.8.8
 works ok.

 I'll send that as extra README_cygwin.pod patch.
>>>
>>> ENOPATCH. =-)
>>>
>>> Can someone with cygwin access go through the README once more so we can
>>> get this ticket closed? Thanks.
>>
>> Ok, since there's now almost an official parrot package, the updates are
>> easy.
>> Patch attached.
>>
>> The parrot-0.6.4-1 packages are in the works.
>
> I found a better link for the SDL link.
> Yesterday the http://cygwinports.dotsrc.org/ site was down, but now it
> is up again.
>
> Please use the attached revised patch instead.
> --
> Reini Urban
> http://phpwiki.org/ http://murbreak.at/
>
>

This looks good; I'd ask you double check which deps are required vs.
optional: I'm pretty sure we don't -require- any of those packages.
Ditto for their correspond build deps. (e.g. libgdbm-devel)

parrot is recursively listed as a build requirement

For the optional items, perl's Moose should be on the list (for
smartlink), and Test::Perl::Critic/Perl::Critic for the critic tests.

(Honestly we should have this documented somewhere else and just refer
to it from here.)

---

Here are some notes about the build/tests..

** I get this warning on configure:

Determining CPU architecture and OS...done.
Determining JIT capability... 10 [main] test 1944 _cygtls::handle_exceptions
: Error while dumping state (probably corrupted stack)
 10 [main] test 4596 _cygtls::handle_exceptions: Error while dumping state (
probably corrupted stack)
..yes.
Generating CPU specific stuff.done.
Verifying that the compiler supports function pointer castsyes.

** Get a similar error running t/src/extend.t

** Bug in t/doc/pod.t - few warnings of the type:

Error: /cygdrive/d/sandbox/parrot/pbc_to_exe is unreadable: No such file or dire
ctory

... But all tests pass.


Regards.
-- 

Will "Coke" Coleda

Re: [perl #51944] [DOCS] Cygwin Readme

[Reini Urban]

2008/7/15 Reini Urban <[EMAIL PROTECTED]>:
> Will Coleda via RT schrieb:
>>
>> On Tue May 13 05:21:32 2008, rurban wrote:
>>>
>>> 2008/5/13 Andrew Whitworth via RT >> [EMAIL PROTECTED]>:

 is this ticket (#51944) resolved? I don't see any outstanding todo
>>>
>>> items

  here that need to be considered further, and the submitted patch
>>>
>>> has

  already been applied. Can we close this, or is this a placeholder
>>>
>>> for us

  to further improve cygwin documentation?
>>>
>>> The list os build preq's is required.
>>>
>>> Well, the tip for smoke to do cpan Test::TAP::HTMLMatrix
>>> could be added. But this should be added in the general README for all
>>> platforms.
>>>
>>> Then platform specific is only that pg.t fails due to missing loadlib
>>> exceptions
>>> and that cygwin perl-5.10.0 fails to send the smoke report. perl-5.8.8
>>> works ok.
>>>
>>> I'll send that as extra README_cygwin.pod patch.
>>
>> ENOPATCH. =-)
>>
>> Can someone with cygwin access go through the README once more so we can
>> get this ticket closed? Thanks.
>
> Ok, since there's now almost an official parrot package, the updates are
> easy.
> Patch attached.
>
> The parrot-0.6.4-1 packages are in the works.

I found a better link for the SDL link.
Yesterday the http://cygwinports.dotsrc.org/ site was down, but now it
is up again.

Please use the attached revised patch instead.
-- 
Reini Urban
http://phpwiki.org/ http://murbreak.at/
Index: README_cygwin.pod
===
--- README_cygwin.pod	(revision 29483)
+++ README_cygwin.pod	(working copy)
@@ -6,37 +6,66 @@
 
 =head1 SYNOPSIS
 
-Parrot builds out of the box under Cygwin.  Some tweaks are needed for
-different names of dynamic loading of some dll's.
+Parrot builds out of the box under Cygwin. 
+Some tweaks are needed for different names for the ffi to some dll's.
+See L
 
+There are official cygwin parrot packages in preparation. 
+See L
+
+  parrot, libparrot0, libparrot-devel, parrot-perl6, parrot-languages
+
 =head1 Packages
 
-You'll need the following Cygwin packages to build Parrot.
+You'll need the following Cygwin packages to run and build Parrot.
 
-=over 4
+Runtime requirements:
 
-=item gcc
+  libreadline6 ncurses libintl8 libicu38 libgmp3 libgdbm4
 
-=item make
+Optional requirements:
 
-=item perl
+  libglut3 xorg-x11-base xorg-x11-bin-dlls libpq5 openssl
 
-=item subversion
+Build requirements:
 
+  gcc make perl parrot readline libncurses-devel libgmp-devel
+  libgdbm-devel pcre-devel libglut-devel
+
+Optional build requirements:
+
+  libicu-devel openssl-devel 
+
+CPAN packages:
+
+  L if you want to run the language smoke tests
+  with C.
+
+  L for some APL language tests.
+
+=over 4
+
+=item Cygwin subversion and perl
+
 If you use SVN to get a copy of Parrot, you should use the Cygwin SVN
 and not the TortoiseSVN client to avoid build errors.  Similarly you will
 need Cygwin Perl rather than ActiveState or Strawberry Perl.
 
-=item ICU
+=item icu
 
-This is no official Cygwin package yet.  However, icu4c-3_8 builds out of the
-box on Cygwin.
-
-  http://download.icu-project.org/files/icu4c/3.8/icu4c-3_8-src.tgz
-
 Note that ICU is now optional, you can build Parrot without it,
 by not installing it or asking Parrot to ignore it (C<--without-icu>).
 
+=item SDL
+
+SDL references F, which is only in cygports
+L
+
+=item aio
+
+libaio-devel "Linux-native asynchronous I/O access" is not available
+for cygwin, and as the name says will never be :)
+
 =back
 
 =head1 BUILD
@@ -53,14 +82,12 @@
 
 =item Makefile tuning
 
-rename libparrot.dll to cygparrot.dll, create an interim libparrot.dll.a
+rename libparrot.dll to cygparrot-0-6.dll and create an interim libparrot.dll.a
 
-fix the blib/lib PATH issue
+=item loadlib DLL versioning
 
-=item DLL versioning
+Use cyg*-1.1.dll instead of lib*.so.1.1 names for loadlib, the FFI.
 
-cyg*-1.1.dll instead of lib*.so.1.1
-
 Thanks to the L and the impossibility of file
 hardlinks, windows dll names are versioned, so the loadlib function or the
 various pir's needs more logic.
@@ -89,6 +116,6 @@
 
 =head1 HISTORY
 
-Last updated: 1 June 2008
+Last updated: 15 July 2008
 
 =cut

Re: [perl #51944] [DOCS] Cygwin Readme

[Reini Urban]

Will Coleda via RT schrieb:

On Tue May 13 05:21:32 2008, rurban wrote:

2008/5/13 Andrew Whitworth via RT :

is this ticket (#51944) resolved? I don't see any outstanding todo

items

 here that need to be considered further, and the submitted patch

has

 already been applied. Can we close this, or is this a placeholder

for us

 to further improve cygwin documentation?

The list os build preq's is required.

Well, the tip for smoke to do cpan Test::TAP::HTMLMatrix
could be added. But this should be added in the general README for all
platforms.

Then platform specific is only that pg.t fails due to missing loadlib
exceptions
and that cygwin perl-5.10.0 fails to send the smoke report. perl-5.8.8
works ok.

I'll send that as extra README_cygwin.pod patch.


ENOPATCH. =-)

Can someone with cygwin access go through the README once more so we can
get this ticket closed? Thanks.


Ok, since there's now almost an official parrot package, the updates are 
easy.

Patch attached.

The parrot-0.6.4-1 packages are in the works.
--
Reini Urban
http://phpwiki.org/  http://murbreak.at/
Index: README_cygwin.pod
===
--- README_cygwin.pod	(revision 29483)
+++ README_cygwin.pod	(working copy)
@@ -6,37 +6,66 @@
 
 =head1 SYNOPSIS
 
-Parrot builds out of the box under Cygwin.  Some tweaks are needed for
-different names of dynamic loading of some dll's.
+Parrot builds out of the box under Cygwin. 
+Some tweaks are needed for different names for the ffi to some dll's.
+See L
 
+There are official cygwin parrot packages in preparation. 
+See L
+
+  parrot, libparrot0, libparrot-devel, parrot-perl6, parrot-languages
+
 =head1 Packages
 
-You'll need the following Cygwin packages to build Parrot.
+You'll need the following Cygwin packages to run and build Parrot.
 
-=over 4
+Runtime requirements:
 
-=item gcc
+  libreadline6 ncurses libintl8 libicu38 libgmp3 libgdbm4
 
-=item make
+Optional requirements:
 
-=item perl
+  libglut3 xorg-x11-base xorg-x11-bin-dlls libpq5 openssl
 
-=item subversion
+Build requirements:
 
+  gcc make perl parrot readline libncurses-devel libgmp-devel
+  libgdbm-devel pcre-devel libglut-devel
+
+Optional build requirements:
+
+  libicu-devel openssl-devel 
+
+CPAN packages: 
+
+  L if you want to run the language smoke tests
+  with C.
+
+  L for some APL language tests.
+
+=over 4
+
+=item Cygwin subversion and perl
+
 If you use SVN to get a copy of Parrot, you should use the Cygwin SVN
 and not the TortoiseSVN client to avoid build errors.  Similarly you will
 need Cygwin Perl rather than ActiveState or Strawberry Perl.
 
-=item ICU
+=item icu
 
-This is no official Cygwin package yet.  However, icu4c-3_8 builds out of the
-box on Cygwin.
-
-  http://download.icu-project.org/files/icu4c/3.8/icu4c-3_8-src.tgz
-
 Note that ICU is now optional, you can build Parrot without it,
 by not installing it or asking Parrot to ignore it (C<--without-icu>).
 
+=item SDL
+
+SDL references F, which is only in cygports
+L
+
+=item aio
+
+libaio-devel "Linux-native asynchronous I/O access" is not available
+for cygwin, and as the name says will never be :)
+
 =back
 
 =head1 BUILD
@@ -53,14 +82,12 @@
 
 =item Makefile tuning
 
-rename libparrot.dll to cygparrot.dll, create an interim libparrot.dll.a
+rename libparrot.dll to cygparrot-0-6.dll and create an interim libparrot.dll.a
 
-fix the blib/lib PATH issue
+=item loadlib DLL versioning
 
-=item DLL versioning
+Use cyg*-1.1.dll instead of lib*.so.1.1 names for loadlib, the FFI.
 
-cyg*-1.1.dll instead of lib*.so.1.1
-
 Thanks to the L and the impossibility of file
 hardlinks, windows dll names are versioned, so the loadlib function or the
 various pir's needs more logic.
@@ -89,6 +116,6 @@
 
 =head1 HISTORY
 
-Last updated: 1 June 2008
+Last updated: 15 July 2008
 
 =cut

[perl #51944] [DOCS] Cygwin Readme

[Will Coleda via RT]

On Tue May 13 05:21:32 2008, rurban wrote:
> 2008/5/13 Andrew Whitworth via RT  [EMAIL PROTECTED]>:
> > is this ticket (#51944) resolved? I don't see any outstanding todo
> items
> >  here that need to be considered further, and the submitted patch
> has
> >  already been applied. Can we close this, or is this a placeholder
> for us
> >  to further improve cygwin documentation?
> 
> The list os build preq's is required.
> 
> Well, the tip for smoke to do cpan Test::TAP::HTMLMatrix
> could be added. But this should be added in the general README for all
> platforms.
> 
> Then platform specific is only that pg.t fails due to missing loadlib
> exceptions
> and that cygwin perl-5.10.0 fails to send the smoke report. perl-5.8.8
> works ok.
> 
> I'll send that as extra README_cygwin.pod patch.

ENOPATCH. =-)

Can someone with cygwin access go through the README once more so we can
get this ticket closed? Thanks.

-- 
Will "Coke" Coleda