Attachment 221571 Details for Bug 325231 – upated patch, fixes typo

[patch] upated patch, fixes typo

codeina.diff (text/plain), 2.54 KB, created by Bill Nottingham on 2007-10-09 17:58:26 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Bill Nottingham

Created: 2007-10-09 17:58:26 UTC

Size: 2.54 KB

patch

obsolete

>diff -ru codeina-0.10.1/codeina/distro/fedora.py codeina-0.10.1/codeina/distro/fedora.py
>--- codeina-0.10.1/codeina/distro/fedora.py	2007-08-27 07:16:41.000000000 -0400
>+++ codeina-0.10.1/codeina/distro/fedora.py	2007-10-09 13:17:39.000000000 -0400
>@@ -23,6 +23,7 @@
> # Boston, MA 02111-1307, USA.
> 
> import gtk
>+import sexy
> import os
> 
> from codeina.extern.log import log
>@@ -31,19 +32,27 @@
> 
> 
> def get_dialog(parent, products):
>+    def show_url(widget, url):
>+        os.system("gnome-open %s" % (url,))
>+        
>     log.debug('fedora', 'products: %r', products)
>     dlg = gtk.MessageDialog(parent=parent, flags=0,
>                             type=gtk.MESSAGE_INFO,
>                             buttons=gtk.BUTTONS_CANCEL,
>                             message_format=\
>                             _('Proprietary and free formats'))
>-    dlg.add_button(_("Go to _webshop browser"), gtk.RESPONSE_OK)
>+    link = sexy.UrlLabel("For more information, see <a href=\"http://fedoraproject.org/codecbuddy/\">About Proprietary and Free Formats</a>.")
>+    dlg.vbox.add(link)
>+    link.connect("url-activated", show_url)
>+    link.show()
>+    dlg.add_button(_("_See available options"), gtk.RESPONSE_OK)
>     dlg.set_default_response(gtk.RESPONSE_OK)
>     # As per HIG, empty dialogue title
>     dlg.set_title(_(''))
>-    str1 = _("Fedora does not condone the use of audio and video codecs that require patent licenses to be written and/or distributed.")
>-    str2 = _("But due to the existence of such patents, Free Software implementations of such codecs might not be legal in the country you live in, or the file you're trying to play back might not be available in free formats such as Ogg.")
>-    str3 = _("The Fluendo webshop has free and for-pay plugins for you to download.")
>+    str1 = _("Fedora has the mission of always being freely re-distributable; this means you are free to give your copy of Fedora to anyone else.")
>+    str2 = _("Unfortunately, that means that we cannot ship support for certain multimedia codecs, as they require patent licenses before you can view or play media that use them. Imagine if you had to pay a license fee before reading your e-mail, or viewing a picture on the web. This is why Fedora supports free formats, such as Ogg Vorbis and Theora.")
>+    str3 = _("However, there are companies and communities that do offer support for certain codecs under Fedora. If you would like to install support, please proceed to see the available options.")
>+    
>     str = str1 + "\n\n" + str2 + "\n\n" + str3
>     dlg.format_secondary_text(str)
>     return dlg

Actions: View | Diff

Attachments on bug 325231: 221521 | 221531 | 221571