[Bug 478504] Review Request: gget - Download Manager for the GNOME desktop.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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=478504


Christoph Wickert <fedora@xxxxxxxxxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|177841                      |
               Flag|fedora-review?              |fedora-review+




--- Comment #32 from Christoph Wickert <fedora@xxxxxxxxxxxxxxxxxxxx>  2009-02-20 17:34:08 EDT ---
(In reply to comment #31)
> 
> Re-added chmod, using spaces not tabs:

The only chmod that is actually needed is 
chmod +x $RPM_BUILD_ROOT%{python_sitelib}/%{name}/metalink.py

You can remove the other one after import.

> > Please add "--add-category=FileTransfer" to desktop-file-install to allow
> > nested menus (we are working on a set of submenu-packages for user who have a
> > lot of stuff installed)
> 
> Done. Should I ask upstream to add this as well?

Yes please. It's not really important, but if it's fixed upstream you have less
work.


8063441d8f95cee10efc9ee72bb429ae  gget-0.0.4-8.fc10.src.rpm
fixes all outstanding issues, so this package is

=================================================================
                        APROVED by cwickert
=================================================================

Ant, please get your self a FAS account, so I can sponsor you.

-- 
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@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]