Bug 433990 - dosbox failed massrebuild attempt for GCC 4.3
dosbox failed massrebuild attempt for GCC 4.3
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: dosbox (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Andreas Bierfert
Fedora Extras Quality Assurance
:
Depends On:
Blocks: gcc43errors
  Show dependency treegraph
 
Reported: 2008-02-22 07:53 EST by Jesse Keating
Modified: 2013-01-09 21:47 EST (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2008-02-22 11:05:35 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:


Attachments (Terms of Use)
Patch to fix gcc 4.3 compilation problems (1.29 KB, patch)
2008-02-22 09:19 EST, Erik van Pienbroek
no flags Details | Diff

  None (edit)
Description Jesse Keating 2008-02-22 07:53:19 EST
This is an automatically filed bug for a failed rebuild attempt for GCC 4.3.

http://fedoraproject.org/wiki/JesseKeating/gcc43MassRebuildProposal

Please verify why this build failed and fix it.
http://koji.fedoraproject.org/koji/taskinfo?taskID=436501
Exit code was 1, check the build.log for the failed buildArch task.
Comment 1 Tyler Owen 2008-02-22 09:15:23 EST
Errors in i386 build.log:

gameblaster.cpp: In function 'void saa1099_update(int, INT16**, int)':
gameblaster.cpp:195: error: 'memset' was not declared in this scope
gameblaster.cpp: In constructor 'CMS::CMS(Section*)':
gameblaster.cpp:443: error: 'memset' was not declared in this scope
Comment 2 Erik van Pienbroek 2008-02-22 09:19:38 EST
Created attachment 295627 [details]
Patch to fix gcc 4.3 compilation problems

Hi,

Attached patch fixes all the GCC 4.3 compilation problems. I've also noticed in
the build logs that you forgot to add a BuildRequires: SDL_sound-devel to your
spec file (the configure check for this fails)
Comment 3 Andreas Bierfert 2008-02-22 11:05:35 EST
Thanks for the patch. Help is appreciated. Also fixed the missing BR.

Note You need to log in before you can comment on or make changes to this bug.