Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: sugar-view-slides - Image serie viewer for Sugar
https://bugzilla.redhat.com/show_bug.cgi?id=508441
Summary: Review Request: sugar-view-slides - Image serie viewer for Sugar Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: fabian@bernewireless.net QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://fab.fedorapeople.org/packages/XO/sugar-view-slides.spec SRPM URL: http://fab.fedorapeople.org/packages/XO/sugar-view-slides-8-1.fc11.src.rpm
Project URL: http://wiki.laptop.org/go/View_Slides
Description: The View Slides activity is meant to allow the XO laptop to read view the contents of a Zip file containing images named sequentially. Project Gutenberg has a few books as raw scanned images, and this can be a useful format for picture books, comic books, magazine articles, photo essays, etc.
The interface to View Slides is similar to the core Read activity, which should not be surprising as the toolbar code was adapted from Read's toolbar. You can use the up and down arrows or the game controller to move from page to page.
Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1438374
rpmlint output: [fab@laptop09 noarch]$ rpmlint sugar-view-slides-8-1.fc11.noarch.rpm sugar-view-slides.noarch: W: non-standard-group Sugar/Activities 1 packages and 0 specfiles checked; 0 errors, 1 warnings.
[fab@laptop09 SRPMS]$ rpmlint sugar-view-slides-8-1.fc11.src.rpm sugar-view-slides.src: W: non-standard-group Sugar/Activities 1 packages and 0 specfiles checked; 0 errors, 1 warnings.
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=508441
Lubomir Rintel lkundrak@v3.sk changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |lkundrak@v3.sk AssignedTo|nobody@fedoraproject.org |lkundrak@v3.sk Flag| |fedora-review?
--- Comment #1 from Lubomir Rintel lkundrak@v3.sk 2009-06-27 06:38:17 EDT --- * package named in accordance with license guidelines * builds fine in mock * source file ok * file lists sane * provides make sense
1.) pygame seems to be needed, but missing from Requires:
/usr/share/sugar/activities/ViewSlides.activity/viewslides.py:import pygame, pygame.display /usr/share/sugar/activities/ViewSlides.activity/viewslides.py:from pygame.locals import *
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=508441
Peter Robinson pbrobinson@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |462625(FedoraOLPCDelta)
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=508441
--- Comment #2 from Lubomir Rintel lkundrak@v3.sk 2009-07-07 05:43:01 EDT --- ping?
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=508441
--- Comment #3 from Fabian Affolter fabian@bernewireless.net 2009-07-30 03:48:45 EDT --- Sorry, thanks for your help and your patience. I will postpone this for two weeks because I'm busy with other stuff.
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=508441
--- Comment #4 from Fabian Affolter fabian@bernewireless.net 2009-08-01 11:24:26 EDT --- * Sat Aug 01 2009 Fabian Affolter fabian@bernewireless.net - 8-2 - Added pygame as a requirement (as mentioned in #508441)
Updated files:
Spec URL: http://fab.fedorapeople.org/packages/XO/sugar-view-slides.spec SRPM URL: http://fab.fedorapeople.org/packages/XO/sugar-view-slides-8-2.fc11.src.rpm
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=508441
Sebastian Dziallas sebastian@when.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |sebastian@when.com Flag| |needinfo?
--- Comment #5 from Sebastian Dziallas sebastian@when.com 2010-01-22 09:26:25 EST --- Ping, what's the status here? Lubomir, do you still want to review this or should I?
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=508441
Peter Robinson pbrobinson@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |558617(SOAS-3)
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=508441
Sebastian Dziallas sebastian@when.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|needinfo? |needinfo?(lkundrak@v3.sk)
--- Comment #6 from Sebastian Dziallas sebastian@when.com 2010-02-18 14:11:29 EST --- Ping? I won't be able to take it on this weekend anyway since I'll be travelling. Probably the weekend after, though.
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=508441
Lubomir Rintel lkundrak@v3.sk changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review?, |fedora-review+ |needinfo?(lkundrak@v3.sk) |
--- Comment #7 from Lubomir Rintel lkundrak@v3.sk 2010-02-18 14:27:22 EST --- Sorry for the delay. I'm happy with this now.
APPROVED
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=508441
Fabian Affolter fabian@bernewireless.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #8 from Fabian Affolter fabian@bernewireless.net 2010-02-20 09:34:59 EST --- New Package CVS Request ======================= Package Name: sugar-view-slides Short Description: Image serie viewer for Sugar Owners: fab Branches: F-11 F-12 F-13 InitialCC:
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=508441
--- Comment #9 from Jason Tibbitts tibbs@math.uh.edu 2010-02-20 13:01:52 EST --- CVS done (by process-cvs-requests.py).
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=508441
--- Comment #10 from Sebastian Dziallas sebastian@when.com 2010-03-04 15:24:00 EST --- Ping?
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=508441
--- Comment #11 from Fabian Affolter fabian@bernewireless.net 2010-03-16 14:05:04 EDT --- At the moment I'm still not able to build this package.
https://koji.fedoraproject.org/koji/packageinfo?packageID=9978
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=508441
--- Comment #12 from Fabian Affolter fabian@bernewireless.net 2010-03-16 14:26:30 EDT --- Sebastian, maybe you can check the spec file for meanderings from the sugar activities template. I can find any...
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=508441
--- Comment #13 from Sebastian Dziallas sebastian@when.com 2010-03-16 18:03:28 EDT --- Uh, sorry. I didn't notice that it was in fact failing. What it misses is the gettext build requirement. The build script should fail in a much nicer way and I think there used to be a ticket, but I guess that is what is preventing it from building right now.
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=508441
--- Comment #14 from Fabian Affolter fabian@bernewireless.net 2010-03-16 19:19:35 EDT --- Thanks for the hint.
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=508441
--- Comment #15 from Fedora Update System updates@fedoraproject.org 2010-03-16 19:25:06 EDT --- sugar-view-slides-8-4.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/sugar-view-slides-8-4.fc12
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=508441
--- Comment #16 from Fedora Update System updates@fedoraproject.org 2010-03-16 19:25:11 EDT --- sugar-view-slides-8-4.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/sugar-view-slides-8-4.fc13
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=508441
--- Comment #17 from Fedora Update System updates@fedoraproject.org 2010-03-16 19:25:15 EDT --- sugar-view-slides-8-4.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/sugar-view-slides-8-4.fc11
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=508441
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |ON_QA
--- Comment #18 from Fedora Update System updates@fedoraproject.org 2010-03-17 23:17:41 EDT --- sugar-view-slides-8-4.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update sugar-view-slides'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/sugar-view-slides-8-4.fc13
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=508441
--- Comment #19 from Fedora Update System updates@fedoraproject.org 2010-03-17 23:22:04 EDT --- sugar-view-slides-8-4.fc12 has been pushed to the Fedora 12 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update sugar-view-slides'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/sugar-view-slides-8-4.fc12
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=508441
--- Comment #20 from Fedora Update System updates@fedoraproject.org 2010-03-17 23:33:47 EDT --- sugar-view-slides-8-4.fc11 has been pushed to the Fedora 11 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update sugar-view-slides'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/sugar-view-slides-8-4.fc11
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=508441
--- Comment #21 from Fedora Update System updates@fedoraproject.org 2010-03-29 22:18:42 EDT --- sugar-view-slides-8-4.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report.
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=508441
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Fixed In Version| |sugar-view-slides-8-4.fc11 Resolution| |ERRATA
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=508441
--- Comment #22 from Fedora Update System updates@fedoraproject.org 2010-03-29 22:19:43 EDT --- sugar-view-slides-8-4.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report.
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=508441
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|sugar-view-slides-8-4.fc11 |sugar-view-slides-8-4.fc12
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=508441
--- Comment #23 from Fedora Update System updates@fedoraproject.org 2010-04-09 00:23:37 EDT --- sugar-view-slides-8-4.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report.
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=508441
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|sugar-view-slides-8-4.fc12 |sugar-view-slides-8-4.fc13
package-review@lists.fedoraproject.org