[Bug 474606] Review Request: SolarModel - Realtime 3D Solar System simulation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=474606 Tom "spot" Callaway <[EMAIL PROTECTED]> changed: What|Removed |Added Depends on||474603 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 474606] Review Request: SolarModel - Realtime 3D Solar System simulation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=474606 Lubomir Rintel <[EMAIL PROTECTED]> changed: What|Removed |Added Status|NEW |ASSIGNED CC||[EMAIL PROTECTED] AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 474606] Review Request: SolarModel - Realtime 3D Solar System simulation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=474606 Lubomir Rintel <[EMAIL PROTECTED]> changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #1 from Lubomir Rintel <[EMAIL PROTECTED]> 2008-12-04 17:58:46 EDT --- Well done! * SPEC file is clean, legible. RPMlint produces no errors and warnings. (I think "Realtime" (in Summary) is correctly spelled "Real-time" in American English, but I'm not sure since I'm not a native speaker.) * Built in fedora-5-i386-epel mock (irrlicht packages pulled in manually) successfully with after a tiny 32bit-specific fix applied (I assume x86_64 builds fine). Anyways, BRs are connect. To build this on i386 a small change is needed, since the length of a light year won't fit in a 32bit doubleword (sorry about the light breaks, but it's trivial to apply this by hand anyways): diff -up SolarModel_src/GameMeasurer.cpp.int SolarModel_src/GameMeasurer.cpp --- SolarModel_src/GameMeasurer.cpp.int 2008-12-04 21:03:05.0 +0100 +++ SolarModel_src/GameMeasurer.cpp 2008-12-04 21:11:32.0 +0100 @@ -313,7 +313,7 @@ core::stringw GameMeasurer::GetDistanceA distanceKM/9460730472581.0f); swprintf(s, (sizeof(s)/sizeof(s[0])), L"%.0lf km%s%s", - distanceKM, distanceKM>149598?au:L"", distanceKM>9460730473?ly:L""); + distanceKM, distanceKM>149598?au:L"", distanceKM>9460730473LL?ly:L""); } return core::stringw(s); It won't block the approval, since at the packages are required to build on at least one architecture, and I don't see why won't this build on x86_64. APPROVED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 474606] Review Request: SolarModel - Realtime 3D Solar System simulation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=474606 --- Comment #2 from Lubomir Rintel <[EMAIL PROTECTED]> 2008-12-05 09:49:25 EDT --- Also -- now I realize this doesn't create a Desktop Entry. Please install a .desktop file with this package. Thanks! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 474606] Review Request: SolarModel - Realtime 3D Solar System simulation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=474606 --- Comment #3 from Tom "spot" Callaway 2009-01-05 17:34:14 EDT --- Here is a new SRPM/SPEC: New SRPM: http://www.auroralinux.org/people/spot/review/new/SolarModel-2.1-2.fc11.src.rpm New SPEC: http://www.auroralinux.org/people/spot/review/new/SolarModel.spec Please note that this package depends on irrlicht (bz 474603). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 474606] Review Request: SolarModel - Realtime 3D Solar System simulation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=474606 Tom "spot" Callaway changed: What|Removed |Added Flag||fedora-cvs+ --- Comment #4 from Tom "spot" Callaway 2009-01-12 11:18:27 EDT --- New Package CVS Request === Package Name: SolarModel Short Description: Real-time 3D Solar System simulation Owners: spot Branches: F-9 F-10 devel InitialCC: ... and it's done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 474606] Review Request: SolarModel - Realtime 3D Solar System simulation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=474606 Bug 474606 depends on bug 474603, which changed state. Bug 474603 Summary: Review Request: irrlicht - A high performance realtime 3D engine https://bugzilla.redhat.com/show_bug.cgi?id=474603 What|Old Value |New Value Status|NEW |CLOSED Resolution||RAWHIDE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 474606] Review Request: SolarModel - Realtime 3D Solar System simulation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=474606 Tom "spot" Callaway changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE --- Comment #5 from Tom "spot" Callaway 2009-01-12 14:54:32 EDT --- Built for F-9, F-10, rawhide. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review