Spec URL: http://fab.fedorapeople.org/packages/XO/sugar-read.spec SRPM URL: http://fab.fedorapeople.org/packages/XO/sugar-read-52-1.fc10.src.rpm Description: The Read activity allows the laptop to act as a book reader. It has a simple interface, and will view many kinds of text and image-based book- like materials. It will have particular strengths in handheld mode, with extremely low power consumption and simple navigation controls. Read can read PDF files, single-page TIFF files, and also read DJVU files. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=889282 [fab@localhost noarch]$ rpmlint -i sugar-read-52-1.fc10.noarch.rpm sugar-read.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. [fab@localhost SRPMS]$ rpmlint -i sugar-read-52-1.fc10.src.rpm sugar-read.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.
Does this activity work? I was under the impression that evince changes are required for sugar-read
Yeah, I guess the activity is not actually working. We need to port sugar-evince to latest poppler (I already did some work on it) and also merge it upstream hopefully.
What's the current status of this package? If it doesn't actually work then there's not much point in reviewing it, so I'll mark it as not being ready for review. Please clear the whiteboard when the situation changes.
As far as I know, is Tomeu Vizoso working on this.
evince 2.25.90 and gnome-python-desktop 2.25.90 have been released and would be enough to run sugar-read. sugar-evince is not needed any more as has been completely upstreamed.
Looking at the spec file should there not be a Requires: evince and gnome-python-desktop? I believe there needs to be a gnome-python2-evince and I've also filed a bug to have the evince libs split out into a sub package as I don't belive sugar-read needs any of the evice gui but rather just the libraries and python binaries. The bug is RHBZ 480729
gnome-python2-evince is in rawhide.
mbarnes rebuilt it a few days ago since it was broken - should be good to go now. Anything that is missing spec wise now for getting this in?
Latest 0.84 version for read is 63 http://download.sugarlabs.org/sources/sucrose/fructose/Read/Read-63.tar.bz2 Building Read depending on the latest evince would only work for rawhide of course.
I will update the spec file and post a new SRPM.
> Building Read depending on the latest evince would only work for rawhide of > course. Yes, but sugar-evince was never in the rest of Fedora mainline anyway so it shouldn't cause any issues, except it will only be supported on F11 and later.
Hi Fabian, have you had any luck with an updated srpm?
SPEC URL: http://shell.sugarlabs.org/~erikos/sugar-read.spec SRPM URL: http://shell.sugarlabs.org/~erikos/sugar-read-65-1.fc10.src.rpm Built fine: http://koji.fedoraproject.org/koji/taskinfo?taskID=1219708 Added evince and gnome-python-desktop to the spec.
The gnome-python requirement will actually need to be the specific sub components so it doesn't pull the whole gnome-python stack and its dependencies which are massive. So the minimum would be gnome-python2-evince and the other components it uses.
Thanks. Updated the spec to require gnome-python2-evince. I guess I do not need to specifically list evince as requirement since this is dragged in by gnome-python2-evince
Going to do the review now. Just a side note that each change should increment the build and add Changelog entry.
APPROVED + rpmlint output $ rpmlint -i /home/perobinson/rpmbuild/SRPMS/sugar-read-65-1.fc10.src.rpm sugar-read.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. + package name satisfies the packaging naming guidelines + specfile name matches the package base name + package should satisfy packaging guidelines + license meets guidelines and is acceptable to Fedora + license matches the actual package license + %doc includes license file + spec file written in American English + spec file is legible + upstream sources match sources in the srpm 3a7c7cc764e139691a7f57c16cd6433c Read-65.tar.bz2 + package successfully builds on at least one architecture tested using koji scratch build + BuildRequires list all build dependencies + %find_lang instead of %{_datadir}/locale/* n/a binary RPM with shared library files must call ldconfig in %post and %postun+ does not use Prefix: /usr n/a package owns all directories it creates n/a no duplicate files in %files + %defattr line + %clean contains rm -rf $RPM_BUILD_ROOT + consistent use of macros + package must contain code or permissible content n/a large documentation files should go in -doc subpackage + files marked %doc should not affect package n/a header files should be in -devel n/a static libraries should be in -static n/a packages containing pkgconfig (.pc) files need 'Requires: pkgconfig' n/a libfoo.so must go in -devel n/a devel must require the fully versioned base + packages should not contain libtool .la files n/a packages containing GUI apps must include %{name}.desktop file + packages must not own files or directories owned by other packages + %install must start with rm -rf %{buildroot} etc. + filenames must be valid UTF-8 Optional: n/a if there is no license file, packager should query upstream n/a translations of description and summary for non-English languages, if available + reviewer should build the package in mock/koji n/a the package should build into binary RPMs on all supported architectures n/a review should test the package functions as described + scriptlets should be sane n/a pkgconfig files should go in -devel + shouldn't have file dependencies outside /etc /bin /sbin /usr/bin or /usr/sbin
New Package CVS Request ======================= Package Name: sugar-read Short Description: A document reader for Sugar Owners: erikos sdz
@peter: thanks very much for the review
I assume you only want a devel branch? cvs done.
@Kevin - yup I only wanted to have a devel branch, thanks for the quick handling The rpm built fine - so i am going to close this ticket.
Package Change Request ====================== Package Name: sugar-read New Branches: EL-6 Owners: pbrobinson sdz
cvs done.