Bug 446160 - Review Request: file-browser-applet - File Browser Applet for the GNOME Panel
Summary: Review Request: file-browser-applet - File Browser Applet for the GNOME Panel
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-05-13 04:55 UTC by Deji Akingunola
Modified: 2008-06-03 20:04 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-06-03 20:04:56 UTC
Type: ---
Embargoed:
mtasaka: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Deji Akingunola 2008-05-13 04:55:40 UTC
Spec URL: ftp://czar.eas.yorku.ca/pub/fba/file-browser-applet.spec
SRPM URL: ftp://czar.eas.yorku.ca/pub/fba/file-browser-apple
Description: File Browser Applet is an applet for the GNOME Panel. It lets you browse
and open files in your home directory from the gnome panel, without having to
open a file manager. It is _not_ intended as replacement to proper file
managers like nautilus. Sometimes you (like me) just want to open a file and
don't want to have to open a file manager, browse to and open the file, then
close the file manager. This Applet just saves you couple of clicks.

Comment 1 Mamoru TASAKA 2008-05-22 15:38:59 UTC
Where is your srpm?

Comment 2 Deji Akingunola 2008-05-22 16:57:34 UTC
(In reply to comment #1)
> Where is your srpm?

Doh!! It's there now, thanks.

Spec URL: ftp://czar.eas.yorku.ca/pub/fba/file-browser-applet.spec
SRPM URL: ftp://czar.eas.yorku.ca/pub/fba/file-browser-applet-0.5.6-1.fc9.src.rpm



Comment 3 Mamoru TASAKA 2008-05-25 13:20:27 UTC
For 0.5.6-1:

* compilar flags
--------------------------------------------------------------
    91  cd /builddir/build/BUILD/file-browser-applet-0.5.6/fedora/src &&
/usr/bin/gcc   -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions
-fstack-protector --param=ssp-buffer-size=4 -m32 -march=i386 -mtune=generic
-fasynchronous-unwind-tables  -O3 -g -Wall -ansi -DORBIT2=1 -pthread
-I/usr/include/panel-2.0 -I/usr/include/gtk-2.0 -I/usr/include/libgnomeui-2.0
-I/usr/include/libbonoboui-2.0 -I/usr/lib/gtk-2.0/include -I/usr/include/atk-1.0
-I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/glib-2.0
-I/usr/lib/glib-2.0/include -I/usr/include/libgnome-2.0
-I/usr/include/libgnomecanvas-2.0 -I/usr/include/gnome-vfs-2.0
-I/usr/lib/gnome-vfs-2.0/include -I/usr/include/libbonobo-2.0
-I/usr/include/orbit-2.0 -I/usr/include/bonobo-activation-2.0
-I/usr/include/libart-2.0 -I/usr/include/gconf/2 -I/usr/include/libglade-2.0
-I/usr/include/gtk-2.0 -I/usr/include/libxml2 -I/usr/lib/gtk-2.0/include
-I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/pango-1.0
-I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -DORBIT2=1 -pthread
-I/usr/include/gnome-desktop-2.0 -I/usr/include/gtk-2.0
-I/usr/include/libgnomeui-2.0 -I/usr/include/startup-notification-1.0
-I/usr/lib/gtk-2.0/include -I/usr/include/atk-1.0 -I/usr/include/cairo
-I/usr/include/pango-1.0 -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include
-I/usr/include/libgnome-2.0 -I/usr/include/libbonoboui-2.0
-I/usr/include/libgnomecanvas-2.0 -I/usr/include/gnome-vfs-2.0
-I/usr/lib/gnome-vfs-2.0/include -I/usr/include/libbonobo-2.0
-I/usr/include/orbit-2.0 -I/usr/include/bonobo-activation-2.0
-I/usr/include/libart-2.0 -I/usr/include/gconf/2   -o
CMakeFiles/file-browser-applet.dir/main.o   -c
/builddir/build/BUILD/file-browser-applet-0.5.6/src/main.c
--------------------------------------------------------------
  - Here -O2 optimization level by %optflags is replaced by the latter
    -O3, which must be fixed on Fedora.

* Directory ownership issue
---------------------------------------------------------------
[root@localhost i386]# rpm -qf /usr/lib/gnome-panel/file-browser-applet 
file-browser-applet-0.5.6-1.fc10.i386
[root@localhost i386]# LANG=C rpm -qf /usr/lib/gnome-panel
file /usr/lib/gnome-panel is not owned by any package
[root@localhost i386]# yum whatprovides /usr/lib/gnome-panel
Loaded plugins: refresh-packagekit
Importing additional filelist information
No Matches found
----------------------------------------------------------------
   - Would you verify the ownership of %_libdir/gnome-panel?

Comment 4 Mamoru TASAKA 2008-06-02 16:12:16 UTC
ping?

Comment 5 Deji Akingunola 2008-06-02 16:39:40 UTC
I've actually dealt with the above issues, see below. But I was waiting to hear
back from upstream, hoping they'll readily sign on to changes I made to fix the
directory ownership issue.

Spec URL: ftp://czar.eas.yorku.ca/pub/fba/file-browser-applet.spec
SRPM URL: ftp://czar.eas.yorku.ca/pub/fba/file-browser-applet-0.5.6-2.fc9.src.rpm


Comment 6 Mamoru TASAKA 2008-06-02 18:30:26 UTC
Well, on rawhide i386 when I
- Click mouse right button on gnome-panel, select "Add to panel",
  then choose "File Browser" and click "Add"
- On gnome-panel click mouse right button on File browser applet
  and choose "Remove from panel"

Then gnome-panel always crash, while trying with other applets gnome-panel
does not crash.... very strange....

Are you also seeing the issue this??
Note: debugging gnome-panel on rawhide does not work for now due to
       bug 449457

Comment 7 Deji Akingunola 2008-06-02 18:41:31 UTC
(In reply to comment #6)
> Well, on rawhide i386 when I
> - Click mouse right button on gnome-panel, select "Add to panel",
>   then choose "File Browser" and click "Add"
> - On gnome-panel click mouse right button on File browser applet
>   and choose "Remove from panel"
> 
> Then gnome-panel always crash, while trying with other applets gnome-panel
> does not crash.... very strange....
> 
> Are you also seeing the issue this??

I'm not seeing any such issue on x86_64 F-9; adding and removing the applet
works just fine without crashing gnome-panel. 


> Note: debugging gnome-panel on rawhide does not work for now due to
>        bug 449457



Comment 8 Mamoru TASAKA 2008-06-02 19:05:15 UTC
Okay, now I assume this is gnome-panel side bug and I will approve
this package.

------------------------------------------------------------------
     This package (file-browser-applet) is APPROVED by me
------------------------------------------------------------------

Comment 9 Deji Akingunola 2008-06-02 20:38:08 UTC
New Package CVS Request
=======================
Package Name: ffile-browser-applet
Short Description: File Browser Applet for the GNOME Panel
Owners: deji
Branches: F-8 F-9
InitialCC: 
Cvsextras Commits: yes


Comment 10 Mamoru TASAKA 2008-06-03 03:27:01 UTC
Would you check the Package Name?

Comment 11 Deji Akingunola 2008-06-03 05:46:18 UTC
New Package CVS Request
=======================
Package Name: file-browser-applet
Short Description: File Browser Applet for the GNOME Panel
Owners: deji
Branches: F-8 F-9
InitialCC: 
Cvsextras Commits: yes



Comment 12 Kevin Fenzi 2008-06-03 16:19:25 UTC
cvs done.

Comment 13 Deji Akingunola 2008-06-03 20:04:56 UTC
Thank you Mamoru for another excellent review.

Package imported and built for rawhide, F-8, and F-9.


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