Bug 225957

Hi there, The following are the review items that stand out.  I realize 
that almost all of the includes are prefixed with "k3b" but nonetheless
it would be nice to put all of them in a subdir such as: /usr/include/k3b/

Anyway, the list of items is:

 - please use the preferred BuildRoot

 - rpmlint complains about a number of things:
   - devel content in non-devel package (many files)
     - please consider creating a sub-dir such as /usr/include/k3b
       to contain all the k3b headers
   - dead patches are still being carried around and should 
     probably be deleted:
W: k3b patch-not-applied Patch1: k3b-0.11.3-kde32.patch
W: k3b patch-not-applied Patch2: k3b-0.11.6-desktopfile.patch
W: k3b patch-not-applied Patch5: k3b-0.11.14-version.patch
W: k3b patch-not-applied Patch7: k3b-0.11.17-dao.patch
W: k3b patch-not-applied Patch6: k3b-0.11.14-suid.patch
W: k3b patch-not-applied Patch9: k3b-0.11.23-proxy.patch
W: k3b patch-not-applied Patch8: k3b-0.12.2-statfs.patch
   - please remove prereq-use /sbin/ldconfig
   - I don't really understand this--can someone else please 
     help explain it:
/tmp/k3b-0.12.17-1.i386.rpm.30099/usr/share/applications/kde-k3b.desktop:
warning: file contains key "DocPath", this key is currently reserved for use
within KDE, and should in the future KDE releases be prefixed by "X-"

 - please consider adding %{?dist} to Release

Bummer.  I hit "save changes" a little too fast.  :-)  There are a few 
other things that should have been listed:

 + Please see builds, logs, and rpmlint output at:
http://linux.dell.com/files/fedora/FixBuildRequires/mock-results-core/i386/k3b-0.12.17-1.src.rpm/result/

 - the /usr/lib/kde3/*.la files should be deleted

 - some of the libs (e.g. /usr/lib/*.so) also belong in a separate
   -devel sub-package

(In reply to comment #2)
>  - the /usr/lib/kde3/*.la files should be deleted
 
Please ensure, that K3B really doesn't need it, else you will have a vague of
bug report from users. Since KDE3 itself is heavily dependent on .la files.
http://bugs.kde.org/show_bug.cgi?id=139445

If you are removing those .la files, ensure proper testing has been carried out.

>  the /usr/lib/kde3/*.la files should be deleted
I can vouge that in many cases, kde apps (still) do need these .la files.  
These are of the type that cause no harm (vs. those in %_libdir), so I'd 
strongly recommend leaving them as-is.

Re:  DocPath in .desktop file
Mostly harmless warning.  

While on the topic of .desktop files, I'd recommend using this instead (to 
preserve upstream .desktop vendor):
desktop-file-install --vendor="" \
  --dir $RPM_BUILD_ROOT%{_datadir}/applications/kde \
  --add-category X-Red-Hat-Base \
  $RPM_BUILD_ROOT%{_datadir}/applications/kde/%{name}.desktop
(not sure the point of adding X-Red-Hat-Base, but a comment justifying it's 
use in the specfile would be helpful for posterity).

Once merge is complete, *this* package can/should:
Obsoletes: k3b-extras < %{version}-%{release}
Provides:  k3b-extras = %{version}-%{release}
BuildRequires:  libmpcdec-devel
BuildRequires:  libsndfile-devel
and add %configure options:  --with-musepack --with-sndfile

See also bug #229420

harold, is this your package?  Can we have some movement here?  
We really need to get the kde stack merged by early next week.

err... 
$ brew latest-pkg dist-fc7 k3b
Build                                     Tag                   Built by
----------------------------------------  --------------------  ----------------
k3b-1.0.0-0.rc6.1.fc7                     dist-fc7              harald

So, I take it then that you incorporated the Ed's suggestions already?

Hrm, looking closer, turns out besides .la files, Ed didn't have any
suggestions.  Ed, anything else blocking approval here?

Hi folks, my apologies for the slow response--have been quite busy with
${WORK}.

My two main suggestions were the removal of *.la files [which I now 
understand are needed by many KDE apps--sorry!], and the creation of 
a -devel subpackage.  I see that the -devel package was created on 
Feb 15th so thats good.

And I don't see anything else at odds with the packaging guidelines so 
its APPROVED.

Harald,
per comment #5, could you add the Obsoletes k3b-extras at least for now (even
before the BR's and Provides) so that we can have a clean upgrade path and
remove k3b-extras from the repos?

After the merge, you can do the rest.  Sound ok?

> After the merge, you can do the rest.  Sound ok?

No :) Need your experience and help :)

OK, let me rephrase... 
After the merge, if you don't mind, I can help co-maintain k3b, and then *we*
can do the rest.
Sound better? :)

Regardless, we'd like to get the Obsoletes in place asap (ie, in time for f7t4).

k3b-1.0-1.fc7

Hrm, we missed actually adding the BR's after the merge. :(

Harold, if you can add me as comaintainer (and add me to pkg.acl), I can help
out here.

Package Change Request
======================
Package Name: k3b
Updated Fedora Owners: harald,rdieter.edu

cvs done.

Closing, all is well in the world.