Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review request: qtpfsgui - A Qt4 graphical user interface that 
provides a workflow for HDR imaging


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


[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[EMAIL PROTECTED]




------- Additional Comments From [EMAIL PROTECTED]  2007-07-03 19:15 EST -------
Looks good to me, I was going to submit this myself.  My suggestions, though you
don't have to follow them:

The summary could be a bit less cryptic:

  A graphical tool for creating and tone-mapping HDR images

..and the description should say a bit more:

  Qtpfsgui is a graphical program for assembling bracketed photos into High
  Dynamic Range (HDR) images.  It also provides a number of tone-mapping
  operators for creating low dynamic range versions of HDR images.

I would fix the newlines in %prep and not %install, but this isn't important.

You can call 'desktop-file-install' with '--delete-original', then you don't
have to manually delete it afterwards

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

_______________________________________________
Fedora-package-review mailing list
[email protected]
http://www.redhat.com/mailman/listinfo/fedora-package-review

Reply via email to