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 170691 - Review Request: basket - Notes taker for KDE
Summary: Review Request: basket - Notes taker for KDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Adrian Reber
QA Contact: David Lawrence
Depends On:
TreeView+ depends on / blocked
Reported: 2005-10-13 19:41 UTC by Aurelien Bompard
Modified: 2010-06-21 01:49 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2005-10-16 21:32:45 UTC
Type: ---
kevin: fedora-cvs+

Attachments (Terms of Use)

Description Aurelien Bompard 2005-10-13 19:41:02 UTC
Spec Name or Url:
SRPM Name or Url:
This application is mainly an all-purpose notes taker. It provides baskets
where any item can be dragged and dropped: text, formatted text, links,
images, sounds, files, colors, application launcher...
BasKet lets you keep all objects in one place, and keep data at hand.

Comment 1 Adrian Reber 2005-10-14 10:19:37 UTC
The package should own "/usr/share/doc/HTML/en/basket/"

And I cannot see the icon in the gnome menu. Even after a restart of the menu.
Can icons in the crystalsvg be used from the gnome panel?

Comment 2 Rex Dieter 2005-10-14 12:30:47 UTC
IMO, the icon missing is an upstream issue, in that basket should be installing
it's icon(s) in the hicolor theme, not crystalsvg.  crystalsvg inherits from
hicolor, so the quick-n-dirty fix would be to:

mv $RPM_BUILD_ROOT%{_datadir}/icons/cyrstalsvg \

and then adjust %post/%postun, to something like:

touch %{_datadir}/icons/hicolor ||:
gtk-update-icon-cache -q %{_datadir}/icons/hicolor 2>/dev/null ||:

if [ $1 -eq 0 ]; then
touch %{_datadir}/icons/hicolor ||:
gtk-update-icon-cache -q %{_datadir}/icons/hicolor 2>/dev/null ||:

Comment 3 Rex Dieter 2005-10-14 12:38:38 UTC
Interestingly, I just looked at /usr/share/icons/Bluecurve/index.theme, and it
so it *should* have inherited from cyrstalsvg too.  ???  Adrian, what icon theme
are you using?

However, I'd still recommend using hicolor instead of crystalsvg, since
all(most?) icon themes should (eventually) inherit from it (which isn't
necesarily true for crystalsvg).

Comment 4 Aurelien Bompard 2005-10-14 16:23:41 UTC
Thanks for the review.

* Fri Oct 14 2005 Aurelien Bompard <gauret[AT]> 0.5.0-2
- own the doc dir
- use hicolor instead of crystalsvg as icon theme
- add extension to the icon in the desktop file

I've change the icon entry in the desktop file from "basket" to "basket.png".
IIRC, GNOME needs the extension. Does the icon show up in GNOME now ?

Comment 5 Rex Dieter 2005-10-14 16:53:48 UTC
Having to add the .png is unfortunate, which now makes kde unable to use
basket's scalable svg icon:

Comment 6 Rex Dieter 2005-10-14 17:04:20 UTC
Just checked the icon spec,
you shouldn't need to add the .png extension.  Implementations are supposed to
automatically search for png, svg(optional), and xpm icons.

Comment 7 Rex Dieter 2005-10-14 17:08:09 UTC
You should also "touch" the top-level icon dir (to update it's mtime) per
comment #2, per the Icon spec (appended).

To be safe (especially if the icon theme dir ever changes), you can ignore my
if [ $1 -eq 0 ];
bit, so the touch and gtk-update-... runs unconditionally.

... any implementation that does caching is required to look at the mtime of the
toplevel icon directories when doing a cache lookup, unless it already did so
less than 5 seconds ago. This means that any icon editor or theme installation
program need only to change the mtime of the the toplevel directory where it
changed the theme to make sure that the new icons will eventually get used.

Comment 8 Aurelien Bompard 2005-10-14 21:31:45 UTC
I tried in GNOME with the .png extension, and it didn't change a thing, so I
removed the hack.

* Fri Oct 14 2005 Aurelien Bompard <gauret[AT]> 0.5.0-3
- don't add the extension to the icon, it's useless
- touch the icon dir in post and postun

Does the icon show up in GNOME ? At some point I tried a symlink to
/usr/share/pixmaps, and the icon appeared. Then I removed the symlink, and now
the icons stays (tried to restart GNOME, still there).
What's the preferred way ?

Comment 9 Aurelien Bompard 2005-10-14 21:32:28 UTC
Wrong URL, it should be release 3 :

Comment 10 Adrian Reber 2005-10-16 15:47:46 UTC
I can see the icon now.

* builds in mock
* clean installation and removal
* source matches upstream
* scripts look sane
* spec looks good
* rpmlint has two warnings:
W: basket dangling-symlink /usr/share/doc/HTML/en/basket/common
W: basket symlink-should-be-relative /usr/share/doc/HTML/en/basket/common
I have seen that other KDE related packages have the same warning with rpmlint
so I suppose this isn't a big problem. It doesn't also sound as if it is a big


Comment 11 Thomas Janssen 2010-06-18 17:08:23 UTC
Package Change Request
Package Name: basket
New Branches: EL-4 EL-5 EL-6
Owners: thomasj

Comment 12 manuel wolfshant 2010-06-18 22:39:19 UTC
Let's use a shorter route ( implicit approval for the below request change was given by Thomas via Email; I am removing EL-4 from the list because I can not perform tests which imply desktop-like software on this platform ):

Package Change Request
Package Name: basket
New Branches:EL-5 EL-6
Owners: thomasj    
InitialCC: wolfy

Comment 13 Thomas Janssen 2010-06-20 18:17:00 UTC
Yes, the above is fine with me.

Comment 14 Kevin Fenzi 2010-06-21 01:49:19 UTC
cvs done.

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