Bug 225967 - Merge Review: kdevelop
Merge Review: kdevelop
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Rex Dieter
Fedora Package Reviews List
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-01-31 14:15 EST by Nobody's working on this, feel free to take it
Modified: 2015-01-08 08:48 EST (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-06-20 10:30:24 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
rdieter: fedora‑review+
limburgher: fedora‑cvs+


Attachments (Terms of Use)
gzip'd rpmlint output (5.53 KB, application/x-tar)
2007-03-08 11:29 EST, Rex Dieter
no flags Details

  None (edit)
Description Nobody's working on this, feel free to take it 2007-01-31 14:15:37 EST
Fedora Merge Review: kdevelop

http://cvs.fedora.redhat.com/viewcvs/devel/kdevelop/
Initial Owner: than@redhat.com
Comment 1 Rex Dieter 2007-03-08 11:27:25 EST
1.  SHOULD fix wrong-file-end-of-line-encoding warnings (see rpmlint.log)

2.  MUST add to %post/%postun scriptlets: /sbin/ldconfig

3.  SHOULD consider using versioned obsoletes:
Obsoletes: kdevelop-c_c++_ref <= 2.0.2 
?
Comment 2 Rex Dieter 2007-03-08 11:29:21 EST
Created attachment 149580 [details]
gzip'd rpmlint output
Comment 3 Rex Dieter 2007-03-08 11:42:00 EST
4.  SHOULD: package c_cpp_references-2.0.2 separately, imo.  (ie, why build and
upgrade users for this static item for *every* kdevelop rebuild?).

5.  SHOULD: omit configure flags:
  --with-qt-libraries=$QTDIR/lib \
  --with-qt-includes=$QTDIR/include \
  --with-extra-libs=%{_libdir}
I don't think they're needed (anymore).

the only MUST item, is relatively minor, and I'll trust you to adjust that
post-review, so APPROVED.
Comment 4 Rex Dieter 2007-06-20 10:30:24 EDT
MUST items are now in cvs, closing...
Comment 5 Jan Grulich 2015-01-08 04:35:45 EST
Package Change Request
======================
Package Name: kdevelop
New Branches: epel7
Owners: jgrulich rdieter kkofler dvratil ltinkl rnovacek
Comment 6 Gwyn Ciesla 2015-01-08 08:48:28 EST
Git done (by process-git-requests).

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