Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 98520 - UTF-8 locale and empty tips
Summary: UTF-8 locale and empty tips
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: gimp
Version: 1
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Nils Philippsen
QA Contact: David Lawrence
URL:
Whiteboard:
: 110329 (view as bug list)
Depends On: 101377
Blocks:
TreeView+ depends on / blocked
 
Reported: 2003-07-03 08:04 UTC by Akira TAGOH
Modified: 2007-11-30 22:10 UTC (History)
2 users (show)

Fixed In Version: 1.2.5-2
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2004-09-13 03:32:11 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
proposed patch (deleted)
2003-12-11 01:55 UTC, Akira TAGOH
no flags Details | Diff
patch to iconv UTF-8 tips files to the native locale (deleted)
2003-12-24 06:35 UTC, Akira TAGOH
no flags Details | Diff
overhauled patch to iconv UTF-8 tips files to the native locale (deleted)
2004-04-08 09:01 UTC, Nils Philippsen
no flags Details | Diff
fixed overhauled patch to iconv UTF-8 tips files to the native locale (deleted)
2004-04-08 09:19 UTC, Nils Philippsen
no flags Details | Diff

Description Akira TAGOH 2003-07-03 08:04:51 UTC
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; ja-JP; rv:1.2.1) Gecko/20030225

Description of problem:
When I run GIMP with UTF-8 locale, empty tips are displayed so that the tips
files is stored as the legacy encoding.

Version-Release number of selected component (if applicable):


How reproducible:
Always

Steps to Reproduce:
1.LANG=ja_JP.UTF-8 gimp, for example
2.
3.
    

Additional info:

Comment 1 Leon Ho 2003-08-05 02:40:48 UTC
Okay.. Should we iconv all tips text file to UTF-8 for Cambridge?

Comment 2 Akira TAGOH 2003-08-05 13:18:10 UTC
Yes, it should solves this problem.

Comment 3 Akira TAGOH 2003-12-11 01:55:26 UTC
Created attachment 96465 [details]
proposed patch

the encoding of tips files can be guessed from the po files, I think. so
attached patch will just try to convert all tips files to UTF-8.

Comment 4 Matt Wilson 2003-12-15 19:45:29 UTC
*** Bug 110329 has been marked as a duplicate of this bug. ***

Comment 5 Akira TAGOH 2003-12-24 06:35:31 UTC
Created attachment 96690 [details]
patch to iconv UTF-8 tips files to the native locale

This patch also supports to be ran on the native locale.

Comment 6 Akira TAGOH 2004-04-07 04:50:11 UTC
how is this going on? and can we get a chance to fix this on FC1 updates?
This problem causes really unusable for the newbie gimp users who want
to know the tips on Japanese.

Comment 7 Nils Philippsen 2004-04-08 08:13:57 UTC
I'm looking at the patch now and will likely put an updated package
out soon.

Comment 8 Nils Philippsen 2004-04-08 08:15:09 UTC
BTW: Am I supposed to apply both patches or only the second one?

Comment 9 Nils Philippsen 2004-04-08 08:52:31 UTC
I looked at your second patch and it seems like it isn't quite
consistent in using newly allocated and static strings... Guess I'll
need to overhaul the patch a bit.

Comment 10 Nils Philippsen 2004-04-08 09:01:52 UTC
Created attachment 99229 [details]
overhauled patch to iconv UTF-8 tips files to the native locale

this patch strdup()s in store_tip() so we only have newly allocated strings in
tips_text[].

Comment 11 Nils Philippsen 2004-04-08 09:19:54 UTC
Created attachment 99230 [details]
fixed overhauled patch to iconv UTF-8 tips files to the native locale

D'oh

Comment 12 Nils Philippsen 2004-04-08 09:42:40 UTC
Guess I'll have to apply the first patch as well...

Comment 13 Akira TAGOH 2004-04-09 03:26:23 UTC
Yes, the patch is assuming all of po files are UTF-8.

Comment 14 Nils Philippsen 2004-04-16 12:49:42 UTC
please test gimp-1.2.5-2

Comment 15 Akira TAGOH 2004-04-19 02:51:03 UTC
confirmed the fix. thanks.

Comment 16 Akira TAGOH 2004-05-17 09:01:33 UTC
doh, this package wasn't pushed to FC1-updates. reopening and
reverting the status.

Comment 17 Lawrence Lim 2004-09-13 03:32:11 UTC
Confirmed fixed in FC1-update; gimp-1.2.5-2

Tested in the following locale:
ja_JP
ja_JP.UTF-8
ko_KR
ko_KR.UTF-8
zh_CN.GB18030
zh_CN.UTF-8
zh_TW.Bug5
zh_TW.UTF-8


Thanks,
Lawrence


Note You need to log in before you can comment on or make changes to this bug.