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 166980
Summary: | Review Request: kmymoney2: Personal finance | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Rex Dieter <rdieter> | ||||
Component: | Package Review | Assignee: | Quentin Spencer <qspencer> | ||||
Status: | CLOSED NEXTRELEASE | QA Contact: | David Lawrence <dkl> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | medium | ||||||
Version: | rawhide | CC: | dennis, dennis, fedora-extras-list, gauret, jamundso, qspencer | ||||
Target Milestone: | --- | ||||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
URL: | http://kmymoney2.sourceforge.net/ | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2005-11-10 20:16:33 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 | ||||||
Attachments: |
|
Description
Rex Dieter
2005-08-29 12:27:41 UTC
Oops, corrected URLS: Spec Name or Url: http://apt.kde-redhat.org/apt/kde-redhat/SPECS/kmymoney2-0.8-1.spec SRPM Name or Url: http://apt.kde-redhat.org/apt/kde-redhat/all/SRPMS.stable/kmymoney2-0.8-1.src.rpm This isn't a complete review, but rpmlint on the SRPM gave me: E: kmymoney2 invalid-spec-name kmymoney2-0.8-1.spec I think the spec file should not have the version in it. I tried building the file and got the following error: File not found by glob: /var/tmp/kmymoney2-0.8-1-root-qspencer/usr/share/applications/*/*.desktop Looking at the spec file, it looks like you have some optional build parameters. I'm not sure what the Fedora Extras policy is on things like this, but I think it would be preferred to just build with all options and simplify the spec file if possible. %changelog * Tue Aug 30 2005 Rex Dieter <rexdieter[AT]users.sf.net> 0.8-2 - simplify specfile - fix build Spec Name or Url: http://apt.kde-redhat.org/apt/kde-redhat/SPECS/kmymoney2-0.8-2.spec SRPM Name or Url: http://apt.kde-redhat.org/apt/kde-redhat/all/SRPMS.stable/kmymoney2-0.8-2.src.rpm OK, the updated version now builds, installs, and uninstalls on my system with no problems, and source matches upstream. No rpmlint problems with SRPM, there are some minor ones with the the RPMs: $ rpmlint kmymoney2-0.8-2.i386.rpm W: kmymoney2 dangling-symlink /usr/share/doc/HTML/en/kmymoney2/common /usr/share/doc/HTML/en/common W: kmymoney2 symlink-should-be-relative /usr/share/doc/HTML/en/kmymoney2/common /usr/share/doc/HTML/en/common $ rpmlint kmymoney2-devel-0.8-2.i386.rpm W: kmymoney2-devel no-provides kmymoney-devel W: kmymoney2-devel no-documentation I'm out of time right now. I'll look in more detail at the spec file later. RPM build errors: Installed (but unpackaged) file(s) found: /usr/lib/kde3/kmm_ofximport.la /usr/lib/kde3/kmm_ofximport.so /usr/share/services/kmm_ofximport.desktop With these installed, --enable-ofxplugin actually works... [root@elm ~]# rpm -qa libofx\* openjade\* libofx-0.7.0-3 openjade-1.3.2-16 libofx-devel-0.7.0-3 openjade-devel-1.3.2-16 Created attachment 118330 [details]
full gzip'd output of rpmbuild
I didn't see those missing files, but we didn't use BuildRequires: libofx-devel I would have expected configure to die if I gave --enable-ofxplugin and libofx wasn't found. %changelog * Thu Sep 01 2005 Rex Dieter <rexdieter[AT]users.sf.net> 0.8-3 - BR: libofx-devel Spec Name or Url: http://apt.kde-redhat.org/apt/kde-redhat/SPECS/kmymoney2-0.8-3.spec SRPM Name or Url: http://apt.kde-redhat.org/apt/kde-redhat/all/SRPMS.stable/kmymoney2-0.8-3.src.rpm build fails on x86_64 checking for Qt... configure: error: Qt (>= Qt 3.0) (library qt-mt) not found. Please check your installation! For more details about this problem, look at the end of config.log. Make sure that you have compiled Qt with thread support! error: Bad exit status from /var/tmp/rpm-tmp.21811 (%build) RPM build errors: Bad exit status from /var/tmp/rpm-tmp.21811 (%build) to fix add to %build unset QTDIR || : ; . /etc/profile.d/qt.sh export QTLIB=${QTDIR}/lib QTINC=${QTDIR}/include for some reason fc4 x86_64 doesnt pick up the qt stuff properly I dont know if for extras the kde-redhat stuff shouldnt be removed from the spec file. on x86 build is ok. going to build on sparc and ppc later today. not that sparc has any effect on inclusion in extras i just like it :). Setting this bug as "under review". Feel free to change the assignment, I just used the last commentor * Fri Oct 21 2005 Rex Dieter <rexdieter[AT]users.sf.net> 0.8-4 - %%post,%%postun: gtk-update-icon-cache, update-desktop-database - omit lib*.la - x86_64: set QTDIR/QTLIB Spec Name or Url: http://apt.kde-redhat.org/apt/kde-redhat/SPECS/kmymoney2-0.8-4.spec SRPM Name or Url: http://apt.kde-redhat.org/apt/kde-redhat/all/SRPMS.stable/kmymoney2-0.8-4.src.rpm Spec Name or Url: http://apt.kde-redhat.org/apt/kde-redhat/SPECS/kmymoney2-0.8.1-1.spec SRPM Name or Url: http://apt.kde-redhat.org/apt/kde-redhat/all/SRPMS.stable/kmymoney2-0.8.1-1.src.rpm %changelog * Sat Nov 05 2005 Rex Dieter <rexdieter[AT]users.sf.net> 0.8.1-1 - 0.8.1 * Fri Nov 04 2005 Rex Dieter <rexdieter[AT]users.sf.net> 0.8-5 - drop useless macros - use %%fedora,%%rhel to conditionalize ofxbanking support I have built the latest version with no problems, and everything installs and uninstalls fine. Output of rpmlint on kmymoney2: W: kmymoney2 dangling-symlink /usr/share/doc/HTML/en/kmymoney2/common /usr/share/doc/HTML/en/common W: kmymoney2 symlink-should-be-relative /usr/share/doc/HTML/en/kmymoney2/common /usr/share/doc/HTML/en/common Output of rpmlint on kmymoney2-devel: W: kmymoney2-devel no-provides kmymoney-devel W: kmymoney2-devel no-documentation I'm not sure why rpmlint thinks that the lack of providing kmymoney-devel is a problem, I don't see it as a blocker. There are a lot of macros in the spec file, as noted by a previous reviewer, which seem to be for kde-redhat. I don't know if that's necessarily a problem, but if any of them are not used in FE, I think some comments should be added to explain what they do and why they are not used. For example, what does _with_kbanking do, and why is it not enabled (or at least it appears not to be)? The others are pretty clear, I think. I would also suggest a more concise package description, though I wouldn't consider that a blocker. > There are a lot of macros in the spec file, as noted by a previous reviewer, > which seem to be for kde-redhat All of those were dropped since 0.8-5 (as noted in the changelog) > For example, what does _with_kbanking do, and why is it not enabled (or at > least it appears not to be)? Not enabled due to missing dependancies: kbanking-devel (and a newer acqbanking rpm). > I would also suggest a more concise package description, > though I wouldn't consider that a blocker. Agreed, I'll trim that and release an update in a bit. Spec Name or Url: http://apt.kde-redhat.org/apt/kde-redhat/SPECS/kmymoney2-0.8.1-2.spec SRPM Name or Url: http://apt.kde-redhat.org/apt/kde-redhat/all/SRPMS.stable/kmymoney2-0.8.1-2.src.rpm %changelog * Wed Nov 09 2005 Rex Dieter <rexdieter[AT]users.sf.net> 0.8.1-2 - qt-3.3.5 patch - 'make check' workaround (kde bug #115863) - trim %%description I looked at the rpmlint warnings again. I now see that the dangling symlink is unavoidable since it apparently links to external kde stuff. However, the non-relative symlink is still there. It appears you tried to change it in the specfile but it didn't work quite right for me. Take a look at the change below and see if that's what you need to make it go away. --- kmymoney2-0.8.1-2.spec 2005-11-09 14:24:21.000000000 -0600 +++ kmymoney2.spec 2005-11-09 12:01:42.000000000 -0600 @@ -87,8 +87,8 @@ for lang in *; do if [ -d $lang ]; then pushd $lang - for i in */*; do - [ -d $i -a -L $i/common ] && rm -f $i/common && ln -sf ../../common $i/common + for i in *; do + [ -d $i -a -L $i/common ] && rm -f $i/common && ln -sf ../common $i/common done popd fi Thanks. Ugh.. I just copy-n-pasted that code from one of the other packages I maintain, so it looks like a few others are probably broken the same way. Spec Name or Url: http://apt.kde-redhat.org/apt/kde-redhat/SPECS/kmymoney2-0.8.1-3.spec SRPM Name or Url: http://apt.kde-redhat.org/apt/kde-redhat/all/SRPMS.stable/kmymoney2-0.8.1-3.src.rpm %changelog * Thu Nov 10 2005 Rex Dieter <rexdieter[AT]users.sf.net> 0.8.1-3 - fix relative symlinks Well, I think this package is ready, but I'm not the assignee. If someone knows how to reassign this to me, I'll approve it. Sorry that i havent had the time to give this a full review. but I really appreciate that someone has had that time. -License is OK. -Spec file appears to meet all the necessary criteria. -Builds and installs on i386 (note that other testers reported build problems on x86_64 with earlier versions, but it is unknown whether the problems persist with the newest update). rpmlint output on main package W: kmymoney2 dangling-relative-symlink /usr/share/doc/HTML/en/kmymoney2/common ../common This is OK, it refers to an external KDE directory that will be present on the system. rpmlint output on devel package: W: kmymoney2-devel no-provides kmymoney-devel W: kmymoney2-devel no-documentation I think the first warning can be ignored. I can't figure out the reason for the second one, as there are other devel packages that don't include a %doc section in the files list and don't produce this error, so for now, I say ignore it. Approved. |