Bug 913367 - Review Request: gpick - Advanced color picker
Summary: Review Request: gpick - Advanced color picker
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Eugene A. Pivnev
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Keywords:
: 853775 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-02-21 05:24 UTC by Luya Tshimbalanga
Modified: 2013-04-25 14:08 UTC (History)
5 users (show)

(edit)
Clone Of:
(edit)
Last Closed: 2013-04-25 14:08:42 UTC
luya: fedora-review+
pbabinca: fedora-cvs+


Attachments (Terms of Use)

Description Luya Tshimbalanga 2013-02-21 05:24:13 UTC
Spec URL: http://luya.fedorapeople.org/packages/SPECS/gpick.spec
SRPM URL: http://luya.fedorapeople.org/packages/SRPMS/gpick-0.2.4-2.fc18.src.rpm
Koji build --scratch: http://koji.fedoraproject.org/koji/taskinfo?taskID=5038720
Description: Advanced color picker written in C++ using GTK+ toolkit
Fedora Account System Username: luya

Comment 1 Luya Tshimbalanga 2013-02-21 05:25:09 UTC
*** Bug 853775 has been marked as a duplicate of this bug. ***

Comment 2 Eugene A. Pivnev 2013-04-11 12:18:51 UTC
Package seems clear.
I get it on review swaping with next qt-based applications:
https://bugzilla.redhat.com/show_bug.cgi?id=929425 - advanced plain text editor
https://bugzilla.redhat.com/show_bug.cgi?id=947049 - keyboard layout switcher
https://bugzilla.redhat.com/show_bug.cgi?id=949154 - clipboard history manager

Comment 3 Luya Tshimbalanga 2013-04-11 19:03:41 UTC
Thank you, I will do the same for one of listed package.

Comment 4 Eugene A. Pivnev 2013-04-12 12:02:51 UTC
BUGS:
* %description: Description is smaller then Summary.
* %post, %postun: "/bin/touch --no-create %{_datadir}/icons/hicolor &>/dev/null || :" is hack. Add hicolor-icon-theme to BuildRequires and Requires.
* %files: As Fedora puts documents into %{_docdir}/%{name}-%{version} - replace current %doc expression with "share/doc/gpick/copyright" (patching SConstruct befor, naturally).

Issues (not blockers):
* Separate expressions inside sections something different then sections between each other. E.g. - double CR between sections and one CR between expression. Or one CR and no CR respectively.
* You can replace BRs with "rpmname-independent" expressions:
	dbus-glib-devel => pkgconfig(dbus-glib-1)
	gtk2-devel => pkgconfig(gtk+-2.0)
	lua-devel => pkgconfig(lua)
	expat-devel => pkgconfig(expat)
* You can replace "export ... scons..." with one line: "scons ... CFLAGS=... CXXFLAGS=..."
* You can replace %{_usr} with %{_prefix}

Questions:
* Why lemon is needed? I created package without it.

Comment 5 Rex Dieter 2013-04-12 13:49:38 UTC
Re comment #4
the BUGS: suggestion about icon-related %post,%postun scriptlets is invalid, please see:
https://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Icon_Cache
(in particular, the scriptlet's here are valid, and adding an extra Requires: hicolor-icon-theme is not, since gtk2 already pulls that in)


While we're on non-blocker topics: my own pet-peave... I'd suggest removing from Summary/description the text "written in C++ using GTK+ toolkit".  End users don't need to know or care what language or toolkit is used.

Comment 6 Luya Tshimbalanga 2013-04-14 07:10:42 UTC
Here is the updated spec and srpms including a short summary and description
Spec URL: http://luya.fedorapeople.org/packages/SPECS/gpick.spec
SRPM URL: http://luya.fedorapeople.org/packages/SRPMS/gpick-0.2.4-3.fc18.src.rpm
Koji build --scratch: http://koji.fedoraproject.org/koji/taskinfo?taskID=5249406

Upstream has released a newest version but I will update the package when it will be available to the repository.

Comment 7 Eugene A. Pivnev 2013-04-14 11:08:52 UTC
Greate. Everything clear but...
To Rex Dieter: I can't find in guidelines - whether "%doc %{_docdir}/%{name}/copyright" is accaptable?

Comment 8 Eugene A. Pivnev 2013-04-14 11:20:17 UTC
(In reply to comment #5)
> End users don't need to know or care what language or toolkit is used.

I agree with removeing language, but not - with GTK. User can want to know - how application look like. As for me - I prefere qt-based or kde-based - due gtk look'n'feel. This is an example - nothing personal :-)

Comment 9 Eugene A. Pivnev 2013-04-14 18:29:58 UTC
(In reply to comment #6)
> Here is the updated spec and srpms including a short summary and description
> Spec URL: http://luya.fedorapeople.org/packages/SPECS/gpick.spec
> SRPM URL:
> http://luya.fedorapeople.org/packages/SRPMS/gpick-0.2.4-3.fc18.src.rpm

Something wrong:

bash-4.2$ fedora-review -b 913367
INFO: Processing bugzilla bug: 913367
INFO: Getting .spec and .srpm Urls from : 913367
INFO:   --> SRPM url: http://luya.fedorapeople.org/packages/SRPMS/gpick-0.2.4-3.fc18.src.rpm
INFO:   --> Spec url: http://luya.fedorapeople.org/packages/SPECS/gpick.spec
INFO: Downloading .spec and .srpm files
ERROR: 'Error 404 downloading http://luya.fedorapeople.org/packages/SRPMS/gpick-0.2.4-3.fc18.src.rpm'

Comment 10 Luya Tshimbalanga 2013-04-14 23:46:41 UTC
The .srpm is locatd on other path. Here is the correct address
Spec URL: http://luya.fedorapeople.org/packages/SPECS/gpick.spec
SRPM URL: http://luya.fedorapeople.org/packages/gpick-0.2.4-3.fc18.src.rpm

I am currently on a place with unstable network.

Comment 11 Luya Tshimbalanga 2013-04-15 03:16:27 UTC
(In reply to comment #10)
> The .srpm is locatd on other path. Here is the correct address
> Spec URL: http://luya.fedorapeople.org/packages/SPECS/gpick.spec
> SRPM URL: http://luya.fedorapeople.org/packages/gpick-0.2.4-3.fc18.src.rpm
> 
> I am currently on a place with unstable network.

Disregard above post, I fixed the correct location of srpm files on 
SRPM url: http://luya.fedorapeople.org/packages/SRPMS/gpick-0.2.4-3.fc18.src.rpm
Could you try again?

Comment 12 Eugene A. Pivnev 2013-04-15 08:18:48 UTC
Ooops... Seems expat-devel in F17 has no pkg-config file:
http://kojipkgs.fedoraproject.org//work/tasks/2410/5252410/mock_output.log
I'm sorry.

Comment 13 Luya Tshimbalanga 2013-04-15 16:16:35 UTC
Here is these updated files
Spec URL: http://luya.fedorapeople.org/packages/SPECS/gpick.spec
SRPM URL: http://luya.fedorapeople.org/packages/SRPMS/gpick-0.2.4-3.fc18.src.rpm

Koji build of f19:
http://koji.fedoraproject.org/koji/taskinfo?taskID=5254527

Koji build for f17 (with expat-devel):
http://koji.fedoraproject.org/koji/taskinfo?taskID=5254552

Once accepted, f18+ version will use pkg-config.

Comment 14 Eugene A. Pivnev 2013-04-16 09:03:19 UTC
gpick.spec contains Relese: 4, but srpm is ...-3.
fedora-package produces error

Comment 15 Luya Tshimbalanga 2013-04-16 09:33:23 UTC
I forgot to change the srpm because release 4 is already published.
Spec URL: http://luya.fedorapeople.org/packages/SPECS/gpick.spec
SRPM URL: http://luya.fedorapeople.org/packages/SRPMS/gpick-0.2.4-4.fc18.src.rpm

Comment 16 Eugene A. Pivnev 2013-04-16 09:36:28 UTC
Be attentive: http://luya.fedorapeople.org/packages/SRPMS/
There is no -4.fc18

As I get - fedora-package gets last Spec URL and SRPM URL in bug thread.

Comment 17 Luya Tshimbalanga 2013-04-16 16:29:21 UTC
(In reply to comment #16)
> Be attentive: http://luya.fedorapeople.org/packages/SRPMS/
> There is no -4.fc18

It is now uploaded.

Comment 18 Eugene A. Pivnev 2013-04-17 06:36:00 UTC
During reviewing:
"[ ]: Latest version is packaged."
http://code.google.com/p/gpick/downloads/list - latest is 0.2.5 but 0.2.4 packaged.

Comment 19 Luya Tshimbalanga 2013-04-17 09:28:55 UTC
I am aware of it but the review was submitted before 0.2.5 was released.

Comment 20 Luya Tshimbalanga 2013-04-17 09:40:58 UTC
In addition, 0.2.5 requires lua >= 5.2 which is not available on Fedora 18.

Comment 21 Eugene A. Pivnev 2013-04-17 09:54:06 UTC
Package Review
==============
===== MUST items =====
OK
===== SHOULD items =====
* Latest version is packaged.
* Patches link to upstream bugs/comments/lists or are otherwise justified.
* Packages should try to preserve timestamps of original installed files.

Rpmlint
-------
OK but source n/a

MD5-sum check
-------------
N/a because of source N/A

APPROVED

Comment 22 Luya Tshimbalanga 2013-04-17 16:15:13 UTC
Weird. Bugzilla somehow changed fedora-review+ to luya fedora-review? which requesting fedora-cvs? . Is it some kind of bug?

Comment 23 Gwyn Ciesla 2013-04-17 19:08:19 UTC
No SCM request found, un-setting fedora-cvs.

Comment 24 Luya Tshimbalanga 2013-04-18 06:37:55 UTC
New Package SCM Request
=======================
Package Name: gpick
Short Description: Advanced color picker
Owners: luya
Branches: f19 f18 f17 el6
InitialCC: luya

Comment 25 Pavol Babinčák 2013-04-18 12:11:42 UTC
Git done (by process-git-requests).

Comment 26 Fedora Update System 2013-04-18 17:57:22 UTC
gpick-0.2.4-4.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/gpick-0.2.4-4.fc18

Comment 27 Fedora Update System 2013-04-18 17:57:35 UTC
gpick-0.2.4-4.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/gpick-0.2.4-4.fc19

Comment 28 Fedora Update System 2013-04-18 17:57:44 UTC
gpick-0.2.4-4.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/gpick-0.2.4-4.el6

Comment 29 Fedora Update System 2013-04-18 17:57:54 UTC
gpick-0.2.4-4.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/gpick-0.2.4-4.fc17

Comment 30 Fedora Update System 2013-04-19 04:48:40 UTC
gpick-0.2.4-4.fc17 has been pushed to the Fedora 17 testing repository.

Comment 31 Fedora Update System 2013-04-25 14:08:44 UTC
gpick-0.2.4-4.fc19 has been pushed to the Fedora 19 stable repository.


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