After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 589123 - Brasero typo: should say finalizing instead of finalising.
Brasero typo: should say finalizing instead of finalising.
Status: RESOLVED FIXED
Product: brasero
Classification: Applications
Component: general
2.27.x
Other Linux
: Normal minor
: 2.26
Assigned To: Brasero maintainer(s)
Brasero maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2009-07-20 12:25 UTC by Pedro Villavicencio
Modified: 2009-07-30 10:48 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch 1 (18.34 KB, patch)
2009-07-26 20:48 UTC, Oskar Wallgren
none Details | Review
Patch 2 (585 bytes, patch)
2009-07-26 21:08 UTC, Oskar Wallgren
rejected Details | Review
Patch 1 corrected (18.44 KB, patch)
2009-07-26 21:54 UTC, Oskar Wallgren
rejected Details | Review
brasero-2.27.4 correction of .po files (18.55 KB, patch)
2009-07-27 01:51 UTC, Oskar Wallgren
rejected Details | Review
brasero-2.27.4 correction of /libbrasero-burn/burn-basics.c (551 bytes, patch)
2009-07-27 01:56 UTC, Oskar Wallgren
committed Details | Review

Description Pedro Villavicencio 2009-07-20 12:25:47 UTC
This report has been filed here:

https://bugs.edge.launchpad.net/ubuntu/+source/brasero/+bug/401488

"Brasero disk burner has a typo. When the disk is finishing burning, the text in the window reads "finalising" when it should say "finalizing"."

Thanks,
Comment 1 Oskar Wallgren 2009-07-26 20:47:25 UTC
I couldn't fix this for 2.27 today but I managed to create a patch (2 actually) for 2.26.1, which is what I'm running right now.

Patch 1 goes into the /po folder and patch 2 into the /src folder.
It worked on my machine.
Comment 2 Oskar Wallgren 2009-07-26 20:48:21 UTC
Created attachment 139249 [details] [review]
Patch 1
Comment 3 Oskar Wallgren 2009-07-26 21:08:15 UTC
Created attachment 139252 [details] [review]
Patch 2
Comment 4 Oskar Wallgren 2009-07-26 21:54:24 UTC
Created attachment 139254 [details] [review]
Patch 1 corrected
Comment 5 Oskar Wallgren 2009-07-27 01:47:57 UTC
The attached files above changes all instances of Finalise to Finalize in the .po files ( msgid ), and in the src/burn-basics.c sourcecode.

In brasero-2.27.4 the burn-basics.c file is situated in libbrasero-burn instead of src/ .

I attach patches for 2.74.4 ( only tested by re-creating the corrected files with the patch command. I haven't actually compiled and tried it but it should work...  ;-)
Comment 6 Oskar Wallgren 2009-07-27 01:51:20 UTC
Created attachment 139259 [details] [review]
brasero-2.27.4 correction of .po files

..
Comment 7 Oskar Wallgren 2009-07-27 01:56:57 UTC
Created attachment 139260 [details] [review]
brasero-2.27.4 correction of /libbrasero-burn/burn-basics.c 

This file goes into /libbrasero-burn/
These are the very first patches I've produced so I hope the format is correct.
Comment 8 Philippe Rouquier 2009-07-30 09:27:19 UTC
Thanks for the report and the patch.
Oskar, you don't need to change the po files. They are regenerated by the translators with intltool-update. You just need to change the offending string in the code as you did in the latest patch (which is correct btw).
As for the patches themselves, I am going to only use your latest as:
- strings cannot be changed for stable version anymore
- it is the one that actually fix the problem in unstable.
I won't apply it right now as I will ask for a string break first. But as soon as the permission is given by translators, I'll commit it.
Thanks again.
Comment 9 Philippe Rouquier 2009-07-30 10:48:21 UTC
Translators were very responsive. I committed your last patch. Thanks again.