Bug 451655 - Review Request: kdeplasmoids - Additional plasmoids for KDE
Review Request: kdeplasmoids - Additional plasmoids for KDE
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Lorenzo Villani
Fedora Extras Quality Assurance
:
Depends On: 451505
Blocks: KDE41
  Show dependency treegraph
 
Reported: 2008-06-16 10:15 EDT by Rex Dieter
Modified: 2008-07-22 12:32 EDT (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2008-07-22 12:32:13 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
lorenzo: fedora‑review+
kevin: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Rex Dieter 2008-06-16 10:15:35 EDT
Spec URL: http://rdieter.fedorapeople.org/kdeplasmoids/kdeplasmoids.spec
SRPM URL: http://rdieter.fedorapeople.org/kdeplasmoids/kdeplasmoids-4.0.82-1.fc10.src.rpm
Description:
Additional plasmoids for KDE.

Upstream renamed extragear-plasma -> kdeplasmoids

No scratch build (yet), depends on kde-4.0.82, which requires phonon (pending review, bug #451505)
Comment 1 Lorenzo Villani 2008-06-18 10:23:00 EDT
Package builds fine but:

- Missing ldconfig call in %post and %postun

The package seems fine, if you fix these things at import time I can approve it
Comment 2 Rex Dieter 2008-06-18 10:27:21 EDT
why is ldconfig needed?  I don't see any shlibs here?
Comment 3 Rex Dieter 2008-06-18 10:27:47 EDT
nevermind, I'm blind, ok, I'll add that.
Comment 4 Lorenzo Villani 2008-06-18 11:16:49 EDT
Ok to do an import-time fix. Approved.
Comment 5 Rex Dieter 2008-06-18 11:19:01 EDT
New Package CVS Request
=======================
Package Name: kdeplasmoids
Short Description: Additional plasmoids for KDE
Owners: rdieter
Branches: F-9
InitialCC:
Cvsextras Commits: yes
Comment 6 Jason Tibbitts 2008-06-18 22:52:14 EDT
CVS done.
Comment 7 Rex Dieter 2008-07-17 09:31:28 EDT
Blarg, upstream renamed this module (again!), now kdeplasma-addons (with
assurances this won't happen again any time soon).

Can we get this either renamed, or another new cvs module created?

New Package CVS Request
=======================
Package Name: kdeplasma-addons
Short Description: Additional plasmoids for KDE
Owners: rdieter
Branches: F-9
InitialCC:
Cvsextras Commits: yes
Comment 8 Kevin Fenzi 2008-07-17 14:30:59 EDT
cvs done. 

Can you follow the package end of life for the old name?

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