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 bugzilla.redhat.com.
Bug 177567
Summary: | Review Request: smb4k - The SMB/CIFS Share Browser for KDE | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Marcin Garski <mgarski> |
Component: | Package Review | Assignee: | Dawid Gajownik <gajownik> |
Status: | CLOSED NEXTRELEASE | QA Contact: | David Lawrence <dkl> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-extras-list, gajownik |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
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: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 163779 |
Description
Marcin Garski
2006-01-11 20:53:11 UTC
Hi! 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. http://manta.univ.gda.pl/~mgarski/fe/smb4k.spec http://manta.univ.gda.pl/~mgarski/fe/smb4k-0.6.5-2.src.rpm I have added kdebase-devel to BR. I'll think about â--disable-dependency-tracking --enable-finalâ, thanks for info. (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 â http://www.kadu.net/forum/viewtopic.php?t=6607 (in Polish ;] ) http://lists.kde.org/?l=kde-devel&m=99197154828408&w=2 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 `update-desktop-database' https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=177588#c3 (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/konqsidebar_smb4k.la 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/core -L/home/y4kk0/rpmbuild/BUILD/smb4k-0.6.5/smb4k/widgets". I'll try to investigate it further. http://manta.univ.gda.pl/~mgarski/fe/smb4k.spec http://manta.univ.gda.pl/~mgarski/fe/smb4k-0.6.5-3.src.rpm - Get rid of desktop-file-utils - Add --disable-dependency-tracking & --enable-final - Changed "/etc/profile.d/qt.sh" to "%{_sysconfdir}/profile.d/qt.sh" Still there is a bug: E: smb4k invalid-directory-reference /usr/lib64/kde3/konqsidebar_smb4k.la 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. (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. Thanks for explanation. --enable-final has caused this errors. (In reply to comment #4) > Still there is a bug: > E: smb4k invalid-directory-reference /usr/lib64/kde3/konqsidebar_smb4k.la I've been reading today Maikefile.am 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 ::" \ $RPM_BUILD_ROOT%{_libdir}/kde3/konqsidebar_smb4k.la sed -i -e "s:-L%{_builddir}/%{name}-%{version}/smb4k/widgets ::" \ $RPM_BUILD_ROOT%{_libdir}/kde3/konqsidebar_smb4k.la before execution of %find_lang macro. I see also in konqsidebar_smb4k.la references to /usr/lib/libsmb4kcore.la and /usr/lib/libsmb4kwidgets.la 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. That's me again ;-) I've forgot to mention, that BR libxml2 is not necessary (it's a dependency of kdebase). http://manta.univ.gda.pl/~mgarski/fe/smb4k.spec http://manta.univ.gda.pl/~mgarski/fe/smb4k-0.6.5-4.src.rpm - 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 rawhide. 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 --- Change %{_datadir}/apps/konqsidebartng/ to %{_datadir}/apps/konqsidebartng/add/smb4k_add.desktop because: [y4kk0@X tmp]$ rpm -qf /usr/share/apps/konqsidebartng/add/ kdebase-3.5.0-2 [y4kk0@X tmp]$ This also applies to: %{_datadir}/icons/crystalsvg/* [y4kk0@X tmp]$ rpm -qf /usr/share/icons/crystalsvg/16x16 kdelibs-3.5.0-5 [y4kk0@X tmp]$ This should be changed to: %{_datadir}/icons/crystalsvg/*/apps/smb4k.png Package is approved but you must correct these two glitches. Thanks for approving it. I'll fix this bugs. - Fix GCC warnings - Don't own KDE directories Set blocker to FE-ACCEPT (please do so when you approve a package). |