search

StrawberryPerl / Perl-Dist-Strawberry

Tooling to build and package releases for Perl on Windows.
https://strawberryperl.com
Other
289 stars 49 forks source link

Perl 5.38.2.2 for 32bit? #177

Open gregoa opened 7 months ago

gregoa commented 7 months ago

I notice that there is a 64bit version for 5.38.2.2 but the 32bit version is still at 5.38.2.1. Would it be possible to update the latter as well?

shawnlaffan commented 7 months ago

The fourth digit in the version number is used for Strawberry Perl versioning, i.e. where changes are made to the SP build options, packaged modules and similar. The changes for 64 bit are listed with the release: https://github.com/StrawberryPerl/Perl-Dist-Strawberry/releases/tag/SP_53822_64bit

There have been no such changes to the 32 bit build for 5.38.2 so there is no need for a new release at this time. If there is a 5.38.3 Perl release then we will likely also build a Strawberry Perl version.

sisyphus commented 7 months ago

A couple of observations about that 32-bit build (5.38.2.1): 1) I can't find any link to it at https://strawberryperl.com/releases.html; 2) It has optimize level -Os.

IME it's only the 64-bit build that can't handle optimize level -O2, and -O2 is fine with the 32-bit builds.

@shawnlaffan, I hadn't realized that 5.38.2.2 had all of those additional optimization flags. (I've just been using plain ol' -Os for my 64-bit builds.) Are those extra flags all included in level -O2 ? (I think I should start including them in my own 64-bit builds.)

mohawk2 commented 7 months ago

What causes the problem with 64-bit and -O2? Is it reported to Perl5?

sisyphus commented 7 months ago

Is it reported to Perl5?

The original report is at https://github.com/Perl/perl5/issues/20024 . The PR that was merged to work around this issue is at https://github.com/Perl/perl5/pull/20136

shawnlaffan commented 7 months ago 
1. I can't find any link to it at https://strawberryperl.com/releases.html;

It does need to be added to the list. cc @genio 

2. It has optimize level `-Os`.

The GNUMakefile does not differentiate between 64 and 32 bit, and neither does the SP build recipe. -Os has nearly all of the -O2 optimisations so hopefully the speed difference is small in practice.

@shawnlaffan, I hadn't realized that 5.38.2.2 had all of those additional optimization flags. (I've just been using plain ol' -Os for my 64-bit builds.) Are those extra flags all included in level -O2 ? (I think I should start including them in my own 64-bit builds.)

The extra flags should all be from -O2. They are the set identified by @kenneth-olwing in https://github.com/Perl/perl5/issues/20081#issuecomment-1215744332

gregoa commented 7 months ago

The fourth digit in the version number is used for Strawberry Perl versioning, i.e. where changes are made to the SP build options, packaged modules and similar. The changes for 64 bit are listed with the release: https://github.com/StrawberryPerl/Perl-Dist-Strawberry/releases/tag/SP_53822_64bit

There have been no such changes to the 32 bit build for 5.38.2 so there is no need for a new release at this time. If there is a 5.38.3 Perl release then we will likely also build a Strawberry Perl version.

Right; I just happened to notice that there's 1 commit which adds a patch and is so far only applied to the 64bit recipe: https://github.com/StrawberryPerl/Perl-Dist-Strawberry/commit/09bf10cae2e7ccb9d6d6442fa2a5702bf7ee0cee#diff-a1fb92f3b097292ccdbfc7cb64931b1bb2ebca12e73ea469d2cd8df62b9b2203

shawnlaffan commented 7 months ago

That patch is needed to build perl 5.38 with gcc 13, which is used for the 64 bit releases (I cannot find the relevant issue right now).

The 32 bit release is built using gcc 8 which does not suffer the underlying issue. It is also built using the same stack as SP 5.32 so is getting a bit long in the tooth. Rebuilding all the external libraries at 32 bits for a newer compiler will likely be a fair amount of work so we are focusing on the 64 bit builds given they are much more commonly used.

gregoa commented 7 months ago

That patch is needed to build perl 5.38 with gcc 13, which is used for the 64 bit releases (I cannot find the relevant issue right now).

Oh, I see, thanks.

genio commented 7 months ago

I'll take a look at it when I land in Portugal for the Toolchain Summit

shawnlaffan commented 5 months ago

@genio - are you able to update the releases list on the website to add the 32-bit 5.38?

genio commented 5 months ago

I'm most of the way through a rebuild of the releases page to have all of that content auto-generated from the releases.json file. It's way too giant of a table mess at the moment and I'd rather make it somewhat more manageable. The front page already does that on the browser side.

Is there any reason we can't parse that releases.json in the browser and populate the page? Or, do we think that people are parsing that page as static content and would be upset at that large of a change? I changed the main page during the Perl ToolChain Summit and half expected to get yelled at by people that were parsing that page, but nobody has complained yet.

shawnlaffan commented 5 months ago

I see no reason not to use this approach.

I suspect most people would use the releases.json file directly rather than scrape the web site.

genio commented 4 months ago

Can I get a :+1: or :-1: on a re-design at https://strawberryperl.com/foo.html ?

shawnlaffan commented 4 months ago

Looks good overall.

A couple of minor comments:

genio commented 4 months ago
  1. Can we add links to the release notes? Yes. That's now done.
  2. Padding out releases might be a bit more difficult. I started with that in mind and changed tack when seeing how inconsistent things were.
  3. I don't think there's an issue. The ones listed on the old page don't distinguish between plain 32-bit vs 32-bit with 64-bit ints.
  4. Same issue as 3. They're all there, just in the more specific tab.

Should we combine the two 32-bit release types into one category (preference to one vs the other?)? Same question for the 64-bit release types.

mohawk2 commented 4 months ago

Can the release notes for each version be expanded to include which version of PDL is included?

shawnlaffan commented 4 months ago
  1. Can we add links to the release notes? Yes. That's now done.

Thanks.

2. Padding out releases might be a bit more difficult. I started with that in mind and changed tack when seeing how inconsistent things were.

This might not be necessary given the full set is now listed, although a marker that there is (currently) no 5.34 (32- or 64-bit) and no 32-bit 5.36 would be useful.

3. I don't think there's an issue. The ones listed on the old page don't distinguish between plain 32-bit vs 32-bit with 64-bit ints.

4. Same issue as `3`. They're all there, just in the more specific tab.

Should we combine the two 32-bit release types into one category (preference to one vs the other?)? Same question for the 64-bit release types.

I see now. Combining the 32-bit releases makes sense. Same for the 64-bit-ld releases given there are none after 5.26.

shawnlaffan commented 4 months ago

Can the release notes for each version be expanded to include which version of PDL is included?

Not easily. The site is built from the releases.json file, and this does not include such information.

The release notes html files list the installed modules, but these seem only to be done for the main installation.
https://github.com/StrawberryPerl/Perl-Dist-Strawberry/blob/39de5b0bb7362887239b4376b2773d17e57fa5c3/share/64bit-5.38.2.2.pp#L523C49-L523C67

I'll see if I can add an extra step after PDL is built in future releases

shawnlaffan commented 3 months ago

I think this issue is closable and will do so in a few days, or when next I do a sweep of the issues. If there are reasons to keep it open then please add a comment.

sisyphus commented 3 months ago

I still don't see any mention of a 32-bit 5.38.2 release at https://strawberryperl.com/releases.html . (Admittedly, I don't know whether that's a reason to keep this issue open.)

shawnlaffan commented 3 months ago

It does seem to be missing from the releases.json file. In that case we'll keep this open.

jsoref commented 3 months ago

@genio asked for 👍 / 👎 about https://github.com/StrawberryPerl/Perl-Dist-Strawberry/issues/177#issuecomment-2211973231

  1. The lack of a loading progress indicator made the page seem broken -- I was staring for a while and it didn't occur to me that I should wait longer -- the fix is to add a spinner (in each column) until the table is ready. image
  2. The page doesn't work for people who don't use JavaScript (that's actually me in my default browser configuration) -- we're a tiny minority, so don't feel obligated to cater to us -- a simple improvement would be to have a
image
  1. You might consider changing the formatting of the line below the gray header rows and the "No Releases" cells. It isn't obvious to me what they are -- thinking about it for a while, I'm guessing that that first row is a description of the column category -- it clearly isn't a thing I can download, but since it's left aligned and kinda in the same font as the things below it... -- I'd be inclined to horizontally center No Releases to make it even more distinguished from the rows that would otherwise appear in its place. image
  2. There should be some text above the four tabs that explains what this page is -- I understand the url won't be /foo.html eventually, but you can't rely on people seeing page titles as they're probably in tiny tabs. I'd suggest both a header and a paragraph... image