Bug 432137 (kcoloredit)

Summary: Review Request: kcoloredit - A color palette Editor for KDE 4
Product: [Fedora] Fedora Reporter: Sebastian Vahl <fedora>
Component: Package ReviewAssignee: Parag AN(पराग) <panemade>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, notting, rdieter
Target Milestone: ---Flags: panemade: fedora-review+
kevin: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-02-13 10:51:17 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Sebastian Vahl 2008-02-09 00:49:15 UTC
Spec URL: http://svahl.fedorapeople.org/kcoloredit/kcoloredit.spec
SRPM URL: http://svahl.fedorapeople.org/kcoloredit/kcoloredit-4.0.1-1.fc8.src.rpm
Description: 
KColorEdit is a palette files editor. It can be used for editing
color palettes and for color choosing and naming

Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=405234

Comment 1 Parag AN(पराग) 2008-02-09 02:32:11 UTC
I confused on Requires: oxygen-icon-theme. Can it be Reqires: hicolor-icon-theme?

Comment 2 Parag AN(पराग) 2008-02-09 02:52:21 UTC
On my rawhide machine, When I rebuilt this srpm and installed rpm, I got

Preparing...                ########################################### [100%]
        file /usr/share/locale/ta/LC_MESSAGES/kcoloredit.mo from install of
kcoloredit-4.0.1-1.fc9.i386 conflicts with file from package
kde-i18n-Tamil-3.5.8-3.fc9.noarch
        file /usr/share/locale/ja/LC_MESSAGES/kcoloredit.mo from install of
kcoloredit-4.0.1-1.fc9.i386 conflicts with file from package
kde-i18n-Japanese-3.5.8-3.fc9.noarch
        file /usr/share/locale/pa/LC_MESSAGES/kcoloredit.mo from install of
kcoloredit-4.0.1-1.fc9.i386 conflicts with file from package
kde-i18n-Punjabi-3.5.8-3.fc9.noarch
        file /usr/share/locale/zh_TW/LC_MESSAGES/kcoloredit.mo from install of
kcoloredit-4.0.1-1.fc9.i386 conflicts with file from package
kde-i18n-Chinese-Big5-3.5.8-3.fc9.noarch
        file /usr/share/locale/zh_CN/LC_MESSAGES/kcoloredit.mo from install of
kcoloredit-4.0.1-1.fc9.i386 conflicts with file from package
kde-i18n-Chinese-3.5.8-3.fc9.noarch


Comment 3 Parag AN(पराग) 2008-02-09 02:58:36 UTC
Just found that 
repoquery --whatprovides oxygen-icon-theme
kdebase-runtime-0:4.0.1-1.fc9.i386


Comment 4 Kevin Kofler 2008-02-11 21:55:44 UTC
These conflicting files will be removed from kde-i18n ASAP. (We want to ship 
the version from the current kcoloredit, not the KDE 3 one.)

As for oxygen-icon-theme, it's needed (it's the only theme to provide some of 
the used icons, and also the global fallback in KDE 4, and also what we're 
defaulting to), and as you've noticed, it's currently in kdebase-runtime anyway 
(it might get split out later).

Comment 5 Parag AN(पराग) 2008-02-12 01:13:41 UTC
Review:
+ package builds in mock (rawhide i386).
koji build => http://koji.fedoraproject.org/koji/taskinfo?taskID=405234
+ rpmlint is silent for SRPM. Bur NOT for RPM.
kcoloredit.i386: W: dangling-symlink /usr/share/doc/HTML/en/kcoloredit/common
/usr/share/doc/HTML/en/common
kcoloredit.i386: W: symlink-should-be-relative
/usr/share/doc/HTML/en/kcoloredit/common /usr/share/doc/HTML/en/common
==> Ok to accept here
+ source files match upstream.
77f69d70f105f83713e427ae0c8f1e5b  kcoloredit-4.0.1.tar.bz2
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc files present.
+ BuildRequires are proper.
+ Compiler flags are honored correctly.
+ defattr usage is correct.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code.
+ no static libraries.
+ no .pc file present.
+ no -devel subpackage exists.
+ no .la files.
+ translations are available.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ xdg-utils scriptlets are used.
+ Desktop file installed correctly.
+ GUI app.

APPROVED.


Comment 6 Kevin Kofler 2008-02-12 01:31:48 UTC
File conflicts fixed in kde-i18n-3.5.8-5.fc9.

Comment 7 Parag AN(पराग) 2008-02-12 01:37:24 UTC
(In reply to comment #6)
> File conflicts fixed in kde-i18n-3.5.8-5.fc9.

thanks.

Comment 8 Sebastian Vahl 2008-02-12 13:49:44 UTC
(In reply to comment #5)
> APPROVED.

Thanks!

New Package CVS Request
=======================
Package Name: kcoloredit
Short Description: A color palette Editor for KDE 4
Owners: svahl,than,rdieter,kkofler,ltinkl
Branches: 
InitialCC: 
Cvsextras Commits: no



Comment 9 Kevin Fenzi 2008-02-12 17:42:19 UTC
cvs done.

Comment 10 Sebastian Vahl 2008-02-13 10:51:17 UTC
Imported and built for rawhide: 
http://koji.fedoraproject.org/koji/buildinfo?buildID=35906