Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 196378 - Review Request: kdegraphics-extras: Extras, including kuickshow
Summary: Review Request: kdegraphics-extras: Extras, including kuickshow
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Dennis Gilmore
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-06-22 20:16 UTC by Rex Dieter
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-07-25 13:39:54 UTC
Type: ---
Embargoed:

Attachments (Terms of Use)

Description Rex Dieter 2006-06-22 20:16:28 UTC 
Spec URL: http://kde-redhat.unl.edu/apt/kde-redhat/SPECS/kdegraphics-extras.spec
SRPM URL: http://kde-redhat.unl.edu/apt/kde-redhat/all/SRPMS.testing/kdegraphics-extras-3.5.3-3.src.rpm
Description:
Extra graphics applications for the K Desktop Environment, including:
* kuickshow (quick picture viewer)
* OpenEXR kfile plugin
Comment 1 Rex Dieter 2006-06-22 20:17:28 UTC 
This is targetted for fc6+
Comment 2 Parag AN(पराग) 2006-06-23 05:20:53 UTC 
Rex i got error in mock build
error: unpacking of archive failed on file
/builddir/build/SOURCES/kdegraphics-3.5.3.tar.bz2;449b7b15: cpio: read failed -
Invalid argument
Error installing srpm: kdegraphics-extras-3.5.3-3.src.rpm
Comment 3 Rex Dieter 2006-06-23 11:24:00 UTC 
Unpacking/packing the srpm (in mock) WORKSFORME (using mock-0.4 on rhel4 host):

/sbin/runuser -c 'rpm -Uvh --nodeps
/builddir/build/originals/kdegraphics-extras-3.5.3-3.src.rpm' mockbuild
warning: /builddir/build/originals/kdegraphics-extras-3.5.3-3.src.rpm: V3 DSA
signature: NOKEY, key ID ff6382fa
kdegraphics-extras          warning: user rdieter1 does not exist - using root
warning: group staff does not exist - using root
warning: user rdieter1 does not exist - using root
warning: group staff does not exist - using root
warning: user rdieter1 does not exist - using root
warning: group staff does not exist - using root
warning: user rdieter1 does not exist - using root
warning: group staff does not exist - using root
warning: user rdieter1 does not exist - using root
warning: group staff does not exist - using root
##################################################
warning: Could not canonicalize hostname: foobar...
Building target platforms: i386
Building for target i386
Wrote: /builddir/build/SRPMS/kdegraphics-extras-3.5.3-3.el4.src.rpm
Comment 4 Jonathan Underwood 2006-06-29 11:44:56 UTC 
Can i make an appeal to you to include kpovmodeler which is not currently built
a part of the core kedgraphics package 

(see https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=179589)

Although povray is incompatible with core/extras licensing requirements, I see
no reason why kpovmodeler would be. (Granted, it won't function unless povray is
installed.)
Comment 5 Dennis Gilmore 2006-06-29 13:50:10 UTC 
I will review this
Comment 6 Rex Dieter 2006-06-29 13:59:48 UTC 
Updated URLs:
Spec URL: http://kde-redhat.unl.edu/apt/kde-redhat/SPECS/kdegraphics-extras.spec
SRPM URL:
http://kde-redhat.unl.edu/apt/kde-redhat/all/SRPMS.stable/kdegraphics-extras-3.5.3-3.src.rpm
Comment 7 Dennis Gilmore 2006-07-10 18:51:33 UTC 
ok  now that imlib is in extras i can review this

[dennis@files extrasreviews]$ 
rpmlint /var/lib/mock/fedora-development-x86_64-core/result/kdegraphics-extras-3.5.3-3.fc6.x86_64.rpm
W: kdegraphics-extras incoherent-version-in-changelog 7:3.5.3-3 3.5.3-3.fc6
E: kdegraphics-extras invalid-soname /usr/lib64/libkdeinit_kuickshow.so 
libkdeinit_kuickshow.so

you have removed the epoch  from the package  but left it in the changelog 
entry.

[dennis@files SPECS]$ md5sum ../SOURCES/kdegraphics-3.5.3.tar.bz2
56bd5537521dc94633f8effaace523cb  ../SOURCES/kdegraphics-3.5.3.tar.bz2

matches what is posted on http://www.kde.org/download/ kdegraphics-3.5.3  
6.9MB  56bd5537521dc94633f8effaace523cb 

builds fine in mock
Comment 8 Rex Dieter 2006-07-10 18:57:48 UTC 
> you have removed the epoch  from the package  but left it in the changelog 
> entry.

A side-effect of my using a single specfile for both kdegraphics and 
kdegraphics-extras, and choosing to omit the Epoch from the new package.
Comment 9 Rex Dieter 2006-07-24 19:03:51 UTC 
Dennis, any more MUSTFIX items?
Comment 10 Dennis Gilmore 2006-07-25 11:18:06 UTC 
I think it looks good
APPROVED
Comment 11 Rex Dieter 2006-07-25 13:39:54 UTC 
Thanks, importing...
Note You need to log in before you can comment on or make changes to this bug.