Skip Menu |

This queue is for tickets about the Perl-Dist-Strawberry CPAN distribution.

Report information
The Basics
Id: 55949
Status: resolved
Priority: 0/
Queue: Perl-Dist-Strawberry

People
Owner: Nobody in particular
Requestors: kmx [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Strawberry 5.12.0-RC1 - feedback
Hi,

some comments to the latest 5.12.0-RC1

1/ Warning "Strawberry cannot be installed in a directory with spaces" has some 3 strange charactes before and 5 strange characters after (strange = not present in my system font = displaying just empty squares).

2/ IMHO it is also necessary to test that the directory name contains only us-ascii characters

3/ Recapitulation screen (after choosing directory)  - "ready to install" - seems to be kind of empty/useless - it might be a good idea to put there at least some info (of course it has no priority).

4/ The final installer screen should say:
   - Go to start menu > "Strawberry Perl"
   - Before you start using Strawberry Perl read Release Notes and README

5/ Please consider including gcctoolchain related license info that I have packed into separate ZIP files
http://strawberryperl.com/package/kmx/32_gcctoolchain/mingw64-w32-20100123-kmx-v2-lic.zip
http://strawberryperl.com/package/kmx/64_gcctoolchain/mingw64-w64-20100123-kmx-v2-lic.zip

6/ We should perhaps update the file licenses/Licenses.rtf (some parts are better covered by licenses/<comonentname>/*)

7/ Consider reorganising startup menu similarly as I have proposed for Strawberry-PRO: separate folders "Tools" + "Related websites"; in root just "Perl (command line)" + "Strawberry Perl 5.12.0 Release notes" + "Strawberry Perl README" (by this I mean link to already existing README.TXT)

I have a feeling that Strawberry 5.12.x will be a success :)

--
kmx
One more thing to record from IRC discussion:

c:\strawberry\perl\site\bin should go before c:\strawberry\perl\bin in PATH

--
kmx
As for the newly introduced c:\strawberry\perl\site\bin in PATH we probably shoud also update README.TXT + portableshell.bat acordingly.

--
kmx

Another important issue concerning 64bit 5.12.0-RC

you have not used the latest gcc toolchain from:
http://strawberryperl.com/package/kmx/64_gcctoolchain/mingw64-w64-20100123-kmx-v2.zip
(it seems that you have used previous version - without "v2")

32bit 5.12.0-RC seems to be OK

--
kmx

On Sun Mar 28 11:30:29 2010, KMX wrote: Show quoted text
> Another important issue concerning 64bit 5.12.0-RC > > you have not used the latest gcc toolchain from: > http://strawberryperl.com/package/kmx/64_gcctoolchain/mingw64-w64- > 20100123-kmx-v2.zip > (it seems that you have used previous version - without "v2") > > 32bit 5.12.0-RC seems to be OK
This has been done, and license files should be installed, as of r11817.
On Sun Mar 28 03:32:38 2010, KMX wrote: Show quoted text
> As for the newly introduced c:\strawberry\perl\site\bin in PATH we
probably Show quoted text
> shoud also update README.TXT + portableshell.bat acordingly.
portableshell.bat: r11818 Strawberry's README.TXT: r11819
On Thu Mar 25 19:31:52 2010, KMX wrote: Show quoted text
> One more thing to record from IRC discussion: > > c:\strawberry\perl\site\bin should go before c:\strawberry\perl\bin in
PATH (already done for the installer, just putting it in the documentation and portableshell.bat file now) portableshell.bat, README.portable.txt: r11820 Strawberry's README.txt: r11821
On Thu Mar 25 19:01:56 2010, KMX wrote: Show quoted text
> Hi, > > some comments to the latest 5.12.0-RC1 > > 1/ Warning "Strawberry cannot be installed in a directory with spaces" > has some > 3 strange charactes before and 5 strange characters after (strange = > not > present in my system font = displaying just empty squares). > > 2/ IMHO it is also necessary to test that the directory name contains > only > us-ascii characters > > 3/ Recapitulation screen (after choosing directory) - "ready to > install" - > seems to be kind of empty/useless - it might be a good idea to put > there at > least some info (of course it has no priority). > > 4/ The final installer screen should say: > - Go to start menu > "Strawberry Perl" > - Before you start using Strawberry Perl read Release Notes and README > > 5/ Please consider including gcctoolchain related license info that I > have > packed into separate ZIP files > http://strawberryperl.com/package/kmx/32_gcctoolchain/mingw64-w32- > 20100123-kmx-v2-lic.zip > http://strawberryperl.com/package/kmx/64_gcctoolchain/mingw64-w64- > 20100123-kmx-v2-lic.zip > > 6/ We should perhaps update the file licenses/Licenses.rtf (some parts > are > better covered by licenses/<comonentname>/*) > > 7/ Consider reorganising startup menu similarly as I have proposed for > Strawberry-PRO: separate folders "Tools" + "Related websites"; in root > just > "Perl (command line)" + "Strawberry Perl 5.12.0 Release notes" + > "Strawberry > Perl README" (by this I mean link to already existing README.TXT) > > I have a feeling that Strawberry 5.12.x will be a success :)
I hate saying that we've run out of time to get to all of these for April, but we have. Just as a recap, 1 and 5 are done, the others will have to wait until after the April-2010-maintenance branch has been created. (They'll go into July's version, they are all good ideas!) --Curtis
I have tried
- to fix 6/ see commit 12018
- improve 5/ by changing destination from c:\strawberry\c\licenses > c:\strawberry\licenses see commit 12007

IMPORTANT - additional points reveled during beta testing

8/ we are missing license information fro GNU make (my "make" packages contain "license" subdir as all my packs)

9/ Installer MSI package version is set to just 5.12.0 not 5.12.0.0 as I would expected (have a look into control panel > list of installed SW)

--
kmx
On Thu Apr 22 16:56:54 2010, KMX wrote: Show quoted text
> I have tried > - to fix 6/ see commit 12018 > - improve 5/ by changing destination from c:\strawberry\c\licenses > > c:\strawberry\licenses see commit 12007 > > IMPORTANT - additional points reveled during beta testing > > 8/ we are missing license information fro GNU make (my "make" packages > contain > "license" subdir as all my packs) > > 9/ Installer MSI package version is set to just 5.12.0 not 5.12.0.0 as > I would > expected (have a look into control panel > list of installed SW)
#9 is known and is an msi limitation. You can only have 3 portions there, and the maximum is 255.255.32767, so the 3rd and 4th numbers in the Strawberry version are shifted together. 5.10.1.2 will be 5.10.258, for instance. There is a display string that should say 5.12.0.0 - I'll have to take another look. As for #8, good catch. I'll take a look at that tomorrow.
Dne Čt 22.dub.2010 23:33:44, CSJEWELL napsal(a):
Show quoted text
> On Thu Apr 22 16:56:54 2010, KMX wrote:
> > I have tried
> > - to fix 6/ see commit 12018
> > - improve 5/ by changing destination from c:\strawberry\c\licenses >
> > c:\strawberry\licenses see commit 12007
> >
> > IMPORTANT - additional points reveled during beta testing
> >
> > 8/ we are missing license information fro GNU make (my "make" packages
> > contain
> > "license" subdir as all my packs)
> >
> > 9/ Installer MSI package version is set to just 5.12.0 not 5.12.0.0 as
> > I would
> > expected (have a look into control panel > list of installed SW)
>
ad #9 - I thought that it is valid to use:
<Product Id='11111111-1234-1234-1234-123456789001' Name='Strawberry Perl' Version='5.12.0.0'>

but I have just a little experience with MSI

--
kmx

Show quoted text
> ad #9 - I thought that it is valid to use: > <Product Id='11111111-1234-1234-1234-123456789001' Name='Strawberry Perl' > Version='5.12.0.0'> > > but I have just a little experience with MSI
You /could/ use it, but because the 5.12.0.1 package would therefore have the same version as the 5.12.0.0 package (the 4th part gets dropped), you wouldn't be able to upgrade to 5.12.0.1 after 5.12.0.0. Therefore, the "shift the third plus the fourth" thing.
On Thu Apr 22 16:56:54 2010, KMX wrote: Show quoted text
> I have tried > - to fix 6/ see commit 12018
For future reference, we try to stay to RTF that WordPad can generate (which is why I took out the table borders.) Other than that, pretty good job, thanks! I did feel I had to mention specifically about licensing of some statically linked libraries that are included in Perl modules (which therefore would not have packs mentioning them.) Can you go back and mention where the license directory is for the toolchain that's still in 5.8/5.10? If so, that would be great. (I'm going to check and see if that onion.ico file you sent me works next.)
Subject: Re: [rt.cpan.org #55949] Strawberry 5.12.0-RC1 - feedback
Date: Sun, 25 Apr 2010 21:46:42 +0200
To: bug-Perl-Dist-Strawberry [...] rt.cpan.org
From: kmx <kmx [...] volny.cz>
Dne 23.4.2010 18:45, Curtis Jewell via RT napsal(a): Show quoted text
> ... > Can you go back and mention where the license directory is for the > toolchain that's still in 5.8/5.10? If so, that would be great. (I'm > going to check and see if that onion.ico file you sent me works next.
The license directory for gcc in strawberry 5.8/5.10 is also "license/gcc" so as for this the new license.rtf is "compatible" with both. There in no license info about binutils and mingw.org runtime in strawberry 5.8/5.10 - so in fact nothing to do. -- kmx
Just a recap what has to be fixed in July 2010

1/ [nice-to-have] IMHO it is also necessary to test that the directory name contains only us-ascii characters

2/ [nice-to-have] The final installer screen should say:
   - Go to start menu > "Strawberry Perl"
   - Before you start using Strawberry Perl read Release Notes and README

3/ [need-to-have] Reorganising startup menu (for newbies it is really hard to understand) - here is my proposal:
   - Tools\
   -      CPAN Client - install additional perl modules (cpan).lnk
   -      Check versions of installed modules (module-version).lnk
   -      Create local library areas (llw32helper).lnk
   - Realted websites\
   -      Strawberry Perl Website.lnk
   -      Win32 Perl Live Support (web chat)
   -      Win32 Perl Wiki.lnk
   -      CPAN Search (perl modules).lnk
   -      Perl 5.12.0 Documentation.lnk
   -      Beginning Perl (free book)
   -      Learning Perl (tutorials, links)
   - Perl (command prompt).lnk
   - Strawberry Perl README.lnk
   - Strawberry Perl Release Notes.lnk

4/ [need-to-have] The final installer screen should say:
   - Go to start menu > "Strawberry Perl"
   - Before you start using Strawberry Perl read Release Notes and README

Let us keep just these 4 cosmetics issues in this RT.

--
kmx
You repeated yourself with #2 and #4. :) Will deal with #4 tomorrow (It's the same type of fix as I'm going to do with the 5.10 versions needing to specify where they're going to install.) #1 is done as of r12274 (and r12278 - I needed the colon in acceptable characters, or nobody could install, plus, we should sat so, or people will be puzzled!) #2 will be an over the weekend/early next week fix, I think.
On Wed May 19 19:39:11 2010, CSJEWELL wrote: Show quoted text
> You repeated yourself with #2 and #4. :) > > Will deal with #4 tomorrow (It's the same type of fix as I'm going to do > with the 5.10 versions needing to specify where they're going to install.) > > #1 is done as of r12274 (and r12278 - I needed the colon in acceptable > characters, or nobody could install, plus, we should sat so, or people > will be puzzled!) > > #2 will be an over the weekend/early next week fix, I think.
#4 should be fixed (I'll check this) in r12281 & r11282.
On Thu May 20 00:48:55 2010, CSJEWELL wrote: Show quoted text
> On Wed May 19 19:39:11 2010, CSJEWELL wrote:
> > You repeated yourself with #2 and #4. :) > > > > Will deal with #4 tomorrow (It's the same type of fix as I'm going to do > > with the 5.10 versions needing to specify where they're going to
install.) Show quoted text
> > > > #1 is done as of r12274 (and r12278 - I needed the colon in acceptable > > characters, or nobody could install, plus, we should sat so, or people > > will be puzzled!) > > > > #2 will be an over the weekend/early next week fix, I think.
> > #4 should be fixed (I'll check this) in r12281 & r11282.
r12282, not r11282. :) #2 is done as of revisions 12290-12293 (with a few minor differences.) So marking 'patched', as that was the last thing on the list.