Currently can't get a connection to this server.

(In reply to comment #1)
> Currently can't get a connection to this server.

Please try again, should be o.k. now

Whilst this bugzilla entry includes spec files for qalculate-gtk and
qalculate-kde, the SRPMs are both for qalculate-gtk. Please raise a separate
bugzilla entry for qalculate-kde since it's actually a separate package, then
remove the reference to qalculate-kde from the title of this entry. The new bug
should depend on bug 169971, like this one does. The two packages can then be
reviewed and approved separately.

(In reply to comment #3)

Following your suggestion, I have created a new entry for qalculate-kde (bug
#170296), and change this to be solely for qalculate-gtk. The spec and srpm
files are still here;
Spec Name or Url: ftp://czar.eas.yorku.ca/pub/qalculate/qalculate-gtk.spec
SRPM Name or Url:
ftp://czar.eas.yorku.ca/pub/qalculate/qalculate-gtk-1.8.1-1.src.rpm

Created attachment 119801 [details]
Patch addressing some of the issues below

Just a first pass, not a review:

* Upstream version 0.8.2 is now available

* Redundant buildreqs:
- pkgconfig is a dep of glib2-devel
- glib2-devel is a dep of gtk2-devel
- gtk2-devel and libxml2-devel are deps of libglade2-devel

* configure script looks for libgnome-devel and uses it if available, so it
  should be added as a buildreq

* Availability of gnome-vfs means that wget shouldn't be needed (gnomevfs-copy
  is used in preference)

* Why the buildreq of ImageMagick? It seems to build just fine without it

* Is LDFLAGS='-export-dynamic' really needed? This linker option appears to be
  set automatically with libgnome-devel as a buildreq.

* I can't get the help to work; yelp complains about null references. Does it
  work for you?

Package have been updated to the new released version, and desktop file installe.
ftp://czar.eas.yorku.ca/pub/qalculate/qalculate-gtk.spec
ftp://czar.eas.yorku.ca/pub/qalculate/qalculate-gtk-1.8.2-1.src.rpm.

Paul,
LDFLAGS='-export-dynamic' used to be needed for the version of libglade in
rawhide. It's now included in the configuration option upstream, so I've dropped
it. Also just noticed yelp complaint, it's there in the updated version too. I
honestly don't know what to do about that. I'll try nd concact the author.

I'm just now noticing Paul's patch to clean up the spec. I'll work on it later
today. 

Updated package with Paul Howarth's changes.
ftp://czar.eas.yorku.ca/pub/qalculate/qalculate-gtk.spec
ftp://czar.eas.yorku.ca/pub/qalculate/qalculate-gtk-1.8.2-2.src.rpm

URL for SRPM is actually:
ftp://czar.eas.yorku.ca/pub/qalculate/qalculate-gtk-0.8.2-2.src.rpm

Explicit Requires: for gnome-vfs2 & libqalculate are not needed because RPM will
add the library dependencies automatically.

The files list still includes:
%{_datadir}/qalculate-gtk/glade
which should be:
%{_datadir}/qalculate-gtk/
otherwise the %{_datadir}/qalculate-gtk directory will be left by RPM if the
package is removed.

Have you had any feedback regarding the yelp issue?

(In reply to comment #9)
New files with corrections here;
ftp://czar.eas.yorku.ca/pub/qalculate/qalculate-gtk.spec
ftp://czar.eas.yorku.ca/pub/qalculate/qalculate-gtk-0.8.2-3.src.rpm

> 
> Have you had any feedback regarding the yelp issue?

I've submitted a bug report for it at the project's sourceforge site, no
response yet.

(In reply to comment #10)

> > 
> > Have you had any feedback regarding the yelp issue?
> 
> I've submitted a bug report for it at the project's sourceforge site, no
> response yet.
> 
Spoke too soon, bug has been fixed. New package with the fix uploaded
ftp://czar.eas.yorku.ca/pub/qalculate/qalculate-gtk.spec
ftp://czar.eas.yorku.ca/pub/qalculate/qalculate-gtk-0.8.2-4.src.rpm

Review:

- rpmlint clean
- package and spec naming OK
- package meets guidelines
- license is GPL, matches spec, text included
- spec file written in English and is legible
- sources match upstream
- package builds OK in mock for FC4 (i386)
- BR's OK
- %find_lang used for locale data
- no libraries, pkgconfigs or subpackages to worry about
- not relocatable
- no directory ownership or permissions problems
- no duplicate files
- %clean section present and correct
- macro usage is consistent
- code, not content
- documentation not excessively large
- docs don't affect runtime, other than online help not working if --excludedocs
is used
- desktop file installed properly
- package appears to function correctly
- scriptlets for scrollkeeper-update appear sane
- all previously-mentioned issues addressed

Approved.