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 339259 - [Patch] Use po/LINGUAS and intltool
[Patch] Use po/LINGUAS and intltool
Status: RESOLVED FIXED
Product: atk
Classification: Platform
Component: atk
git master
Other Linux
: High enhancement
: ---
Assigned To: bill.haneman
bill.haneman
Depends on: 344383
Blocks:
 
 
Reported: 2006-04-21 07:22 UTC by Przemysław Grzegorczyk
Modified: 2008-10-06 11:58 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch (3.61 KB, patch)
2006-04-21 07:23 UTC, Przemysław Grzegorczyk
none Details | Review
Updated patch - intltool 0.35.0 released (3.61 KB, patch)
2006-05-15 18:12 UTC, Przemysław Grzegorczyk
rejected Details | Review
Use po/LINGUAS without dependency on intltool (1.31 KB, patch)
2008-06-14 01:20 UTC, Thomas Andersen
none Details | Review

Description Przemysław Grzegorczyk 2006-04-21 07:22:57 UTC
This patch is based on the new guidelines (http://live.gnome.org/GnomeGoals/PoLinguas).
It adds po/LINGUAS file and intltool support.
Comment 1 Przemysław Grzegorczyk 2006-04-21 07:23:39 UTC
Created attachment 64006 [details] [review]
Proposed patch
Comment 2 Przemysław Grzegorczyk 2006-05-15 18:12:01 UTC
Created attachment 65522 [details] [review]
Updated patch - intltool 0.35.0 released
Comment 3 bill.haneman 2006-06-09 16:31:05 UTC
I don't like the dependency on 0.35.0 intltool, is that absolutely necessary?
Comment 4 bill.haneman 2006-06-09 16:36:46 UTC
Since atk is a dependency of gtk+, I think maybe we should wait to do this when glib/gtk+ do it, otherwise someone building gtk+ will still encounter the intltool dependency.
Comment 5 bill.haneman 2006-06-14 09:57:04 UTC
I am marking this bug as dependent on 344383, since atk is a dependency of gtk+.  If we apply the patch to atk, we'll be forcing the issue raised in 344383, so I'd rather wait for the gtk+ team here.

In order to flag the fact that we're in a "holding pattern" with respect to this patch, I am also marking the bug 'NEEDINFO', though the only info we really need is knowing whether/when gtk+ plans to adopt the new PO strategy.  If the gtk+ plans to reject the patch indefinitely, then we too should IMO close as WONTFIX.  Personally I don't have an objection to the patch appearing in HEAD, but I think we should be mindful of the impact on gtk+

(If we get a solution to the circular dependency issue for glib/intltool then it seems reasonable to me to use LINGUAS for everything at glib and above)
Comment 6 bill.haneman 2006-09-14 11:05:42 UTC
Changing from 'minor' severity to 'enhancement' and leaving NEEDINFO (really should be BLOCKED bug we don't have a bugzilla status for that I guess)
Comment 7 Thomas Andersen 2008-06-14 01:20:08 UTC
Created attachment 112719 [details] [review]
Use po/LINGUAS without dependency on intltool

patch based on the one used for glib and proposed for gtk.
Comment 8 André Klapper 2008-08-02 15:56:26 UTC
Bug 344383 has been fixed, this is the LAST GNOME MODULE waiting to get fixed.

Bill, can you please review and commit?
Comment 9 Gil Forcada 2008-08-13 21:48:18 UTC
ping :)
Comment 10 Li Yuan 2008-08-14 02:57:35 UTC
I should review the patch but I am not in the cc list.

After applying the patch, I got error when "make dist".

cp: `./af.po' and `/home/liyuan/src/atk/atk-1.23.5/po/af.po' are the same file
......
......
......
cp: `./zh_CN.gmo' and `/home/liyuan/src/atk/atk-1.23.5/po/zh_CN.gmo' are the same file
cp: `./zh_HK.gmo' and `/home/liyuan/src/atk/atk-1.23.5/po/zh_HK.gmo' are the same file
cp: `./zh_TW.gmo' and `/home/liyuan/src/atk/atk-1.23.5/po/zh_TW.gmo' are the same file
make[1]: *** [distdir] Error 1
make[1]: Leaving directory `/home/liyuan/src/atk/po'
make: *** [distdir] Error 1
Comment 11 André Klapper 2008-08-14 09:17:45 UTC
Hmm.... Thomas, any idea?
Comment 12 André Klapper 2008-08-21 21:09:00 UTC
Thomas, any idea?
Comment 13 Thomas Andersen 2008-10-06 11:58:20 UTC
Li Yuan committed the patch in bug #551474 (rev 1272) so this bug can be closed now. I hope I don't step on anybodies toes when I change status on bugs in others modules. If I do just let me know and I will stop doing that.