[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #87 from Fedora Update System --- speed-dreams-2.1.0-12.trunk_r4810.fc18 has been pushed to the Fedora 18 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=T9VaQMRcfI&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=785371 Fedora Update System changed: What|Removed |Added Resolution|ERRATA |CURRENTRELEASE -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=akK9w0H80a&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #86 from Fedora Update System --- speed-dreams-2.1.0-12.trunk_r4810.fc18 has been submitted as an update for Fedora 18. https://admin.fedoraproject.org/updates/speed-dreams-2.1.0-12.trunk_r4810.fc18 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=PvxwiImOWa&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 MartinKG changed: What|Removed |Added Version|17 |rawhide -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #85 from Fedora Update System --- speed-dreams-2.1.0-11.trunk_r4810.fc18 has been pushed to the Fedora 18 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #84 from Fedora Update System --- speed-dreams-2.1.0-11.trunk_r4810.fc18 has been submitted as an update for Fedora 18. https://admin.fedoraproject.org/updates/speed-dreams-2.1.0-11.trunk_r4810.fc18 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #83 from Fedora Update System --- speed-dreams-2.1.0-11.trunk_r4810.fc17 has been pushed to the Fedora 17 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed|2012-08-15 03:39:23 |2012-08-31 17:21:42 --- Comment #82 from Fedora Update System --- speed-dreams-2.1.0-11.trunk_r4810.fc16 has been pushed to the Fedora 16 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #81 from MartinKG --- pushed the package to stable, now this is the result: AutoQA: upgradepath test FAILED on noarch. [FAIL] f18 + f18-updates Latest package: speed-dreams-2.1.0-3.trunk_r4810.fc18 Latest pending package: None Error: Tested package must be less than or equal to the latest (pending) package. http://autoqa.fedoraproject.org/results/425420-autotest/virt04.qa/upgradepath/results/speed-dreams-2.1.0-1.html I have not a working speed-dreams-2.1.0-3.trunk_r4810.fc18 all woring packages are available on http://koji.fedoraproject.org/koji/packageinfo?packageID=14505 what to do ? -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #80 from Fedora Update System --- speed-dreams-2.1.0-11.trunk_r4810.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/speed-dreams-2.1.0-11.trunk_r4810.fc17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #79 from Fedora Update System --- speed-dreams-2.1.0-11.trunk_r4810.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/speed-dreams-2.1.0-11.trunk_r4810.fc16 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371
--- Comment #78 from MartinKG ---
(In reply to comment #77)
> Isnt a/the requirement to the data package (is there any yet?) missing?
yes a requirement to the robots-base package should be necessary.
if the package is installed, then you should see a message such as:
rpm -ivh speed-dreams-2.1.0-11.trunk_r4810.fc17.x86_64.rpm
error: Failed dependencies:
speed-dreams-robots-base = 2.1.0 is needed by
speed-dreams-2.1.0-11.trunk_r4810.fc17.x86_64
could be solved with the attached patch. Is the release tag necessary ?
( %{name}-robots-base = %{version}-release )
--- speed-dreams-old.spec2012-08-22 19:00:49.316607745 +0200
+++ speed-dreams.spec2012-08-22 19:00:26.949605799 +0200
@@ -36,6 +36,7 @@
Source5: %{name}.desktop
Provides: %{name} = %{version}-%{release}
+Requires: %{name}-robots-base = %{version}
Requires: opengl-games-utils
BuildRequires: cmake
BuildRequires: chrpath
--
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 Rudolf Kastl changed: What|Removed |Added CC||che...@gmail.com --- Comment #77 from Rudolf Kastl --- Isnt a/the requirement to the data package (is there any yet?) missing? -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #76 from MartinKG --- (In reply to comment #73) it's seems to be a AutoQA depcheck bug, discussed on: http://lists.fedoraproject.org/pipermail/devel/2012-August/171005.html AutoQA bug report: https://fedorahosted.org/fedora-infrastructure/ticket/3445 @Jeremy Can you please verify it, and close bug review as NEXTRELEASE again. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #75 from Fedora Update System --- speed-dreams-2.1.0-10.trunk_r4810.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/speed-dreams-2.1.0-10.trunk_r4810.fc16 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #74 from Fedora Update System --- speed-dreams-2.1.0-10.trunk_r4810.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/speed-dreams-2.1.0-10.trunk_r4810.fc17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #73 from Jeremy Newton --- (In reply to comment #72) > Sorry, I read this comment first today, because i hat problems with my email > account. There is no longer access to the version > speed-dreams-2.1.0-6.trunk_r4810.fc17 possible in Bohdi because I deleted > all previous versions. There are now only the versions > speed-dreams-2.1.0-9.trunk_r4810.fc17 and > speed-dreams-2.1.0-9.trunk_r4810.fc17 available. As long as no new issues have been introduced and the noarch for the sub package issue is fixed, this version should be fine. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #72 from MartinKG --- Sorry, I read this comment first today, because i hat problems with my email account. There is no longer access to the version speed-dreams-2.1.0-6.trunk_r4810.fc17 possible in Bohdi because I deleted all previous versions. There are now only the versions speed-dreams-2.1.0-9.trunk_r4810.fc17 and speed-dreams-2.1.0-9.trunk_r4810.fc17 available. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #71 from Fedora Update System --- speed-dreams-2.1.0-9.trunk_r4810.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/speed-dreams-2.1.0-9.trunk_r4810.fc16 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #70 from Fedora Update System --- speed-dreams-2.1.0-9.trunk_r4810.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/speed-dreams-2.1.0-9.trunk_r4810.fc17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|ON_QA --- Comment #69 from Fedora Update System --- Package speed-dreams-2.1.0-6.trunk_r4810.fc17: * should fix your issue, * was pushed to the Fedora 17 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing speed-dreams-2.1.0-6.trunk_r4810.fc17' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2012-12032/speed-dreams-2.1.0-6.trunk_r4810.fc17 then log in and leave karma (feedback). -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #68 from Fedora Update System --- speed-dreams-2.1.0-7.trunk_r4810.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/speed-dreams-2.1.0-7.trunk_r4810.fc17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #67 from Jeremy Newton --- (In reply to comment #66) > speed-dreams-robots-base needs to be noarch'ed. It contains only files under > /usr/share and therefore must not contain any arch-dependent files. > > Furthermore, as this package is amongst the biggest files (ca. 1G) in > Fedora, this package being arch'ed adds significant bloat to Fedora, which > noarch'ing it would at least somewhat dampen (In repositories on servers, > noarch packages can be hard-linked. I.e. the diskspace requirements for this > package be halved). Good point, I must have missed that in review, sorry. @MartinKG Can you please take note of this and fix it? -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 Ralf Corsepius changed: What|Removed |Added CC||rc040...@freenet.de --- Comment #66 from Ralf Corsepius --- speed-dreams-robots-base needs to be noarch'ed. It contains only files under /usr/share and therefore must not contain any arch-dependent files. Furthermore, as this package is amongst the biggest files (ca. 1G) in Fedora, this package being arch'ed adds significant bloat to Fedora, which noarch'ing it would at least somewhat dampen (In repositories on servers, noarch packages can be hard-linked. I.e. the diskspace requirements for this package be halved). -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #65 from Alec Leamas --- Well. Hate to be formal, but still: this is not the place to ask for help. And reopening the bug is a certainly Bad Idea. The bug request is for a review, and eventually to get an approval. See http://fedoraproject.org/wiki/Package_Review_Process. Since you need help, I would suggest a message to the mailing list. When you send thus message, it might also be a good idea to ask what to do with the bug, which now is in a even worse state than when I messed it up recently. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 MartinKG changed: What|Removed |Added Status|CLOSED |ASSIGNED Resolution|NEXTRELEASE |--- Keywords||Reopened --- Comment #64 from MartinKG --- sorry for opening bug review, Bohdi AutoQA: depcheck test FAILED on x86_64 for fc16 and fc17. fc16: http://autoqa.fedoraproject.org/report/bnxp fc17: http://autoqa.fedoraproject.org/report/bnzk i need help ! -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #63 from Fedora Update System --- speed-dreams-2.1.0-6.trunk_r4810.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/speed-dreams-2.1.0-6.trunk_r4810.fc17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #62 from Fedora Update System --- speed-dreams-2.1.0-6.trunk_r4810.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/speed-dreams-2.1.0-6.trunk_r4810.fc16 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #61 from Fedora Update System --- speed-dreams-2.1.0-5.trunk_r4810.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/speed-dreams-2.1.0-5.trunk_r4810.fc17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #60 from Fedora Update System --- speed-dreams-2.1.0-5.trunk_r4810.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/speed-dreams-2.1.0-5.trunk_r4810.fc16 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #59 from Fedora Update System --- speed-dreams-2.1.0-3.trunk_r4810.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/speed-dreams-2.1.0-3.trunk_r4810.fc17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #58 from Fedora Update System --- speed-dreams-2.1.0-3.trunk_r4810.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/speed-dreams-2.1.0-3.trunk_r4810.fc16 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 MartinKG changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |NEXTRELEASE Last Closed||2012-08-15 03:39:23 --- Comment #57 from MartinKG --- the package built successfully http://koji.fedoraproject.org/koji/packageinfo?packageID=14505 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 Jeremy Newton changed: What|Removed |Added Status|NEW |MODIFIED --- Comment #56 from Jeremy Newton --- Mistakes happen. Should be fixed now :) -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 Alec Leamas changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #55 from Alec Leamas --- I have no idea, but obviously I did some kind of mistake. However, it is indeed reviewed and approved. I'll talk to Jeremy, who is the one to reset the status and review flags. It's something with me and these flags. We're not compatible. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #54 from Jason Tibbitts --- Why did the status and flags get changed? I thought this was already reviewed and approved. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #53 from Jeremy Newton --- (In reply to comment #52) > Manpage typo ticket: > https://sourceforge.net/apps/trac/speed-dreams/ticket/739 Awesome thanks :) -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 Alec Leamas changed: What|Removed |Added Status|MODIFIED|NEW Flags|fedora-review+ |fedora-review? --- Comment #52 from Alec Leamas --- Manpage typo ticket: https://sourceforge.net/apps/trac/speed-dreams/ticket/739 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #51 from Jon Ciesla --- Git done (by process-git-requests). Added f18. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 MartinKG changed: What|Removed |Added Flags||fedora-cvs? -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #50 from MartinKG --- New Package SCM Request === Package Name: speed-dreams Short Description: speed-dreams - The Open Racing Car Simulator Owners: martinkg Branches: f16 f17 InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #49 from Alec Leamas --- Thanks for the review! Besides the effort it always is, the download sizes and related build times certainly made this review quite a task. Thanks again. Martin: congrats! -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 Jeremy Newton changed: What|Removed |Added Status|NEW |MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 Jeremy Newton changed: What|Removed |Added Flags|fedora-review? |fedora-review+ -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 Jeremy Newton changed: What|Removed |Added Attachment #602569|0 |1 is obsolete|| --- Comment #48 from Jeremy Newton --- Created attachment 604039 --> https://bugzilla.redhat.com/attachment.cgi?id=604039&action=edit Review Alec's Review template pretty much sums up my whole review, so much of it is a re-post, but I'll post it anyway. Looks good APPROVED -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #47 from MartinKG --- thanks for your review. %changelog * Mon Aug 13 2012 Martin Gansser 2.1.0-2.trunk_r4810 - removed extra docs folder because of file redundancy - leave comment, why removed docs folder Spec URL: http://dev.speed-dreams.org/Martin/speed-dreams-2.1.0-2.trunk_r4810.fc17/speed-dreams.spec -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 Jeremy Newton changed: What|Removed |Added Assignee|nob...@fedoraproject.org|alexjn...@hotmail.com -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #46 from Jeremy Newton --- Sorry I made a typo, the new %doc line should be something like this: %doc docs/*.xml docs/*.txt -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 Jeremy Newton changed: What|Removed |Added Flags||fedora-review? --- Comment #45 from Jeremy Newton --- It seems it builds just fine under Mock. Test cases, Fedora 16 32bit and Fedora 17 64bit were both successful. It seems I missed the rm -rf docs; mkdir docs part in the spec file, so please ignore my previous statements about documentation save for the rpmlint manpage warning. Everything seems good, though I still think the extra docs folder is sloppy; The extra docs folder should be removed, as it's completely unnecessary. This can be quickly fixed by changing the $doc line to: %doc *.xml *.txt Fix this minor issue and I SHOULD be able to approve this for you. Also it maybe a good idea to leave a comment above the rm -rf docs line stating why it needs to be removed (I would assume it's because the documentation is depreciated, which should be reported upstream at least) -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371
--- Comment #44 from Jeremy Newton ---
I would like to remark that the Manpage error from rpmlint should be fixed or
at least reported upstream, though this is not a blocking issue for getting
this accepted.
To underline the issues, it seems the following are blocking issues in order to
get this accepted in my eyes, assuming I haven't missed anything:
-The license needs to be included in %doc, by just adding COPYING.txt to %doc
-If I'm not mistaken, the man pages are being included with %doc because the
whole doc folder is being included. This seems sloppy and full of redundancy.
It would be better to do something like this:
%doc COPYING.txt doc/userman doc/faq
Note that the "Multiple Release tags found" error can be ignored; it's just
being caught up on the %(echo %{src_release} | tr '-' '_'), which is fine in my
eyes.
Also note that I'm having trouble building or downloading. I think it's
something on my side, but I'll post back here when I figure it out (if
relevant).
I'll report back if I see any issues after I get it to build.
--
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #43 from Alec Leamas --- Created attachment 602569 --> https://bugzilla.redhat.com/attachment.cgi?id=602569&action=edit fedora-review review template Since the fedora-review template raises some issues, I attach my own, private review. ATM, it's purpose is to sort out the issues in raised in comment #42. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #42 from MartinKG --- Package Review == Key: - = N/A x = Pass ! = Fail ? = Not evaluated C/C++ [x]: MUST Header files in -devel subpackage, if present. [x]: MUST Package does not contain any libtool archives (.la) [ ]: MUST Package does not contain kernel modules. [ ]: MUST Package contains no static executables. [x]: MUST Rpath absent or only used for internal libs. [ ]: MUST Development (unversioned) .so files in -devel subpackage, if present. Note: Unversioned so-files in non-devel package (fix or explain):speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/drivers/human/human.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/drivers/kilo2008/kilo2008.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/drivers/simplix/simplix.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/drivers/simplix_36GP/simplix_36GP.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/drivers/simplix_lp1/simplix_lp1.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/drivers/simplix_ls1/simplix_ls1.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/drivers/simplix_ls2/simplix_ls2.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/drivers/simplix_mp5/simplix_mp5.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/drivers/simplix_sc/simplix_sc.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/drivers/simplix_trb1/simplix_trb1.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/drivers/usr/usr.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/drivers/usr_36GP/usr_36GP.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/drivers/usr_lp1/usr_lp1.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/drivers/usr_ls1/usr_ls1.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/drivers/usr_ls2/usr_ls2.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/drivers/usr_rs/usr_rs.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/drivers/usr_sc/usr_sc.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/drivers/usr_trb1/usr_trb1.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/lib/liblearning.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/lib/libnetworking.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/lib/libraceengine.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/lib/librobottools.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/lib/libtgf.so speed-dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed-dreams-2/lib/libtgfclient.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/lib/libtgfdata.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/modules/graphic/ssggraph.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/modules/simu/simuv2.1.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/modules/simu/simuv2.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/modules/simu/simuv3.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/modules/telemetry/telemetry.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/modules/track/track.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/modules/userinterface/legacymenu.so speed- dreams-2.1.0-1.trunk_r4810.fc17.x86_64.rpm : /usr/lib64/games/speed- dreams-2/modules/userinterface/textonly.so Generic [x]: EXTRA Rpmlint is run on all installed packages. Note: There are rpmlint messages (see attachment). [x]: EXTRA Spec file according to URL is the same as in SRPM. [ ]: MUST Package is licensed with an open-source compatible license and meets other legal requirements as defined in the legal section of Packaging Guidelines. [
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 MartinKG changed: What|Removed |Added CC||alexjn...@hotmail.com --- Comment #41 from MartinKG --- SRPM URL: http://dev.speed-dreams.org/Martin/speed-dreams-2.1.0-1.trunk_r4810.fc17/speed-dreams-2.1.0-1.trunk_r4810.fc17.src.rpm Spec URL: http://dev.speed-dreams.org/Martin/speed-dreams-2.1.0-1.trunk_r4810.fc17/speed-dreams.spec %changelog * Thu Jul 26 2012 Alec Leamas 2.1.0-1.trunk_r4810 - Conditionalize spec on post-release or ordinary. - Updating to latest git. - Handle build system bugs (#728,#729,#731). - Fix 'identical binaries copied, not linked' in drivers(#730). - Simplified file list, use temporary docs dir, claim all dirs. - Sorted deps. - Fix Release: field. - Fix non-working handling of spurious-executable-perm (#605) - Added rpath for internal libraries. - Filter out internal libs in requires/provides. - Removed unneeded install fixes and macros. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 Bug 785371 depends on bug 787713, which changed state. Bug 787713 Summary: Review request: FreeSOLID - A 3D collision detection C++ library https://bugzilla.redhat.com/show_bug.cgi?id=787713 What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |NEXTRELEASE -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #40 from Alec Leamas --- PLease ignore comment #38 and #39 and their attachments. They were aimed for FreeSOLID (!) -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #39 from Alec Leamas --- Created attachment 597315 --> https://bugzilla.redhat.com/attachment.cgi?id=597315&action=edit freesolid.config.patch update Updated w parts of Ralf's patch -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #38 from Alec Leamas --- Created attachment 597314 --> https://bugzilla.redhat.com/attachment.cgi?id=597314&action=edit pkgconfig.patch update Updated with parts of Ralf's patch -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 Michael Schwendt changed: What|Removed |Added Blocks|177841 (FE-NEEDSPONSOR) | -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #37 from MartinKG --- Spec URL: http://dev.speed-dreams.org/Martin/speed-dreams.spec SRPM URL: http://dev.speed-dreams.org/Martin/speed-dreams-2.0.0-1.fc17.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 MartinKG changed: What|Removed |Added Version|16 |17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #36 from MartinKG --- Building SD2.0.0 binaries for Fedora 16/17/18 A description is now available on the speed dreams homepage under: http://dev.speed-dreams.org/Martin/index.html -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #35 from MartinKG --- Building SD2.0.0 binaries for Fedora 16/17/18 At the moment, only a unoffical srpm source package is available on my private homepage. mock build procedur to build rpm binaries for Fedora of speed-dreams-2.0.0: wget https://www.disk.dsl.o2online.de/FclyPlh/RPMS/speed-dreams/speed-dreams-2.0.0-1.fc17/speed-dreams-2.0.0-1.fc17.src.rpm?a=7bnlFo2Objc -O speed-dreams-2.0.0-1.fc17.src.rpm choose the command for your platform: mock -r fedora-16-i386--resultdir=/tmp/mock speed-dreams*src.rpm mock -r fedora-16-x86_64 --resultdir=/tmp/mock speed-dreams*src.rpm mock -r fedora-17-i386--resultdir=/tmp/mock speed-dreams*src.rpm mock -r fedora-17-x86_64 --resultdir=/tmp/mock speed-dreams*src.rpm mock -r fedora-18-i386--resultdir=/tmp/mock speed-dreams*src.rpm mock -r fedora-18-x86_64 --resultdir=/tmp/mock speed-dreams*src.rpm This requires: - A x86_64 machine. - rpmdevtools installed, and rpmdev-setuptree being run. - mock installed. - Some patience... -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #34 from MartinKG --- sorry im little bit confused, please ignore last post. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
https://bugzilla.redhat.com/show_bug.cgi?id=785371 --- Comment #33 from MartinKG --- Spec URL: https://www.disk.dsl.o2online.de/FclyPlh/RPMS/VDR/vdr-skinenigmang/0.1.2-4/vdr-skinenigmang.spec?a=eV_POBvkYYw SRPM URL: https://www.disk.dsl.o2online.de/FclyPlh/RPMS/VDR/vdr-skinenigmang/0.1.2-4/vdr-skinenigmang-0.1.2-4.fc17.src.rpm?a=c3c7_k3pkZI %changelog * Thu Jun 14 2012 Martin Gansser - 0.1.2-4 - more spec file cleanups - fixed copying of source file to build dir -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #32 from MartinKG 2012-05-09 15:03:15 EDT --- (In reply to comment #31) > Besides fedora packaging rules, are these rpm:s good enough to be exposed at > speed-dreams own website? What do you think? I think the rpms are good enough, but I will wait until 2.1 and all Fedora requirements are complied. By the way I have posted the spec file on http://forums.fedoraforum.org/showthread.php?t=247525&page=3 . -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #31 from Alec Leamas 2012-04-27 15:59:12 EDT --- Besides fedora packaging rules, are these rpm:s good enough to be exposed at speed-dreams own website? What do you think? -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #30 from MartinKG 2012-04-27 13:26:42 EDT --- Spec URL: https://www.disk.dsl.o2online.de/FclyPlh/RPMS/speed-dreams/rc2_r4687.21/speed-dreams.spec?a=fqTzUFodk4Y SRPM URL: https://www.disk.dsl.o2online.de/FclyPlh/RPMS/speed-dreams/rc2_r4687.21/speed-dreams-2.0.0-0.1.rc2_r4687.21.fc16.src.rpm?a=WGv8MVL1YIE -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #29 from Alec Leamas 2012-03-05 02:02:18 EST --- Basically it is (actually it's a fork). This has been discussed, and the overall understanding is to replace txml w expat after upcoming 2.0 release. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #28 from Volker Fröhlich 2012-03-04 20:08:28 EST --- Hm, what they call "txml" actually looks like Expat. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #27 from Alec Leamas 2012-02-29 16:54:23 EST --- Upstream has decided to not unbundle the txml library in the upcoming release. This fact blocks this bug ATM. Upstream reference: https://sourceforge.net/apps/trac/speed-dreams/ticket/634 -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #26 from MartinKG 2012-02-27 11:41:15 EST --- New Links: SRPM URL: https://www.disk.dsl.o2online.de/FclyPlh/RPMS/speed-dreams/rc1_r4420.19/speed-dreams-2.0.0-0.1.rc1_r4420.19.fc16.src.rpm?a=mVan601JeAA Spec URL: https://www.disk.dsl.o2online.de/FclyPlh/RPMS/speed-dreams/rc1_r4420.19/speed-dreams.spec?a=TVvfyQECgZM I need a sponsor for this package !!! -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371
--- Comment #24 from Alec Leamas 2012-02-08 11:58:24
EST ---
Sorry for last spam, let's hope somebody smiled. Just a little, but still.
I have made a quick check (*not* a review) of your spec file. I have attached a
diff of the changes I made. This diff *cannot* just be applied, but should
give you a hint. Feel free to ask if in doubt..
Some major issues:
- I think the %pkg_name macro adds nothing but confusion. Remove it, and rename
patches and sources to %{name} instead.
- It is not ok to add the non-standard paths to ld.so.conf.d, these unversioned
libs must be private. One solution might be to ask upstream to revert to use
rpath (it's OK here, although rmplint will warn.) Another way might be to add a
(edit existing?) wrapper script to use LD_LIBRARY_PATH. We need to settle on
this, but not until upstream have handled what they already have at hands from
us.
- Don't repeat the overall description in sub-packages.
- Remove unused macros.
- Order sources before patches, and in consecutive numbers.
Otherwise, see the patch. Look for TBD besides what's changed.
--a
--
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #25 from Alec Leamas 2012-02-08 11:59:55 EST --- Created attachment 560316 --> https://bugzilla.redhat.com/attachment.cgi?id=560316 Not-to-be applied spec file diff -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #23 from Alec Leamas 2012-02-07 13:20:51 EST --- -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #22 from Alec Leamas 2012-02-07 05:26:37 EST --- Created attachment 559905 --> https://bugzilla.redhat.com/attachment.cgi?id=559905 Fixes the wrong-fsf-address warnings, only comments changed. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #21 from Alec Leamas 2012-02-07 05:07:49 EST --- You forgot to increment the release tag when submitting spec + srpm. Please increment it each time you submit a new version, like in last free-solid.spec. You have a lot of errors and warnings. We need to apply a number of fixes to get it easier. - It seems that your spec file is not valid utf. See http://fedoraproject.org/wiki/Common_Rpmlint_issues#file-not-utf8 - You seem to have network issues. Is possible for you to run rpmlint with a working internet connection? - Description line(s) are too long. See http://fedoraproject.org/wiki/Common_Rpmlint_issues#description-line-too-long - You have a lot of unstripped-binary-or-object, see http://fedoraproject.org/wiki/Common_Rpmlint_issues#unstripped-binary-or-object - You need to be more careful when writing the changelog, and run rpmlint before submitting to avoid incoherent-version-in-changelog (the changelog entry should match %version-%release) - Fix the non-conffile-in-etc warning, see http://fedoraproject.org/wiki/Common_Rpmlint_issues#non-conffile-in-etc You also have some things to talk with upstream about: - speed-dreams.x86_64: W: shared-lib-calls-exit /usr/lib64/games/speed-dreams-2/lib/libtgfclient.so exit@GLIBC_2.2.5 - speed-dreams.x86_64: W: shared-lib-calls-exit /usr/lib64/games/speed-dreams-2/lib/libtgf.so exit@GLIBC_2.2.5 - I think I can provide you a patch for the incorrect-fsf-address, to send to upstream. An updated COPYING is at http://www.gnu.org/licenses/gpl-2.0.txt - A lot of empty readme.txt, ask what to do (remove in distribution?). - The missing manpage warnings. This is not a blocker for Fedora, but for Debian/Ubuntu. So some simple manpages might help. - Multiple invalid-soname errors. See e. g., https://bugzilla.redhat.com/show_bug.cgi?format=multiple&id=176617. In general, it seems like a bad idea to have non-versioned names, but it might possibly be acceptable if the libraries are outside of ld.so's search path (?). -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #20 from MartinKG 2012-02-06 13:58:27 EST --- speed-dreams-rpmlint-output: https://www.disk.dsl.o2online.de/FclyPlh/RPMS/speed-dreams/speed-dreams-rpmlint-output.txt?a=8NX5xTacCfI -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 MartinKG changed: What|Removed |Added Depends on||787713 -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #19 from MartinKG 2012-02-06 10:50:28 EST --- New Links: SRPM URL: https://www.disk.dsl.o2online.de/FclyPlh/RPMS/speed-dreams/speed-dreams-2.0.0-0.1.rc1_r4420.12.fc16.src.rpm?a=UAl5-rFrWFI Spec URL: https://www.disk.dsl.o2online.de/FclyPlh/RPMS/speed-dreams/speed-dreams.spec?a=3aybLWZuFFI -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #18 from Alec Leamas 2012-02-04 05:56:08 EST --- There are three issues: - The txml lib is the important one. Unless this is fixed, it will be hard. - The tgf.h header is (to my understanding) no issue at all. - The SOLID lib is an issue . However, to patch cmake to detect and use a possible system variant should not be hard (adding a pkgconfig .pc file to solid would help, I guess). This could be done upstream or as a packaging post-release patch. Since the txml issue is not just a policy but also a legal issue, it should have the highest priority. Not fixing this will block all kind of legal binary distribution. Is such a release meaningful? Bottom line: if txml is fixed, I think a packaging might be possible. Of course, this is besides whatever other problems which might occur. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #17 from MartinKG 2012-02-03 15:37:08 EST --- Here's another answer from Simon of the SD user forum: >> The only thing I can say is that we can setup a plan for getting rid >> of these 3 bundled libraries : technically speaking, it'll be some work, >> but no major blocker AFAICS. >> >> But for 2.0, too late for fixing the all 3 ones. Definitely. >> > But this is very sad > Given that we're in RC then I would agree that it seems too much of a hurdle to try to strip out the bundled libraries before releasing SD2.0. However: 1). This does not mean that someone can't provide RPMs for Fedora, they just will not be accepted into the Fedora repo. Does Fedora have the equivilant of PPA's? 2). It might also be possible for someone to do the work based on SD2.0, but have a patch to remove the libraries and use this as a basis for a PRM in the Fedora repos. You have to remember that the SD dev team is very small, only a few people working on the coding side. If we take on too much, we'll never get SD2.0 out the door... -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #16 from Alec Leamas 2012-02-03 06:01:24 EST --- I was to quick. From the list discussion, there are one or two more bundled libs. The src/modules/simu/simuv2.1/SOLID-2.0 lib is clear, it should be packaged separately. Since it doesn't exist in Fedora today, I suggest that you make a new request for this package, blocking this one. Note that qhull, which SOLID possibly depends on, exists in Fedora (v 2003.1). In my eyes, the pasted part of a header file in src/libs/tgf/tgf.h is not a bundled library. BTW, the upstream discussion describes this as "Fedora's law". But it's also Debian's law, see http://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries (links at at the very end). Since repackaging the SOLID-2 library should mean exactly the same code as today, besides build system patches, maybe something is possible? Note that the library can still be part of the source, just the system variant is used if it exists. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #15 from Alec Leamas 2012-02-03 04:31:06 EST --- OK, there are open issues about txml lib, fix under way from upstream. There seems to be no other bundled libs(?). Next step is that you run rpmlint on the spec file and binary rpm, and show the output here (possibly after fixes, see http://fedoraproject.org/wiki/Common_Rpmlint_issues). -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #14 from Alec Leamas 2012-02-02 14:57:30 EST --- hm... maybe I'm growing up, who knows? About time,then. If the definition of a bundled library is "something which exists and is maintained separately", I don't think there's a need to check these libs. They seem to be TORCS sw only. However, if anyone has any more input on this it would be great to have it now instead of later,I guess. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #13 from Alec Leamas 2012-02-02 13:41:11 EST --- I think we need help from some of the grown-ups. There are several libraries, some of which can/should/might be considered bundled. To simplify, I have made the libs available at http://dl.dropbox.com/u/17870887/speed-dreams-libs.tar.gz. After some checking I don't think these other libs exist in Fedora today. However, they might be considered bundled anyway. So: have someone time to judge what libs (if any) in link above besides txml should be considered bundled, and thus should be packaged separately? -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #12 from Alec Leamas 2012-02-02 13:04:39 EST --- So, this library is just the expat library. This means that this is not only a license issue, but also an example of a bundled library. Basically,bundled libraries are not allowed in Fedora, see http://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries. From the upstream discussion, it looks that they will try to use the system expat library instead for next version. Basically, I think this is the only feasible solution within reach. However,it also raises the question if there are any more bundled libraries, GPLv2 or not. Are you aware of any? -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #11 from MartinKG 2012-02-02 12:35:14 EST --- Hi Alec, there is at the moment a discussion at the speed-dreams user forum regarding the license issue http://sourceforge.net/mailarchive/forum.php?thread_name=1d35033a06a2a6cf4cbe6ecac97050d5.squirrel%40mungewell.org&forum_name=speed-dreams-users I hope there will be a solution before 2.0.0 finale version of speed-dreams. Martin -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #10 from Alec Leamas 2012-02-01 14:42:40 EST --- I'm afraid it's not that simple. But first, thanks for fixing the framework: URL:s, heading, release number etc. Seems we're up & running. But the licensing is trickier. If you look into the link above, there's compatibility matrix. From there, you can find out that the Mozilla license (MPL1.0) is incompatible with GPLv2. Practically, this means that a binary RPM can't be made from both MPL1.0 and GPLv2 sources (that they exist together in the srpm is no problem). So, be prepared that this is no minor issue. I guess you need to look into that link again to get the details... Now, what's your options? I'm by no means a specialist, and I have not looked into the code at all. Let's hope someone of the more experienced people corrects me if I'm wrong. My understanding: - You can't have a License: tag saying "GPLv2 and MPL1.0", they are incompatible. - In some cases one can just remove the offending code (that's what happened yesterday in another review). - In other cases it's possible to package it in a subpackage (which can have a license of it's own). - You could talk to the MPL code upstream to see if they can relicense with a dual license. BTW, the situation implies that you should talk to upstream, since this is a problem for anyone trying to package it. What's their opinion? Hope this helps... (one could also hope I'm wrong, that there's a simple solution) -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 --- Comment #9 from MartinKG 2012-02-01 13:42:30 EST --- Thanks Alec, a new modified speed-dreams.spec file is available: https://www.disk.dsl.o2online.de/FclyPlh/RPMS/speed-dreams.spec?a=XkVakSJVH6o -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785371] Review request: speed-dreams - The Open Racing Car Simulator
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=785371 MartinKG changed: What|Removed |Added Summary|Review Request: |Review request: |speed-dreams - a fork of|speed-dreams - The Open |TORCS |Racing Car Simulator -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
