Bug 429333 - Review Request: kscope - KDE front-end to Cscope
Summary: Review Request: kscope - KDE front-end to Cscope
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review   
(Show other bugs)
Version: rawhide
Hardware: All Linux
Target Milestone: ---
Assignee: Rex Dieter
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2008-01-18 19:01 UTC by Tom "spot" Callaway
Modified: 2008-01-20 22:20 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-01-20 22:20:32 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
rdieter: fedora-review+
tcallawa: fedora-cvs+

Attachments (Terms of Use)

Description Tom "spot" Callaway 2008-01-18 19:01:49 UTC
Spec URL: http://www.auroralinux.org/people/spot/review/new/kscope.spec
SRPM URL: http://www.auroralinux.org/people/spot/review/new/kscope-1.6.1-1.fc9.src.rpm
KScope is a KDE front-end to Cscope. It provides a source-editing
environment for large C projects, such as the Linux kernel.

KScope is by no means intended to be a replacement to any of the leading
Linux/KDE IDEs, such as KDevelop. First of all, it is not an Integrated
Development Environment: it does not provide the usual write/compile/debug
cycle supported by most IDE's. Instead, KScope is focused on source
editing and analysis.

Comment 1 Rex Dieter 2008-01-18 20:54:24 UTC
2 Quick-n-dirty suggestions
1.  use desktop-file-install on .desktop file, with final location being 
(something like): %_datadir/applications/kde/kscope.desktop
(you may need to add a category or 2).
2.  drop Requires: kdelibs-common 
(it'll get pulled in by the default kde runtime pkgs)

Comment 2 Rex Dieter 2008-01-18 20:55:06 UTC
I'll get started on more of the formal stuff in a bit...

Comment 3 Rex Dieter 2008-01-18 21:14:57 UTC
3.  SHOULD: drop libtool hacks, use
%configure --disable-rpath 

4.  MUST: add missing BR: kdebase3-devel (links against libkate from 

src.rpm and locally built x86_64 pkgs are rpmlint clean (other than harmless 
warning about dangling ../common symlink).

tested local built rpm, app launches/runs ok.

Consider 1-4, and I'll approve this.

Comment 4 Rex Dieter 2008-01-18 21:16:54 UTC
submitted scratch build (w/ items 3,4 fixed) for giggles:

Comment 6 Rex Dieter 2008-01-20 21:17:53 UTC
Close enough, APPROVED.

1 small thing (I'll leave it to you to address before building anything) 
regarding d-f-i usage, you used:
--vendor="fedora" ... --dir $RPM_BUILD_ROOT%{_datadir}/applications/kde
in effect having 2 vendors (fedora prefix + kde dir prefix), I'd suggest using
--vendor="" ... --dir $RPM_BUILD_ROOT%{_datadir}/applications/kde

Comment 7 Tom "spot" Callaway 2008-01-20 22:00:48 UTC
New Package CVS Request
Package Name: kscope
Short Description: KDE front-end to Cscope
Owners: spot
Branches: F-7 F-8 EL-4 EL-5 
Cvsextras Commits: yes

Comment 8 Rex Dieter 2008-01-20 22:06:58 UTC
noting the request for epel branches: only fedora kde packages include Provides
for kdelibs3/kdebase3, for rhel, you'll need kdelibs/kdebase, respectively.

Comment 9 Tom "spot" Callaway 2008-01-20 22:08:12 UTC
Good to know, thanks. 

cvs done.

Comment 10 Tom "spot" Callaway 2008-01-20 22:20:32 UTC
Builds away, thanks for the quick review.

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