Bug #49948 for Perl-Dist-Strawberry: strawberry-perl-5.10.x.x.zip needs a README file.

Wed Sep 23 10:36:35 2009 kmx [...] cpan.org - Ticket created

Subject:

strawberry-perl-5.10.x.x.zip vs. 5.10.x.x-portable.zip

Do you think it is still necessary to build both: - strawberry-perl-5.10.x.x.zip (suitable for c:\strawberry\ only) - strawberry-perl-5.10.x.x-portable.zip Would not be just the portable ZIP enough? I am using portable 5.10 ZIP for a couple of months without any problems. -- kmx

Thu Sep 24 03:54:51 2009 kmx [...] cpan.org - Correspondence added

Hi, I take into account the reasons discussed with Alias+CSJewell yesterday on IRC. However, it would be definitely a good idea to include at least a short README.TXT inside non-portable ZIPs saying: 1) you have to unpack the ZIP into c:\strawberry 2) you have to manually add c:\strawberry\perl\bin;c:\strawberry\c\bin into your PATH Otherwise it is quite confusing for strawberry newcomers (even for advanced users) as there is no remark that this ZIP should go to only to c:\straberry. -- kmx

Thu Sep 24 03:54:52 2009 kmx [...] cpan.org - Status changed from 'new' to 'open'

Sun Sep 27 23:59:55 2009 csjewell [...] cpan.org - Correspondence added

Subject:

strawberry-perl-5.10.x.x.zip needs a README file.

On Thu Sep 24 03:54:51 2009, KMX wrote: Show quoted text

> However, it would be definitely a good idea to include at least a short > README.TXT inside non-portable ZIPs saying:

Could you take a look at the share\strawberry\README.txt.tt included in r9351? It may need a little more description of what Strawberry is. [It's meant to be included in both .zip and .msi builds. It's easier that way.] [NOTE: .tt -> Template Toolkit template, so it can call any Perl::Dist::Strawberry or Perl::Dist::WiX routine as [% dist.x %].]

Mon Sep 28 02:13:38 2009 csjewell [...] cpan.org - Severity Normal added

Mon Sep 28 02:22:01 2009 csjewell [...] cpan.org - Reference by http://fisheye2.atlassian.com/changelog/cpan/?cs=9351 added

Mon Sep 28 02:22:01 2009 csjewell [...] cpan.org - Subject changed from 'strawberry-perl-5.10.x.x.zip vs. 5.10.x.x-portable.zip' to 'strawberry-perl-5.10.x.x.zip needs a README file.'

Mon Sep 28 05:40:00 2009 kmx [...] volny.cz - Correspondence added

Subject:	Re: [rt.cpan.org #49948] strawberry-perl-5.10.x.x.zip needs a README file.
Date:	Mon, 28 Sep 2009 11:39:10 +0200
To:	bug-Perl-Dist-Strawberry [...] rt.cpan.org
From:	kmx <kmx [...] volny.cz>

Show quoted text

> Could you take a look at the share\strawberry\README.txt.tt included in > r9351? >

See r9353 Question1 - Do we want to recommend: * set PATH=%PATH%;[% dist.image_dir %]\c\bin;[% dist.image_dir %]\perl\bin or rather * set PATH=[% dist.image_dir %]\c\bin;[% dist.image_dir %]\perl\bin;%PATH% Question2 - is setting of TERM=dumb + FTP_PASSIVE=1 really necessary? -- kmx

Mon Sep 28 11:58:52 2009 csjewell [...] cpan.org - Correspondence added

On Mon Sep 28 05:40:00 2009, kmx@volny.cz wrote: Show quoted text

>

> > Could you take a look at the share\strawberry\README.txt.tt included in > > r9351? > >

> See r9353

Looks great! Thanks. Think you missed one word, but other than that, it's a keeper. Show quoted text

> Question1 - Do we want to recommend: > * set PATH=%PATH%;[% dist.image_dir %]\c\bin;[% dist.image_dir %]\perl\bin > or rather > * set PATH=[% dist.image_dir %]\c\bin;[% dist.image_dir %]\perl\bin;%PATH%

The first. While we're getting better, there still could be a few .dll problems, and the second would make us break other programs when those problems happen. Show quoted text

> Question2 - is setting of TERM=dumb + FTP_PASSIVE=1 really necessary?

Only if you plan to use "cpan Module::To::Install" or "cpan". :) The Term::Read{Line,Key} packages require the first, and Net::FTP requires the second to get across some firewalls, if I recall correctly, so we play it safe and recommend it to everyone. And yes, the .msi sets those.

Mon Sep 28 22:10:37 2009 adam [...] ali.as - Correspondence added

Subject:	Re: [rt.cpan.org #49948] strawberry-perl-5.10.x.x.zip needs a README file.
Date:	Tue, 29 Sep 2009 12:10:19 +1000
To:	bug-Perl-Dist-Strawberry [...] rt.cpan.org
From:	Adam Kennedy <adamkennedybackup [...] gmail.com>

We theoretically can dump the second once CPAN client reaches a point where we're sure it will never attempt to load a ftp cpan mirror. We can dump the former, if we can get bug fixes into the Term::ReadLine family such that it autodetects on Windows. Adam K 2009/9/29 Curtis Jewell via RT <bug-Perl-Dist-Strawberry@rt.cpan.org>: Show quoted text

> Queue: Perl-Dist-Strawberry > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=49948 > > > On Mon Sep 28 05:40:00 2009, kmx@volny.cz wrote:

>>

>> > Could you take a look at the share\strawberry\README.txt.tt included in >> > r9351? >> >

>> See r9353

> > Looks great! Thanks. Think you missed one word, but other than that, > it's a keeper. >

>> Question1 - Do we want to recommend: >> * set PATH=%PATH%;[% dist.image_dir %]\c\bin;[% dist.image_dir %]\perl\bin >> or rather >> * set PATH=[% dist.image_dir %]\c\bin;[% dist.image_dir %]\perl\bin;%PATH%

> > The first. While we're getting better, there still could be a few .dll > problems, and the second would make us break other programs when those > problems happen. >

>> Question2 - is setting of TERM=dumb + FTP_PASSIVE=1 really necessary?

> > Only if you plan to use "cpan Module::To::Install" or "cpan". :) The > Term::Read{Line,Key} packages require the first, and Net::FTP requires > the second to get across some firewalls, if I recall correctly, so we > play it safe and recommend it to everyone. > > And yes, the .msi sets those. > > >

Tue Sep 29 03:08:18 2009 kmx [...] volny.cz - Correspondence added

Subject:	Re: [rt.cpan.org #49948] strawberry-perl-5.10.x.x.zip needs a README file.
Date:	Tue, 29 Sep 2009 09:07:56 +0200
To:	bug-Perl-Dist-Strawberry [...] rt.cpan.org
From:	kmx <kmx [...] volny.cz>

Reserved Local Account via RT napsal(a): Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=49948 > > > We theoretically can dump the second once CPAN client reaches a point > where we're sure it will never attempt to load a ftp cpan mirror. > > We can dump the former, if we can get bug fixes into the > Term::ReadLine family such that it autodetects on Windows. >

In that case we probably HAVE TO add set TERM=dumb also to newly introduced portableshell.bat And I would also vote for trying to propose a patch to Term::ReadLine - the actual patch seems to be very easy and simple, however I do not know how they will be open to apply it (Term::Readline seems to be a core module). -- kmx

Wed Sep 30 04:14:59 2009 kmx [...] cpan.org - Correspondence added

patched in SVN - see r9407 + r9408 http://fisheye2.atlassian.com/changelog/cpan/?cs=9407 http://fisheye2.atlassian.com/changelog/cpan/?cs=9408 -- kmx

Wed Sep 30 04:15:00 2009 kmx [...] cpan.org - Status changed from 'open' to 'patched'

Thu Oct 15 03:06:39 2009 kmx [...] cpan.org - Correspondence added

Subject:

bad README.TXT in RC1

README.TXT in RC1 was not correctly generated 1) the first line 2) thye section: set PATH=%PATH%;\c\bin;\perl\bin -- kmx

Thu Oct 15 03:06:41 2009 The RT System itself - Status changed from 'patched' to 'open'

Fri Oct 16 17:14:01 2009 csjewell [...] cpan.org - Correspondence added

On Thu Oct 15 03:06:39 2009, KMX wrote: Show quoted text

> README.TXT in RC1 was not correctly generated > > 1) the first line > > 2) thye section: set PATH=%PATH%;\c\bin;\perl\bin

r9694 should fix it. Testing.

Mon Oct 26 15:32:34 2009 csjewell [...] cpan.org - Status changed from 'open' to 'patched'

Wed Oct 28 13:38:03 2009 csjewell [...] cpan.org - Correspondence added

Resolving since October release happened.

Wed Oct 28 13:38:04 2009 The RT System itself - Status changed from 'patched' to 'open'

Wed Oct 28 13:38:05 2009 csjewell [...] cpan.org - Status changed from 'open' to 'resolved'