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 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.