Bug 1493937 (plasma-drkonqi) - Review Request: plasma-drkonqi - crash handler for KF5/Plasma5
Summary: Review Request: plasma-drkonqi - crash handler for KF5/Plasma5
Keywords:
Status: CLOSED RAWHIDE
Alias: plasma-drkonqi
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Jan Grulich
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: kde-reviews
TreeView+ depends on / blocked
 
Reported: 2017-09-21 07:57 UTC by Martin Kyral
Modified: 2017-10-17 14:10 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-10-17 14:10:18 UTC
Type: Bug
Embargoed:
jgrulich: fedora-review+


Attachments (Terms of Use)

Description Martin Kyral 2017-09-21 07:57:09 UTC
Description of problem:
Spec URL: http://copr-dist-git.fedorainfracloud.org/cgit/mkyral/plasma-unstable/plasma-drkonqi.git/tree/plasma-drkonqi.spec
SRPM URL: https://copr-be.cloud.fedoraproject.org/results/mkyral/plasma-unstable/fedora-26-x86_64/00604336-plasma-drkonqi/plasma-drkonqi-5.10.95-0.1.fc26.src.rpm
Description: DrKonqi crash handler for KF5/Plasma5
Fedora Account System Username: mkyral

Version-Release number of selected component (if applicable):
5.10.95


Additional info:
Drkonqi has been split from plasma-workspace into a component of its own in Plasma 5.11.

To be imported in rawhide together with the rest of Plasma 5.11 (beta or final)

Comment 1 Jan Grulich 2017-10-03 08:47:57 UTC
1) URL for SRPM doesn't work
2) I think that this is useless:
Obsoletes: plasma-workspace < 5.4.2-2
3) Missing to install installdbgsymbols_fedora.sh to libexec dir, which we have installed together with plasma-workspace-drkonqi
4) You don't need to run /sbin/ldconfig

Comment 2 Kevin Kofler 2017-10-03 10:19:14 UTC
The Obsoletes: plasma-workspace-drkonqi should also be versioned, something like:
Obsoletes: plasma-workspace-drkonqi < 5.10.95

Comment 3 Kevin Kofler 2017-10-03 10:21:44 UTC
And you may also want to add a:
Provides: plasma-workspace-drkonqi = %{version}-%{release}
so that people trying to install the old package name will get the new package.

Comment 5 Jan Grulich 2017-10-11 09:41:14 UTC
Looks good now. All issues have been fixed. Approved.

Comment 6 Gwyn Ciesla 2017-10-12 13:48:57 UTC
(fedrepo-req-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/plasma-drkonqi

Comment 7 Martin Kyral 2017-10-17 10:07:07 UTC
plasma-drkonqi landed in rawhide. Time to close the review bug?

Comment 8 Rex Dieter 2017-10-17 14:10:18 UTC
yep, closing


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