Bug 654139 - Review Request: ease - GNOME desktop presentation application
Summary: Review Request: ease - GNOME desktop presentation application
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Arun S A G
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-11-17 00:21 UTC by Michel Lind
Modified: 2011-01-09 21:00 UTC (History)
4 users (show)

Fixed In Version: ease-0.4-1.fc13
Clone Of:
Environment:
Last Closed: 2011-01-09 20:58:06 UTC
Type: ---
Embargoed:
sagarun: fedora-review+
j: fedora-cvs+


Attachments (Terms of Use)

Description Michel Lind 2010-11-17 00:21:37 UTC
Spec URL: http://salimma.fedorapeople.org/specs/gnome/ease.spec
SRPM URL: http://salimma.fedorapeople.org/specs/gnome/ease-0.4-1.fc14.src.rpm
Description:
Ease is a modern desktop presentation application for GNOME, with
Clutter-based visuals.

With a simple design, Ease is painless to use. It is intended to
integrate well into your current GNOME-based desktop.

Comment 1 Michel Lind 2010-11-17 00:31:29 UTC
Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2605434

rpmlint:
SRPM:
ease.src:53: E: hardcoded-library-path in /usr/lib
==> this is due to the rpath fix, adding /lib64 and /usr/lib64 to the system
    libdirs so that libtool does not add RPATHs for /usr/lib64 in the linked
    binaries

ease.src: W: no-cleaning-of-buildroot %clean
ease.src: W: no-buildroot-tag
ease.src: W: no-%clean-section

==> only targeting recent Fedora releases

1 packages and 0 specfiles checked; 1 errors, 3 warnings.

ease-*.x86_64.rpm:
ease.x86_64: W: no-manual-page-for-binary ease
ease-devel.x86_64: W: no-documentation
3 packages and 0 specfiles checked; 0 errors, 2 warnings.

note: rpmlint-ing the installed RPM results in a lot of undefined-non-weak-symbol warnings; this might shed some light into the issue: http://sourceware.org/ml/libc-alpha/2003-05/msg00034.html

Comment 2 Michel Lind 2010-11-17 00:37:56 UTC
I'm taking a slightly unusual approach to specifying BuildRequires:, 
specifying pkgconfig modules instead of the usual -devel package names. 
This aligns better with configure.ac (so one can easily verify that the 
two are in sync), and build problems due to API changes would abort the 
build earlier (during root FS setup rather than during the rebuild phase).

Comment 3 Michel Lind 2010-11-17 16:46:38 UTC
oh, p.s. this does not currently build on Rawhide -- Vala'd need to be updated first for some of the updated bindings (clutter-gtk-1.0 and rest-0.7)

Comment 4 Arun S A G 2010-11-25 02:05:08 UTC
I will do the review by this weekend.

Comment 5 Michel Lind 2010-11-25 11:00:21 UTC
Thanks! FYI -- I've tested basic functionality on a Fedora 14 machine with an Intel card and it works fine (it won't work with the open-source radeon or nouveau drivers, even with the experimental Mesa DRI drivers, but would probably work with the binary nVidia drivers -- but those have issues on my workstation's older nVidia card).

Comment 6 Arun S A G 2010-12-07 16:27:59 UTC
I see some .pc files in -devel subpackage, Don't they require 'pkgconfig' ?

Comment 7 Michel Lind 2010-12-14 23:49:39 UTC
(In reply to comment #6)
> I see some .pc files in -devel subpackage, Don't they require 'pkgconfig' ?

Hi Arun,

That's handled automatically by RPM:

$ rpm -qp --requires ease-devel-0.4-1.fc14.x86_64.rpm | grep pkg-config
/usr/bin/pkg-config  
$ rpm -qf /usr/bin/pkg-config
pkgconfig-0.25-2.fc14.x86_64

Unfortunately the Koji build results have been garbage collected by now; you can check the rebuild I just made here:

http://koji.fedoraproject.org/koji/taskinfo?taskID=2664560

Just in case, though, the Koji-built RPMs are now cached in 
http://salimma.fedorapeople.org/specs/gnome/

as well. Please check them if the Koji builds are no longer accessible.

Comment 8 Arun S A G 2010-12-15 07:28:22 UTC
[+] OK
[X] NOT OKAY
[-] NA


[+] Package meets naming and packaging guidelines
[+] Spec file matches base package name.
[+] Spec has consistant macro usage.
[+] Meets Packaging Guidelines.
[+] License
[+] License field in spec matches
[+] License file included in package
[+] Spec in American English
[+] Spec is legible.
[+] Sources match upstream md5sum:
[zer0c00l@gnubox SOURCES]$ md5sum ~/Downloads/ease-0.4.tar.bz2 
9134b74a2ff3701b62d08f5b4d954232  /home/zer0c00l/Downloads/ease-0.4.tar.bz2

-- done
[-] Package needs ExcludeArch
[+] BuildRequires correct
[+] Spec handles locales/find_lang
[-] Package is relocatable and has a reason to be.
[+] Package has %defattr and permissions on files is good.
[-] Package has a correct %clean section.
[-] Package has correct buildroot
[+] Package is code or permissible content.
[-] Doc subpackage needed/used.
[-] Packages %doc files don't affect runtime.

[+] Headers/static libs in -devel subpackage.
[+] Spec has needed ldconfig in post and postun
[X] .pc files in -devel subpackage/requires pkgconfig
[+] .so files in -devel subpackage.
[+] -devel package Requires: %{name} = %{version}-%{release}
[+] .la files are removed.

[+] Package is a GUI app and has a .desktop file

[+] Package compiles and builds on at least one arch.
[+] Package has no duplicate files in %files.
[+] Package doesn't own any directories other packages own.
[+] Package owns all the directories it creates.

[+] No rpmlint output : ignorable
[zer0c00l@gnubox i686]$ rpmlint ease-0.4-1.fc13.i686.rpm 
ease.i686: W: no-manual-page-for-binary ease
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
[zer0c00l@gnubox i686]$ rpmlint ease-devel-0.4-1.fc13.i686.rpm 
ease-devel.i686: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
[zer0c00l@gnubox i686]$ rpmlint ~/rpmbuild/SRPMS/ease-0.4-1.fc13.src.rpm 
ease.src:53: E: hardcoded-library-path in /usr/lib
ease.src: W: no-cleaning-of-buildroot %clean
ease.src: W: no-buildroot-tag
ease.src: W: no-%clean-section

Note: hardcoded-library-path in /usr/lib @ Line 53. Where the line is "sed -i 's|/lib /usr/lib|/lib /usr/lib /lib64 /usr/lib64|' libtool".
It removes rpath. Not an error.


SHOULD Items:

[+] Should build in mock.
[+] Should build on all supported archs
[+] Should function as described.
[-] Should have sane scriptlets.
[-] Should have subpackages require base package with fully versioned depend.
[+] Should have dist tag
[+] Should package latest version
[+] check for outstanding bugs on package. (For core merge reviews)


XXX APPROVED XXX

Comment 9 Michel Lind 2010-12-15 08:52:34 UTC
Thanks!

New Package SCM Request
=======================
Package Name: ease
Short Description: GNOME desktop presentation application 
Owners: salimma
Branches: f13 f14
InitialCC:

Comment 10 Jason Tibbitts 2010-12-15 13:54:26 UTC
Git done (by process-git-requests).

Comment 11 Fedora Update System 2010-12-16 13:58:39 UTC
ease-0.4-1.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/ease-0.4-1.fc13

Comment 12 Fedora Update System 2010-12-16 13:58:46 UTC
ease-0.4-1.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/ease-0.4-1.fc14

Comment 13 Fedora Update System 2010-12-17 08:24:55 UTC
ease-0.4-1.fc14 has been pushed to the Fedora 14 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 ease'.  You can provide feedback for this update here: https://admin.fedoraproject.org/updates/ease-0.4-1.fc14

Comment 14 Fedora Update System 2011-01-09 20:58:00 UTC
ease-0.4-1.fc14 has been pushed to the Fedora 14 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 15 Fedora Update System 2011-01-09 21:00:37 UTC
ease-0.4-1.fc13 has been pushed to the Fedora 13 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.