Bug 1030073 - Review Request: libtimezonemap - Time zone map widget for Gtk+
Summary: Review Request: libtimezonemap - Time zone map widget for Gtk+
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Christopher Meng
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-11-13 21:22 UTC by David Shea
Modified: 2013-11-15 17:12 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-11-14 18:19:41 UTC
Type: ---
Embargoed:
i: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description David Shea 2013-11-13 21:22:00 UTC
Spec URL: http://dshea.fedorapeople.org/libtimezonemap/libtimezonemap.spec
SRPM URL: http://dshea.fedorapeople.org/libtimezonemap/libtimezonemap-0.4.1-1.fc19.src.rpm

Description:
libtimezonemap is a time zone map widget for Gtk+. The widget displays a world
map with a highlighted region representing the selected time zone, and the
location can be changed by clicking on the map.

This library is a fork of the of the code from gnome-control-center's datetime
panel, which was itself a fork of Ubiquity's timezone map.

Fedora Account System Username: dshea

Comment 1 Christopher Meng 2013-11-14 13:26:14 UTC
1. Better use macro consistently as:

Summary:        Development files for %{name}
Requires:       %{name}%{?_isa} = %{version}-%{release}

%description    devel
This package contains development files for
developing applications that use %{name}.

2. 
Rpmlint (installed packages)
----------------------------
# rpmlint libtimezonemap libtimezonemap-devel
libtimezonemap.i686: W: spelling-error %description -l en_US datetime -> date time, date-time, daytime
libtimezonemap.i686: W: unused-direct-shlib-dependency /usr/lib/libtimezonemap.so.1.0.0 /lib/libpangocairo-1.0.so.0
libtimezonemap.i686: W: unused-direct-shlib-dependency /usr/lib/libtimezonemap.so.1.0.0 /lib/libpango-1.0.so.0
libtimezonemap.i686: W: unused-direct-shlib-dependency /usr/lib/libtimezonemap.so.1.0.0 /lib/libatk-1.0.so.0
libtimezonemap.i686: W: unused-direct-shlib-dependency /usr/lib/libtimezonemap.so.1.0.0 /lib/libcairo-gobject.so.2
libtimezonemap-devel.i686: W: no-documentation
2 packages and 0 specfiles checked; 0 errors, 6 warnings.
# echo 'rpmlint-done:'


3. GPL (v2 or later)
-----------------
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/libtimezonemap-0.4.1/src/cc-timezone-map.c
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/libtimezonemap-0.4.1/src/cc-timezone-map.h
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/libtimezonemap-0.4.1/src/tz.c
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/libtimezonemap-0.4.1/src/tz.h

GPL (v3)
--------
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/libtimezonemap-0.4.1/src/timezone-completion.c
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/libtimezonemap-0.4.1/src/timezone-completion.h

------------------

PACKAGE APPROVED.

Can you fix 3 issues above? Or notify upstream?

Comment 2 David Shea 2013-11-14 14:51:47 UTC
(In reply to Christopher Meng from comment #1)
> 1. Better use macro consistently as:
> 
> Summary:        Development files for %{name}
> Requires:       %{name}%{?_isa} = %{version}-%{release}
> 
> %description    devel
> This package contains development files for
> developing applications that use %{name}.

Ok. Updated the spec and srpm on fedorapeople.

> 
> 2. 
> Rpmlint (installed packages)
> ----------------------------
> # rpmlint libtimezonemap libtimezonemap-devel
> libtimezonemap.i686: W: spelling-error %description -l en_US datetime ->
> date time, date-time, daytime

Intentional, this is referring a GNOME control-center panel named "datetime"

> libtimezonemap.i686: W: unused-direct-shlib-dependency
> /usr/lib/libtimezonemap.so.1.0.0 /lib/libpangocairo-1.0.so.0
> libtimezonemap.i686: W: unused-direct-shlib-dependency
> /usr/lib/libtimezonemap.so.1.0.0 /lib/libpango-1.0.so.0
> libtimezonemap.i686: W: unused-direct-shlib-dependency
> /usr/lib/libtimezonemap.so.1.0.0 /lib/libatk-1.0.so.0
> libtimezonemap.i686: W: unused-direct-shlib-dependency
> /usr/lib/libtimezonemap.so.1.0.0 /lib/libcairo-gobject.so.2

pango and cairo and other accouterments are pulled in by the gtk+ pkgconfig whether they're used or not, so this is more gtk's fault than anything.

> libtimezonemap-devel.i686: W: no-documentation

Yeah. I'd like this package to include some API documentation generated from doc comments, but right now there are no doc comments. Working with upstream on getting those in place.

> 2 packages and 0 specfiles checked; 0 errors, 6 warnings.
> # echo 'rpmlint-done:'
> 
> 
> 3. GPL (v2 or later)
> -----------------
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/libtimezonemap-0.
> 4.1/src/cc-timezone-map.c
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/libtimezonemap-0.
> 4.1/src/cc-timezone-map.h
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/libtimezonemap-0.
> 4.1/src/tz.c
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/libtimezonemap-0.
> 4.1/src/tz.h
> 
> GPL (v3)
> --------
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/libtimezonemap-0.
> 4.1/src/timezone-completion.c
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/libtimezonemap-0.
> 4.1/src/timezone-completion.h
>

I updated the spec file to use GPLv3 as the license. I'll ask upstream if they want to change the boilerplate on the stuff inherited from GNOME to GPLv3, but since it's already v2 or later I don't think it should be an issue as-is.

Comment 3 Christopher Meng 2013-11-14 15:49:03 UTC
PACKAGE APPROVED.

Comment 4 David Shea 2013-11-14 16:13:46 UTC
New Package SCM Request
=======================
Package Name: libtimezonemap
Short Description: Time zone map widget for Gtk+
Owners: dshea
Branches: 
InitialCC:

Comment 5 Gwyn Ciesla 2013-11-14 17:22:43 UTC
Git done (by process-git-requests).

Comment 6 David Shea 2013-11-14 18:19:41 UTC
Package built for devel.

Comment 7 David Shea 2013-11-15 16:18:47 UTC
Package Change Request
======================
Package Name: libtimezonemap
New Branches: f20
Owners: dshea
InitialCC: 

Changed my mind about it being >= f21 only, since we'd like to be able to build anaconda on F20. Sorry, thanks.

Comment 8 Gwyn Ciesla 2013-11-15 17:12:27 UTC
Git done (by process-git-requests).


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