[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-03-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903


Luya Tshimbalanga  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-02-18 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #37 from Kevin Fenzi   2009-02-18 15:17:44 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-02-18 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903





--- Comment #36 from MERCIER   2009-02-18 06:04:02 
EDT ---
quote: mtas...@ioa.s.u-tokyo.ac.jp  2009-02-05 13:43:29 EDT   (-) [reply]
---

Well,
- This package itself is now good
- For your other review requests:
  - I suggets that gdesklets related packages should be renamed
to gdesklets-XX
(there are already gdesklets-goodweather, gdesklets-quote-of-the-day
 on Fedora. It seems that both of these were reviewed by me)
  - I suggest that earth-and-moon rpm should be renamed to
earth-and-moon-backgrounds (as other packages do).
You can check how other packages are renamed by
$ rpm -qf /usr/share/backgrounds/*

   (note: please post new srpms or your comments for this package
on the corresponding review request, not on "this" review
ticket)
___
mtasaka say this but in https://bugzilla.redhat.com/show_bug.cgi?id=481022 when
sponsorized me for another package

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-02-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903





--- Comment #35 from MERCIER   2009-02-17 18:23:52 
EDT ---
sorry for the waiting. the package is ready. i wait the cvs
thanks

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-02-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903


Luya Tshimbalanga  changed:

   What|Removed |Added

   Flag||fedora-cvs?




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-02-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903





--- Comment #34 from Pierre-YvesChibon   2009-02-17 
02:09:36 EDT ---
maybe you could be a bit more extensive, Jonhatan has just been approved, seing
some more review done is always interesting

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-02-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903


Luya Tshimbalanga  changed:

   What|Removed |Added

   Flag||fedora-review+




--- Comment #33 from Luya Tshimbalanga   2009-02-16 
20:47:58 EDT ---
Done, I oversight that fedora-review part so request for cvs can be proceeded.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-02-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903


Luya Tshimbalanga  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-02-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |




--- Comment #32 from Kevin Fenzi   2009-02-16 16:11:07 EDT ---
I don't see fedora-review + here, and it's not clear to me who is reviewing
this. 

Please wait until a reviewer marks it fedora-review + for requesting cvs.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-02-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903





--- Comment #31 from MERCIER   2009-02-14 14:44:50 
EDT ---
New Package CVS Request
===
Package Name: gdesklet-SlideShow
Short Description: A slideshow of collection for gdesklets
Owners: bioinfornat...@gmail.com
Branches: F9 F10
InitialCC: bioinfornatics

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-02-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903


MERCIER  changed:

   What|Removed |Added

   Flag||fedora-cvs?




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-02-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903


Mamoru Tasaka  changed:

   What|Removed |Added

 Blocks|177841  |




--- Comment #30 from Mamoru Tasaka   2009-02-06 
10:06:10 EDT ---
(Removing NEEDSPONSOR)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-02-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903





--- Comment #29 from MERCIER   2009-02-06 07:19:57 
EDT ---
I change the name of rpm SlideShow to gdesklet-SlideShow

rpmlint issue:
_

$ rpmlint rpmbuild/SPECS/gdesklet-SlideShow.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

$ rpmlint rpmbuild/SRPMS/gdesklet-SlideShow-0.9-5.fc10.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint rpmbuild/RPMS/noarch/gdesklet-SlideShow-0.9-5.fc10.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
_
http://bioinformatiques.free.fr/gdesklet-SlideShow-0.9-5.fc10.src.rpm

http://bioinformatiques.free.fr/gdesklet-SlideShow-0.9-5.fc10.noarch.rpm

http://bioinformatiques.free.fr/gdesklet-SlideShow.spec

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-02-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903





--- Comment #28 from Luya Tshimbalanga   2009-02-02 
05:45:31 EDT ---
(In reply to comment #27)
> where i can i find a sponsor?
> i have creat some package:
> Review Request: SPE - Stani's Python Editor:
> https://bugzilla.redhat.com/show_bug.cgi?id=481022

https://admin.fedoraproject.org/accounts/group/members/packager/*/sponsor
You should be able to see them with your Fedora account.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-02-02 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903





--- Comment #27 from MERCIER   2009-02-02 04:24:31 
EDT ---
where i can i find a sponsor?
i have creat some package:
Review Request: SPE - Stani's Python Editor:
https://bugzilla.redhat.com/show_bug.cgi?id=481022

 A collection of quotes in French for gdesklets:
https://bugzilla.redhat.com/show_bug.cgi?id=483403

Modern background: https://bugzilla.redhat.com/show_bug.cgi?id=483498

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-01-31 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903





--- Comment #26 from Luya Tshimbalanga   2009-01-31 
16:01:18 EDT ---
Next step is to find a sponsor.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-01-31 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903





--- Comment #25 from MERCIER   2009-01-31 12:42:33 
EDT ---
(In reply to comment #22)
> Information is on https://fedoraproject.org/wiki/PackageMaintainers

ok, i read all the document

i am at :
koji build --scratch dist-f10 rpmbuild/SRPMS/SlideShow-0.9-4.fc10.src.rpm 
Uploading srpm: rpmbuild/SRPMS/SlideShow-0.9-4.fc10.src.rpm
[] 100% 00:00:04 143.64 KiB  28.83 KiB/sec
Created task: 1096424
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=1096424
Watching tasks (this may be safely interrupted)...
1096424 build (dist-f10, SlideShow-0.9-4.fc10.src.rpm): open
(xenbuilder2.fedora.redhat.com)
  1096425 buildArch (SlideShow-0.9-4.fc10.src.rpm, noarch): free
  1096425 buildArch (SlideShow-0.9-4.fc10.src.rpm, noarch): free -> open
(ppc10.fedora.phx.redhat.com)
  1096425 buildArch (SlideShow-0.9-4.fc10.src.rpm, noarch): open
(ppc10.fedora.phx.redhat.com) -> closed
  0 free  1 open  1 done  0 failed
1096424 build (dist-f10, SlideShow-0.9-4.fc10.src.rpm): open
(xenbuilder2.fedora.redhat.com) -> closed
  0 free  0 open  2 done  0 failed

1096424 build (dist-f10, SlideShow-0.9-4.fc10.src.rpm) completed successfully

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-01-31 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903





--- Comment #24 from MERCIER   2009-01-31 10:47:12 
EDT ---
(In reply to comment #23)
> ok i have update the summary.
> but i have a little problem with release 3 gdesklet do'nt add the control then
> the desklet do'nt work???

no it's good, is me :p

rpmlint:

$ rpmlint rpmbuild/SPECS/SlideShow.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

$ rpmlint rpmbuild/SRPMS/SlideShow-0.9-4.fc10.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint rpmbuild/RPMS/noarch/SlideShow-0.9-4.fc10.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

I have update spec rpm and src rpm
http://bioinformatiques.free.fr/SlideShow-0.9-4.fc10.noarch.rpm

http://bioinformatiques.free.fr/SlideShow-0.9-4.fc10.src.rpm

http://bioinformatiques.free.fr/SlideShow.spec

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-01-31 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903





--- Comment #23 from MERCIER   2009-01-31 07:23:26 
EDT ---
ok i have update the summary.
but i have a little problem with release 3 gdesklet do'nt add the control then
the desklet do'nt work???

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-01-31 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903





--- Comment #21 from Luya Tshimbalanga   2009-01-31 
04:30:22 EDT ---
Make sure to change Summary in something like: A slideshow of collection for
gdesklets. Also get a Fedora accound, a sponsor to get approval.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-01-31 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903





--- Comment #22 from Luya Tshimbalanga   2009-01-31 
04:31:16 EDT ---
Information is on https://fedoraproject.org/wiki/PackageMaintainers

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-01-31 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903


Luya Tshimbalanga  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|l...@fedoraproject.org




--- Comment #20 from Luya Tshimbalanga   2009-01-31 
04:05:21 EDT ---
Yes, the package is fine

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903





--- Comment #19 from MERCIER   2009-01-30 20:30:00 
EDT ---
(In reply to comment #18)
> (In reply to comment #16)
> > Maybe the gdesklets maintainer can comment on this?
> > 
> > Bottom of comment 14 is some oddity in Fedora's gdesklets package. It is
> > reproducible with the "Calendar" v0.62 Display in F10 "gdesklets" package.
> > Steps to reproduce:
> > 
> > 1. Run "Calendar" desklet from date/time category.
> > 2. Right-click on desklet display for "Configure desklet" pop-up menu.
> > 3. Click "Open" button next to "Month image" input field.
> > 
> > Result: An error dialog on top of the file manager is displayed:
> > 
> > | The folder contents could not be displayed
> > |
> > | Operation not supported  [ OK ]
> > 
> > It looks like a bug in gdesklets or an API incompatibility.
> 
> I confirmed the bug that affect other desklets with image path and submitted 
> it
> to upstream.
> https://bugs.launchpad.net/fedora/+source/gdesklets/+bug/323347
Then the package it's ok?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903


Luya Tshimbalanga  changed:

   What|Removed |Added

 CC||l...@fedoraproject.org




--- Comment #18 from Luya Tshimbalanga   2009-01-30 
15:56:09 EDT ---
(In reply to comment #16)
> Maybe the gdesklets maintainer can comment on this?
> 
> Bottom of comment 14 is some oddity in Fedora's gdesklets package. It is
> reproducible with the "Calendar" v0.62 Display in F10 "gdesklets" package.
> Steps to reproduce:
> 
> 1. Run "Calendar" desklet from date/time category.
> 2. Right-click on desklet display for "Configure desklet" pop-up menu.
> 3. Click "Open" button next to "Month image" input field.
> 
> Result: An error dialog on top of the file manager is displayed:
> 
> | The folder contents could not be displayed
> |
> | Operation not supported  [ OK ]
> 
> It looks like a bug in gdesklets or an API incompatibility.

I confirmed the bug that affect other desklets with image path and submitted it
to upstream.
https://bugs.launchpad.net/fedora/+source/gdesklets/+bug/323347

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903





--- Comment #17 from MERCIER   2009-01-30 13:49:24 
EDT ---
i have found this file:
/usr/lib64/gdesklets/Controls/URI/__init__py
this file is call for openning a file

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903


Michael Schwendt  changed:

   What|Removed |Added

 CC||luya_...@thefinalzone.net




--- Comment #16 from Michael Schwendt   2009-01-30 
06:53:48 EDT ---
Maybe the gdesklets maintainer can comment on this?

Bottom of comment 14 is some oddity in Fedora's gdesklets package. It is
reproducible with the "Calendar" v0.62 Display in F10 "gdesklets" package.
Steps to reproduce:

1. Run "Calendar" desklet from date/time category.
2. Right-click on desklet display for "Configure desklet" pop-up menu.
3. Click "Open" button next to "Month image" input field.

Result: An error dialog on top of the file manager is displayed:

| The folder contents could not be displayed
|
| Operation not supported  [ OK ]

It looks like a bug in gdesklets or an API incompatibility.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-01-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903





--- Comment #15 from MERCIER   2009-01-30 05:24:35 
EDT ---
In first, thanks for you answer.

1/ Why did you change it...?

> %dir %{_datadir}/gdesklets/Controls/%{_ctrlname}
> %dir %{_datadir}/gdesklets/Displays/%{_appname}
> %{_datadir}/gdesklets/Controls/%{_ctrlname}/*
> %{_datadir}/gdesklets/Displays/%{_appname}/*

someone on IRC #fedora-fr say is better.

i have upadte the spec file:
> %{_datadir}/gdesklets/Controls/ImageSlideShow/
> %{_datadir}/gdesklets/Displays/SlideShow/

2/ > %setup -q -n %{_appname}
> tar -xf %{SOURCE1} -C ../

it's ok, i use %setup -q -c %{name}-%{version} -a1

3/ 
The LICENSE, MANIFEST, README and "todo" files are installed in addition to
%doc, and it is important to note that the LICENSE file must not be removed
from /usr/share/gdesklets/Displays/SlideShow/LICENSE - or else it cannot be
displayed by the desklet.

ok, it's done

4/ 
Opening "Configure desklet" and selecting a folder from which to load images
always displays an error dialog:

| The folder contents could not be displayed
| 
| Operation not supported

It seems it always starts in the root directory due to that.


yes i have the same error, i did'nt find the source code who make this error!

5/ they are no directory image 
i have see in Display/SideShow.script line 259:
Prefs.image_directory.value= value[:value.rfind( "/" )]
if you change the value by:
Prefs.image_directory.value= "/home/user/images"
is better.
but where put os.environĀ ['HOME']?

6/ i have update spec rpm and src rpm
http://bioinformatiques.free.fr/SlideShow-0.9-3.fc10.noarch.rpm

http://bioinformatiques.free.fr/SlideShow-0.9-3.fc10.src.rpm

http://bioinformatiques.free.fr/SlideShow.spec


7/ rpmlint:
$ rpmlint rpmbuild/SRPMS/SlideShow-0.9-3.fc10.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint rpmbuild/RPMS/noarch/SlideShow-0.9-3.fc10.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint rpmbuild/SPECS/SlideShow.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-01-27 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903


Michael Schwendt  changed:

   What|Removed |Added

 CC||bugs.mich...@gmx.net




--- Comment #14 from Michael Schwendt   2009-01-27 
08:52:33 EDT ---
The original %files section was fine:

> %{_datadir}/gdesklets/Controls/ImageSlideShow/
> %{_datadir}/gdesklets/Displays/SlideShow/

Why did you change it...?

> %dir %{_datadir}/gdesklets/Controls/%{_ctrlname}
> %dir %{_datadir}/gdesklets/Displays/%{_appname}
> %{_datadir}/gdesklets/Controls/%{_ctrlname}/*
> %{_datadir}/gdesklets/Displays/%{_appname}/*

Both are equivalent.


> %setup -q -n %{_appname}
> tar -xf %{SOURCE1} -C ../

The way the two tarballs are extracted is strange. Sure, you can do it like
that, but having to deal with the "../" in subsequent commands decreases the
readability of the spec file. I strongly encourage you to use

%setup -q -c %{name}-%{version} -a1

to extract both tarballs into the same root directory, and then either use your
two macros %_appname and %_ctrlname when accessing files or entering
directories.


The LICENSE, MANIFEST, README and "todo" files are installed in addition to
%doc, and it is important to note that the LICENSE file must not be removed
from /usr/share/gdesklets/Displays/SlideShow/LICENSE - or else it cannot be
displayed by the desklet.


Opening "Configure desklet" and selecting a folder from which to load images
always displays an error dialog:

| The folder contents could not be displayed
| 
| Operation not supported

It seems it always starts in the root directory due to that.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-01-21 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479903


Mamoru Tasaka  changed:

   What|Removed |Added

Summary|gdesklets plug-in, Cycle|Review Request:
   |through a collection of |gdesklet-slideshow - Cycle
   |pictures|through a collection of
   ||pictures




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review