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 166207 - Review Request: rekall : A KDE database front-end application
Summary: Review Request: rekall : A KDE database front-end application
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Aurelien Bompard
QA Contact: David Lawrence
URL: http://www.rekallrevealed.org/
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2005-08-17 20:57 UTC by Tom "spot" Callaway
Modified: 2007-11-30 22:11 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-02-12 11:08:07 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Tom "spot" Callaway 2005-08-17 20:57:34 UTC
Spec Name or Url: 
http://www.auroralinux.org/people/spot/review/rekall.spec

SRPM Name or Url:
http://www.auroralinux.org/people/spot/review/rekall-2.2.3-1.src.rpm

Description: 

Rekall is a database front-end, somewhat in the style of MS Access(tm).
However, Rekall is not itself a database, and does not include a
database. By this we mean that data is stored somewhere else in an SQL
server, and Rekall is fundementally just a tool to extract, display
and update that data (of course, it does lots more than that, it does
forms and reports and scripting...). It is database agnostic, and does
not have any preferred database (plugins are available for MySQL and
PostgreSQL).

Comment 1 Aurelien Bompard 2005-09-02 16:04:10 UTC
* BuildRequires: desktop-file-utils is missing
* rpmlint says: zero-length /usr/share/apps/rekall/stock/component/py/dummy.
Maybe it's justified, since it's a dummy file
* rpmlint of rekall-runtime says: library-without-ldconfig (missing ldconfig in
post and postun for this subpackage), devel-file-in-non-devel-package
/usr/lib/librek{all,base}rt.so. Maybe it's not necessary to make a
rekall-runtime-devel subpackage, but in this case please have rekall-runtime
provide it.
* Typo in rekall-runtime %description: s/design functions or removed/design
functions are removed/g
* --vendor in desktop files should be fedora, not kde
* in the desktop file: missing "Application" category. I would also add the
"Office" category.
* when creating a new database with the xbase driver, I get the message :
« xbase: cannot load driver. Library files for "libkbase_driver_xbase.la" not
found in paths. ». Same thing for mysql driver and pgsql. It looks like rekall
needs those *.la files.
* rekall apparently uses .rlk files. It could be useful to associate those files
with rekall, using a mime entry in the desktop file (--add-mime-type switch) and
running update-desktop-database in %post.
* the desktop entry does not have an icon. Symlinking
%{_datadir}/apps/rekall/icons/hicolor/32x32/apps/rekall.png to
%{_datadir}/pixmaps should do the trick.

Comment 2 Tom "spot" Callaway 2005-09-04 21:13:47 UTC
All of these issues should be covered in -2. I tested making an xbase/xbsql
database and it didn't throw any errors.

New SRPM: http://www.auroralinux.org/people/spot/review/rekall-2.2.3-2.src.rpm
New SPEC: http://www.auroralinux.org/people/spot/review/rekall.spec

Comment 3 Adrian Reber 2005-09-05 06:05:18 UTC
> * the desktop entry does not have an icon. Symlinking
> %{_datadir}/apps/rekall/icons/hicolor/32x32/apps/rekall.png to
> %{_datadir}/pixmaps should do the trick.

I am not really sure about this but I think the better way for the icon to
appear would be running gtk-update-icon-cache instead of creating this symlink.
This should, at least in gnome, show the icon in the menu.

Comment 4 Aurelien Bompard 2005-09-05 07:55:52 UTC
Yes, I'm not sure what the Right Way is, but IIRC KDE looks for its icons in
_datadir/pixmaps and in _datadir/icons/<theme>/<size>/apps, and not in
_datadir/apps/rekall anyway. I don't think there is such thing as
gtk-update-icon-cache for KDE.

Last problems I see :
* subpackage -runtime should also require desktop-file-utils (it uses
update-desktop-database in scriptlets)
* while making the icon symlink, please change /usr/share into %{datadir}
* you symlink rekall's icon, but you copy rekallrt's one. Please symlink it too
(or copy the symlink, not the icon)

Comment 5 Tom "spot" Callaway 2005-09-08 22:06:39 UTC
Fixed in -3:

- runtime now requires d-f-u
- use %{_datadir} everywhere

I need to copy the rekallrt icon, otherwise, rekall-runtime would have to
require the "rekall" package, and the whole point of the runtime is that it can
be distributed without the rest of rekall.

SRPM:
http://www.auroralinux.org/people/spot/review/rekall-2.2.3-3.src.rpm

SPEC:
http://www.auroralinux.org/people/spot/review/rekall.spec

Comment 6 Aurelien Bompard 2005-09-09 06:35:24 UTC
> I need to copy the rekallrt icon, otherwise, rekall-runtime would have to
> require the "rekall" package

Yes of course...

However, it seems that rekall-runtime is not really independant. Having no
rekall package installed, here's what I get:
$ sudo rpm -Uvh rekall-runtime-2.2.3-3.fc4.i386.rpm
error: Failed dependencies:
   libkbase_common.so.0 is needed by rekall-runtime-2.2.3-3.fc4.i386
   libkbase_kde.so.0 is needed by rekall-runtime-2.2.3-3.fc4.i386
   libkbase_tkwidgets.so.0 is needed by rekall-runtime-2.2.3-3.fc4.i386

It's been built in mock. And the "rekall" package provides these libs.


Comment 7 Tom "spot" Callaway 2005-09-17 22:48:28 UTC
-4 fixes this by creating a "rekall-common" subpackage, which both rekall and
rekall-runtime need to function. Tested and confirmed that it enables
rekall-runtime to work.

New SRPM: http://www.auroralinux.org/people/spot/review/rekall-2.2.3-4.src.rpm
New SPEC: http://www.auroralinux.org/people/spot/review/rekall.spec

Comment 8 Aurelien Bompard 2005-09-18 07:18:08 UTC
Review for release 4:
* RPM name is OK
* Source rekall-2.2.3-2.tar.gz is the same as upstream
* Builds fine in mock
* rpmlint looks OK
* File list looks OK
* rekall-runtime can be installed independently
* Works as expected

APPROVED. After import, you may want to remove the following duplicate
BuildRequires :
- xbase-devel (required by xbsql-devel)
- qt-devel (required by kdelibs-devel)


Comment 9 Aurelien Bompard 2006-02-12 11:08:07 UTC
Rekall is published, closing bug


Note You need to log in before you can comment on or make changes to this bug.