Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at

Bug 177567

Summary: Review Request: smb4k - The SMB/CIFS Share Browser for KDE
Product: [Fedora] Fedora Reporter: Marcin Garski <mgarski>
Component: Package ReviewAssignee: Dawid Gajownik <gajownik>
Status: CLOSED NEXTRELEASE QA Contact: David Lawrence <dkl>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-extras-list, gajownik
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-02-01 21:01:32 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 163779    

Description Marcin Garski 2006-01-11 20:53:11 UTC
Spec Name or Url:
SRPM Name or Url:
Smb4K is an SMB/CIFS share browser for KDE. It uses the Samba software suite to
access the SMB/CIFS shares of the local network neighborhood. Its purpose is to
provide a program that's easy to use and has as many features as possible.

Comment 1 Dawid Gajownik 2006-01-11 23:26:56 UTC

I did not have time to take a closer look at this package but I found two problems:
- Missing builRequires: kdebase-devel. Without it compilation fails:

smb4k_konqplugin.h:28:31: error: konqsidebarplugin.h: No such file or directory

- âBuildRequires:	kdelibs-devel >= 3.2.0â -- it will not work as expected in
Fedora because kdelibs has âEpochâ tag set to â6â. You can change it to:

BuildRequires: kdelibs-devel >= 6:3.2

or remove â>= 6:3.2â part completely. FE is only for FC3+ so it's always true.

BTW You may want to add â--disable-dependency-tracking --enable-finalâ to the
configure script (I see that Rex Dieter adds these flags to his KDE packages).

If no one will be willing to make a review, I can do that at the weekend.

Comment 2 Marcin Garski 2006-01-12 12:19:16 UTC

I have added kdebase-devel to BR. I'll think about
â--disable-dependency-tracking --enable-finalâ, thanks for info.

Comment 3 Dawid Gajownik 2006-01-14 17:44:56 UTC
(In reply to comment #2)
> I have added kdebase-devel to BR. I'll think about
> â--disable-dependency-tracking --enable-finalâ, thanks for info.

I found some more info about these options â (in Polish ;] )
These options are useful for packagers but not for the developers so it would be
nice to turn them on :D

Things that need some work:
- *.desktop file does not have âMimeTypeâ tag so you don't have to run
(please don't forget to remove unneeded Requires)
- you may want to change â/etc/profile.d/qt.shâ to â%{_sysconfdir}/profile.d/qt.shâ
- broken libtool archive:

[y4kk0@X i386]$ rpmlint smb4k-0.6.5-2.i386.rpm
E: smb4k invalid-directory-reference /usr/lib/kde3/
W: smb4k dangling-relative-symlink /usr/share/doc/HTML/en/smb4k/common ../common
[y4kk0@X i386]$

For example on my machine âdependency_libsâ variable from that file contains
-L/home/y4kk0/rpmbuild/BUILD/smb4k-0.6.5/smb4k/widgets". I'll try to investigate
it further.

Comment 4 Marcin Garski 2006-01-15 14:11:52 UTC

- Get rid of desktop-file-utils
- Add --disable-dependency-tracking & --enable-final
- Changed "/etc/profile.d/" to "%{_sysconfdir}/profile.d/"

Still there is a bug:
E: smb4k invalid-directory-reference /usr/lib64/kde3/

Comment 5 Marcin Garski 2006-01-16 12:06:32 UTC
It's strange. I've added --disable-dependency-tracking & --enable-final to
Krusader's configure (in devel tree) and compilation failed with errors like:
/usr/include/stdlib.h:626: error: ambiguates old declaration 'void abort()'
/usr/lib/qt-3.3/include/qprinter.h:151: error: non-member function 'bool
aborted()' cannot have cv-qualifier
/usr/lib/qt-3.3/include/qprinter.h:155: error: non-member function 'PaperSource
paperSource()' cannot have cv-qualifier
/usr/lib/qt-3.3/include/qprinter.h:156: error: virtual outside class declaration

I've removed --disable-dependency-tracking, the same errors occur, then remove
--enable-final and compilation went fine, I didn't checked what will happen with
--disable-dependency-tracking only.

Comment 6 Ralf Corsepius 2006-01-16 14:15:24 UTC
(In reply to comment #5)
> I didn't checked what will happen with
> --disable-dependency-tracking only.

--disable-dependency-tracking has nothing to do with your problem and must not
have any effect on your problems above. If it has, the package is broken.

All it does it is to let the compilation rules being used inside of Makefiles
not to use rules to generated source file dependencies. This speeds up building
significantly during one time "configure/build/install" runs like those inside
of a build system.

Comment 7 Marcin Garski 2006-01-16 21:50:58 UTC
Thanks for explanation. --enable-final has caused this errors.

Comment 8 Dawid Gajownik 2006-01-17 23:51:39 UTC
(In reply to comment #4)
> Still there is a bug:
> E: smb4k invalid-directory-reference /usr/lib64/kde3/

I've been reading today files but I'm still a newbie in an
Autotools world :/ The only thing I can propose right now is adding these lines:

# ugly workaround for broken libtool archive
sed -i -e "s:-L%{_builddir}/%{name}-%{version}/smb4k/core ::" \
sed -i -e "s:-L%{_builddir}/%{name}-%{version}/smb4k/widgets ::" \

before execution of %find_lang macro. I see also in
references to /usr/lib/ and /usr/lib/ in
dependency_libs field. I'm not shure whether they can be removed (Konqueror
plugin works without them fine, though). Someone more knowledgeable should
answer this question.

Yuck, libtool archives make me sick... ;-)

(In reply to comment #5)
> It's strange. I've added --disable-dependency-tracking & --enable-final to
> Krusader's configure (in devel tree) and compilation failed

I've been tesing these options with smb4k on my Rawhide box and it compiles fine.

Comment 9 Dawid Gajownik 2006-01-18 13:31:42 UTC
That's me again ;-) I've forgot to mention, that BR libxml2 is not necessary
(it's a dependency of kdebase).

Comment 10 Marcin Garski 2006-01-18 19:21:33 UTC

- Remove libxml2 from BR
- Add workaround for broken libtool archive (made by Dawid Gajownik)

Thanks for workaround, I'm also autotools newbie.

I'm compile smb4k with this two options on my FC4 box and it's also went fine.
But setting --enable-final for digikam and krusader caused strange errors on

Comment 11 Dawid Gajownik 2006-01-30 22:58:12 UTC
Here's the review:

- rpmlint errors:

[y4kk0@X i386]$ rpmlint smb4k-*
W: smb4k dangling-relative-symlink /usr/share/doc/HTML/en/smb4k/common ../common
W: smb4k-devel no-documentation
[y4kk0@X i386]$

They can be disregarded.
- proper licence (GPL)
- source matches upstream

--- Things that should be fixed ---





[y4kk0@X tmp]$ rpm -qf /usr/share/apps/konqsidebartng/add/
[y4kk0@X tmp]$

This also applies to:


[y4kk0@X tmp]$ rpm -qf /usr/share/icons/crystalsvg/16x16
[y4kk0@X tmp]$

This should be changed to:


Package is approved but you must correct these two glitches.

Comment 12 Marcin Garski 2006-01-30 23:47:50 UTC
Thanks for approving it. I'll fix this bugs.

Comment 13 Marcin Garski 2006-02-01 21:01:32 UTC
- Fix GCC warnings
- Don't own KDE directories

Comment 14 Christian Iseli 2006-03-29 14:17:50 UTC
Set blocker to FE-ACCEPT (please do so when you approve a package).