Opened 15 years ago
Closed 15 years ago
#6900 closed defect (fixed)
win32-packager fixes - SDL ( and QT )
| Reported by: | Owned by: | markk | |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.22 |
| Component: | Ports - Windows | Version: | head |
| Severity: | medium | Keywords: | win32 |
| Cc: | Ticket locked: | no |
Description
SDL default Makefile from tar file is broken under mingw. A redo of ./configure fixes it.
SDL no longer has a target of 'make install-sdl' it's just 'make install'
...also... QT software takes forever to download, and doesn't warn. it appears hung.
Attachments (2)
Change History (8)
Changed 15 years ago by
| Attachment: | SDL-6900.patch added |
|---|
comment:1 Changed 15 years ago by
| Owner: | changed from Nigel to markk |
|---|---|
| Status: | new → accepted |
comment:2 Changed 15 years ago by
Buzz, Did 'make install-sdl' fail while using the script, or were you doing this manually? I ran the existing script today, and 'make install-sdl' worked fine. Perhaps you were using the source tarball instead of the binaries? The script is downloading the binaries (SDL-devel-1.2.13-mingw32.tar.gz), which doesn't include the top-level configure that your patch calls for.
comment:3 Changed 15 years ago by
the script fails for me when trying to install SDL ( assume binaries ), so I manually attempted install of them ( I assume I accidentially used the source version? ).
So, I guess this invalidates the SDL part of my patch, but doesn't fix the problem, which was that the script aborted when the SDL-related code failed to behave as expected. Buzz.
comment:4 Changed 15 years ago by
The script works on a clean machine, so probably you have a source copy of SDL leftover from an older version of the script. Both binaries and source tarballs extract into sources/SDL-1.2.13, and the script is only checking the existence of the directory. To cover this upgrade scenario, the script could be pointed instead to some unique file in the binary tarball.
Changed 15 years ago by
| Attachment: | 6900-w32pkg.patch added |
|---|
Revised patch: change SDL extract condition, add QT warning, and stream downloads to disk
comment:5 Changed 15 years ago by
Revised patch attached. Changes: make SDL extract conditional on SDL.dll, and stream downloads to disk (see below). Also catch SIGINT to allow cleanup if script is aborted during a download.
It turns out the QT download was taking /forever/ due to Perl being inefficient at handling the large file in memory. I've replaced the fetch code with a version that streams the file to disk rather than holding it in memory. With this version, the download should run at full speed, and Perl won't consume CPU/memory as it was.
win32 SDL fixes.