Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 271777 - Running desklet-ImageSlideShow-0.8 in a multi-user environment with linked-in pictures
Summary: Running desklet-ImageSlideShow-0.8 in a multi-user environment with linked-in...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: gDesklets packages
URL: http://gdesklets.de/index.php?q=contr...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-05-29 14:20 UTC by Peter GAAL
Modified: 2009-09-10 02:21 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
Patch for desklet-ImageSlideShow-0.8 (ImageSlideShow.patch,907 bytes, patch)
2009-05-29 14:22 UTC, Peter GAAL
Details | Diff
Patch for the ebuild (directorywalk.diff,442 bytes, patch)
2009-05-29 14:24 UTC, Peter GAAL
Details | Diff
Patch for the ebuild (desklet-ImageSlideShow-0.8-r11.ebuild.diff,444 bytes, patch)
2009-05-29 14:33 UTC, Peter GAAL
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Peter GAAL 2009-05-29 14:20:33 UTC
+++ This bug was initially created as a clone of Bug #231864 +++

This bug relates to two small problems of desklet-ImageSlideShow-0.8:
-- The current version doesn't work well in a multi-user environment, as explained in the parent bug;
-- It cannot deal with linked-in pictures, e.g. those lying on another partition.
Comment 1 Peter GAAL 2009-05-29 14:22:54 UTC
Created attachment 192890 [details, diff]
Patch for desklet-ImageSlideShow-0.8

Changes:
Patching away the single-user & non-link limitations in python code.
Comment 2 Peter GAAL 2009-05-29 14:24:42 UTC
Created attachment 192893 [details, diff]
Patch for the ebuild

It's not enough to patch in the private directory. The temporary "homedir" must also be prepared before installing the desklet, otherwise emerge would display an unnecessary warning.
Comment 3 Peter GAAL 2009-05-29 14:33:05 UTC
Created attachment 192895 [details, diff]
Patch for the ebuild

(In reply to comment #2)
> Created an attachment (id=192893) [edit]
> Patch for the ebuild
> 
Sorry, this one was for the developper.

> It's not enough to patch in the private directory. The temporary "homedir" must
> also be prepared before installing the desklet, otherwise emerge would display
> an unnecessary warning.
>
This is still true, of course :-)
>
Comment 4 Joe Sapp (RETIRED) gentoo-dev 2009-09-10 02:21:44 UTC
Patch is in CVS.  I changed os.mkdir() to os.makedirs() to avoid the src_install() nastiness.  Please test it out :)

Thanks for all your work on this.  Re-open if I screwed up.