Bug 233142 - Review Request: asc - Advanced Strategic Command
Summary: Review Request: asc - Advanced Strategic Command
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Gwyn Ciesla
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On: SDLmm 233140
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-03-20 16:47 UTC by Hans de Goede
Modified: 2008-04-14 16:05 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-04-06 08:48:24 UTC
Type: ---
Embargoed:
gwync: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Hans de Goede 2007-03-20 16:47:55 UTC
Spec URL: http://people.atrpms.net/~hdegoede/asc.spec
SRPM URL: http://people.atrpms.net/~hdegoede/asc-1.16.4.0-1.fc7.src.rpm
Description:
ASC is a free, turn based strategy game.

Comment 1 Hans de Goede 2007-03-20 16:50:43 UTC
Note this requires SDLmm (review bug 233139) and paragui (review bug 233140)

Comment 2 Hans de Goede 2007-03-21 10:35:08 UTC
New version:
* Wed Mar 21 2007 Hans de Goede <j.w.r.degoede> 1.16.4.0-2
- Use system verion of SDLmm, rand_r instead of own private copy

Spec URL: http://people.atrpms.net/~hdegoede/asc.spec
SRPM URL: http://people.atrpms.net/~hdegoede/asc-1.16.4.0-2.fc7.src.rpm


Comment 3 Gwyn Ciesla 2007-04-02 17:36:57 UTC
Building on FC-6:
../../edmisc.cpp:4564: internal compiler error: Segmentation fault
Please submit a full bug report,
with preprocessed source if appropriate.
See <URL:http://bugzilla.redhat.com/bugzilla> for instructions.
The bug is not reproducible, so it is likely a hardware or OS problem.
make[4]: *** [edmisc.o] Error 1
make[4]: Leaving directory
`/usr/src/redhat/BUILD/asc-1.16.4.0/source/unix/mapeditor'
make[3]: *** [all-recursive] Error 1
make[3]: Leaving directory `/usr/src/redhat/BUILD/asc-1.16.4.0/source/unix'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/usr/src/redhat/BUILD/asc-1.16.4.0/source'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/usr/src/redhat/BUILD/asc-1.16.4.0'
make: *** [all] Error 2
error: Bad exit status from /var/tmp/rpm-tmp.57624 (%build)


RPM build errors:
    Bad exit status from /var/tmp/rpm-tmp.57624 (%build)


Comment 4 Gwyn Ciesla 2007-04-02 18:35:55 UTC
Second attempt.

../../objecttype.cpp:933: warning: unused variable 'copycount'
g++: Internal error: Segmentation fault (program cc1plus)
Please submit a full bug report.
See <URL:http://bugzilla.redhat.com/bugzilla> for instructions.
make[4]: *** [objecttype.o] Error 1
make[4]: Leaving directory `/usr/src/redhat/BUILD/asc-1.16.4.0/source/tools/linux'
make[3]: *** [all-recursive] Error 1
make[3]: Leaving directory `/usr/src/redhat/BUILD/asc-1.16.4.0/source/tools'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/usr/src/redhat/BUILD/asc-1.16.4.0/source'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/usr/src/redhat/BUILD/asc-1.16.4.0'
make: *** [all] Error 2
error: Bad exit status from /var/tmp/rpm-tmp.21812 (%build)


RPM build errors:
    Bad exit status from /var/tmp/rpm-tmp.21812 (%build)


Comment 5 Gwyn Ciesla 2007-04-02 19:07:51 UTC
Vanilla source builds fine.. . .trying the rpm sans patch. . .

Comment 6 Gwyn Ciesla 2007-04-02 19:23:39 UTC
Nah, that's worse.  What's going on? I assume it's a problem in the sed scripts,
but my sed-foo is only a tiny blossom.

Comment 7 Hans de Goede 2007-04-02 19:39:43 UTC
Erm, I don't want to be negative, but gcc segfaulting instead of giving an
error, and especially that happening twice with different source files, is an
(serious) indication of RAM / CPU problems. Are you running devel / Fedora 7?
Maybe there is an actual bug in gcc there, try upgrading. But chances are you're
hardware is not 100% I know it may seem to be working fine, both lamost nothing
hits / stresses the hardware as much as gcc, it could for example also be a
cooling problem.


Comment 8 Gwyn Ciesla 2007-04-02 19:52:45 UTC
I'll try it on a different box.  I've been building on this box for awhile, no
major issues so far.  I'll see what happens.

Comment 9 Gwyn Ciesla 2007-04-03 11:41:58 UTC
Ah, that worked.  Crud, now I have to build a new dev box.  Oh well.  Anyway, it
builds, rpmlint is clean.  TBC. . .

Comment 10 Gwyn Ciesla 2007-04-03 12:23:42 UTC
Named accoring to naming Guidelines.
Spec name good.
Meets PackagingGuidelines.
License is good.
Spec is legible American English.


Comment 11 Gwyn Ciesla 2007-04-03 12:56:54 UTC
Source MD5 matches.
Builds on 1386.
BRs are OK.
Locales N/A.
No ldconfig needed.
Not relocatable.
Ownership OK.
No dupes.
Perms OK.
Clean present.


Comment 12 Gwyn Ciesla 2007-04-03 13:04:31 UTC
Macro use OK.
Code, not content.
No large docs.
No runtime doc deps.
No header or static files.
No libraries.
No .pc files.
Desktop file is good, handled properly.
All MUSTS good.

Comment 13 Gwyn Ciesla 2007-04-03 13:53:19 UTC
Builds in mock.
SHOULDS look fine.
APPROVED.

Comment 14 Hans de Goede 2007-04-03 14:21:32 UTC
New Package CVS Request
=======================
Package Name:      acs
Short Description: Advanced Strategic Command
Owners:            j.w.r.degoede
Branches:          FC-6 devel
InitialCC:         <empty>



Comment 15 Hans de Goede 2007-04-03 14:22:37 UTC
OOPS make that:

New Package CVS Request
=======================
Package Name:      asc
Short Description: Advanced Strategic Command
Owners:            j.w.r.degoede
Branches:          FC-6 devel
InitialCC:         <empty>


Comment 16 Jens Petersen 2007-04-06 06:31:53 UTC
done

Comment 17 Hans de Goede 2007-04-06 08:48:24 UTC
Thanks for the review!

Imported and build, closing.


Comment 18 Hans de Goede 2008-04-14 12:37:02 UTC
Early asc F-10 branch request:

Package Change Request
======================
Package Name: asc
New Branches: F-9



Comment 19 Kevin Fenzi 2008-04-14 16:05:27 UTC
cvs done.


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