[Bug 473679] Review Request: xfburn - Simple CD/DVD burning tool based on libburnia

[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=473679


Christoph Wickert <fedora@xxxxxxxxxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |denis@xxxxxxxxxxxxx
               Flag|                            |fedora-cvs?




--- Comment #2 from Christoph Wickert <fedora@xxxxxxxxxxxxxxxxxxxx>  2008-12-03 15:24:50 EDT ---
Thanks for the review.

(In reply to comment #1)
> Fine package. Burnt a CD-RW as a test without any problems. In GNOME.
> 
> Only adjustments I could think of:
>  * Requiring "hicolor-icon-theme" for the icon hierarchy.

Xfburn already requires hicolor-icon theme trough gtk2, but I can add it
explicitly.

>  * Removing the Xfce reference in the summary.
>  * Be careful with the --enable-debug option as not to enable anything you may
> not like. Currently it's okay, though.

This is actually a relict of debuging a reproducable crash in 3.2. Good catch.

> APPROVED


@Denis: I'd like you to be able to do a rebuild of this package when necessary
(soname bump of libburn or libisofs), so I add you as owner.

New Package CVS Request
=======================
Package Name: xfburn
Short Description: Simple CD/DVD burning tool based on libburnia
Owners: cwickert denis
Branches: F-9 F-10
InitialCC:

-- 
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]