Bug 735128 - (libkdeedu) Review Request: libkdeedu - Libraries used by KDE Education applications
Review Request: libkdeedu - Libraries used by KDE Education applications
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Jaroslav Reznik
Fedora Extras Quality Assurance
:
Depends On:
Blocks: kde-reviews marble blinken cantor 735132 kalzium kanagram kbruch kgeography khangman kig kiten klettres kmplot kstars ktouch kturtle kwordquiz parley rocs step
  Show dependency treegraph
 
Reported: 2011-09-01 11:15 EDT by Rex Dieter
Modified: 2011-09-26 09:16 EDT (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2011-09-15 11:21:46 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
jreznik: fedora‑review+
limburgher: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Rex Dieter 2011-09-01 11:15:50 EDT
Spec URL: http://rdieter.fedorapeople.org/rpms/kdeedu/libkdeedu.spec
SRPM URL: http://rdieter.fedorapeople.org/rpms/kdeedu/libkdeedu-4.7.0-10.fc15.src.rpm
Description: Libraries used by KDE Education applications
Comment 1 Jaroslav Reznik 2011-09-07 08:58:48 EDT
Name: ok
Summary: ok
License: ok, thanks for info in spec
URL: ok
Source: ok (md5sum 10e59e1b68ffa940637a16369fef88da)
BRs/Reqs: ok, just kdelibs needed

This package was split, replaces kdeedu-libs, conflicts are ok. Package does not provide kdeedu-libs but I prefer not to as the whole kdeedu package split is done in one batch and these are all packages that requires libkdeedu. So should Provides be added?

Summary: ok
Macros used consistently: ok
ldconfig: ok
Icon cache: ok
Desktop file validations: NA
Docs: ok

Devel: contains static library libqtmmlwidget! According to package guidelines it has to be in standalone -static package.

rpmlint libkdeedu-4.7.0-10.fc15.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

Package builds, part of unofficial kde 47 f15 repo already.
Comment 2 Rex Dieter 2011-09-14 14:56:58 EDT
Re: kdeedu-libs
the kdeedu noarch metapackage in kde47 includes:
Obsoletes: kdeedu-libs < 4.7.0-10

Re: libqtmmlwidget.a
we can either make it standalone, or simply add
Provides: libkddeedu-static
then any pkg that uses it will need to change to
BuildRequires: libkdeedu-static
accordingly too

Any help to identify libqtmmlwidget.a using apps would be appreciated, I suppose splitting it out *would* help identify those... ie, what ftbfs after the split
Comment 3 Rex Dieter 2011-09-14 15:10:57 EDT
Spec URL: http://rdieter.fedorapeople.org/rpms/kdeedu/libkdeedu.spec
SRPM URL:
http://rdieter.fedorapeople.org/rpms/kdeedu/libkdeedu-4.7.0-11.fc15.src.rpm

%changelog
* Wed Sep 14 2011 Rex Dieter <rdieter@fedoraproject.org> 4.7.0-11
- -static subpkg for libqtmmlwidget
Comment 4 Jaroslav Reznik 2011-09-15 10:54:47 EDT
Thanks Rex,
approved.
Comment 5 Rex Dieter 2011-09-15 11:02:32 EDT
New Package SCM Request
=======================
Package Name: libkdeedu
Short Description: Libraries used by KDE Education applications
Owners: than jreznik ltinkl rnovacek rdieter kkofler
Branches: f16
Comment 6 Jon Ciesla 2011-09-15 11:16:36 EDT
Git done (by process-git-requests).
Comment 7 Rex Dieter 2011-09-15 11:21:46 EDT
imported.

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