Spec URL: http://fab.fedorapeople.org/packages/XO/sugar-speak.spec SRPM URL: http://fab.fedorapeople.org/packages/XO/sugar-speak-9-1.fc10.src.rpm Description: Speak is a talking face for the XO laptop. Anything you type will be spoken aloud using the XO's speech synthesizer, espeak. You can adjust the accent, rate and pitch of the voice as well as the shape of the eyes and mouth. This is a great way to experiment with the speech synthesizer, learn to type or just have fun making a funny face for your XO. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=889310 [fab@localhost SRPMS]$ rpmlint -i sugar-speak-9-1.fc10.src.rpm sugar-speak.src: W: non-standard-group Sugar/Activities The value of the Group tag in the package is not valid. Valid groups are: "Amusements/Games", "Amusements/Graphics", "Applications/Archiving", "Applications/Communications", "Applications/Databases", "Applications/Editors", "Applications/Emulators", "Applications/Engineering", "Applications/File", "Applications/Internet", "Applications/Multimedia", "Applications/Productivity", "Applications/Publishing", "Applications/System", "Applications/Text", "Development/Debug", "Development/Debuggers", "Development/Languages", "Development/Libraries", "Development/System", "Development/Tools", "Documentation", "System Environment/Base", "System Environment/Daemons", "System Environment/Kernel", "System Environment/Libraries", "System Environment/Shells", "User Interface/Desktops", "User Interface/X", "User Interface/X Hardware Support". 1 packages and 0 specfiles checked; 0 errors, 1 warnings. [fab@localhost noarch]$ rpmlint -i sugar-speak-9-1.fc10.noarch.rpm sugar-speak.noarch: W: non-standard-group Sugar/Activities The value of the Group tag in the package is not valid. Valid groups are: "Amusements/Games", "Amusements/Graphics", "Applications/Archiving", "Applications/Communications", "Applications/Databases", "Applications/Editors", "Applications/Emulators", "Applications/Engineering", "Applications/File", "Applications/Internet", "Applications/Multimedia", "Applications/Productivity", "Applications/Publishing", "Applications/System", "Applications/Text", "Development/Debug", "Development/Debuggers", "Development/Languages", "Development/Libraries", "Development/System", "Development/Tools", "Documentation", "System Environment/Base", "System Environment/Daemons", "System Environment/Kernel", "System Environment/Libraries", "System Environment/Shells", "User Interface/Desktops", "User Interface/X", "User Interface/X Hardware Support". 1 packages and 0 specfiles checked; 0 errors, 1 warnings.
See items below: OK - Package name XX - License info is accurate License is GPLv3 OK - License tag is correct and licenses are approved OK - License files are installed as %doc OK - Specfile name OK - Specfile is legible OK - No prebuilt binaries included OK - BuildRoot value (one of the recommended values) OK - PreReq not used ?? - Source md5sum matches upstream - Barring a "release" from upstream, can you export the source code usig the fixed snapshot url (http://dev.laptop.org/git?p=activities/speak;a=snapshot;h=f76cea50b0fdee97e92ce94ac379155d6f8d52ac) OK - No hardcoded pathnames OK - Package owns all the files it installs OK - 'Requires' create needed unowned directories OK - BuildRequires sufficient OK - File permissions set properly OK - Macro usage is consistent ?? - rpmlint is silent - rpmlint on the srpm shows this (the last 3 are worriesome) sugar-maze.src: W: non-standard-group Sugar/Activities sugar-maze.src: W: strange-permission sugar-maze-checkout.sh 0755 sugar-maze.src: W: strange-permission Maze-6.tar.bz2 0755 sugar-maze.src: W: strange-permission sugar-maze.spec 0755
(In reply to comment #1) > sugar-maze.src: W: non-standard-group Sugar/Activities Copy-and-paste error? ;-) Anyway, the permission of the file from comment #1 are fixed
Can you re-post the spec file and srcrpm with the new licens info. If I can see those I can approve (Speak is GPLv3)
Sorry, I didn't upload a new version. After a new check of the license. The license is GPLv3+. COPYING says GPLv3, the file headers says "either version 3 of the License, or (at your option) any later version" Spec URL: http://fab.fedorapeople.org/packages/XO/sugar-speak.spec SRPM URL: http://fab.fedorapeople.org/packages/XO/sugar-speak-9-2.fc10.src.rpm
Bryan, don't go ahead with this review. For this package 'mkdir -p %{buildroot}/%{sugaractivitydir}' is perhaps not necessary, too.
Looks like you need to add requires for the following pakcages: numpy espeak Add those, and I am fine. I honestly am fine with the extra mkdir. Post the new requires and I will approve it.
Thanks Bryan Here are the new files: Spec URL: http://fab.fedorapeople.org/packages/XO/sugar-speak.spec SRPM URL: http://fab.fedorapeople.org/packages/XO/sugar-speak-9-3.fc10.src.rpm
APPROVED Please follow http://fedoraproject.org/wiki/CVSAdminProcedure and import the package. Close this bug as RAWHIDE once it's been successfully imported and built.
New Package CVS Request ======================= Package Name: sugar-speak Short Description: Speak for Sugar Owners: fab Branches: F-10 InitialCC:
cvs done.
sugar-speak-9-3.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/sugar-speak-9-3.fc10
sugar-speak-9-3.fc10 has been pushed to the Fedora 10 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-speak'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2008-11788
sugar-speak-9-3.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.