[Bug 474606] Review Request: SolarModel - Realtime 3D Solar System simulation

2009-01-12 Thread bugzilla
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 tcall...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs+




--- Comment #4 from Tom spot Callaway tcall...@redhat.com  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

2009-01-12 Thread bugzilla
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

2009-01-12 Thread bugzilla
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 tcall...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Comment #5 from Tom spot Callaway tcall...@redhat.com  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


[Bug 474606] Review Request: SolarModel - Realtime 3D Solar System simulation

2009-01-05 Thread bugzilla
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 tcall...@redhat.com  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

2008-12-05 Thread bugzilla
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

2008-12-04 Thread bugzilla
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

2008-12-04 Thread bugzilla
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

2008-12-04 Thread bugzilla
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, distanceKM149598?au:L,
distanceKM9460730473?ly:L);
+   distanceKM, distanceKM149598?au:L,
distanceKM9460730473LL?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