cht電腦資訊gcin
adm Find login register

一些小 patches

Tetralet

joined: 2007-11-27
posted: 255
promoted: 35
bookmarked: 13
1subject: 一些小 patchesPromote 0 Bookmark 02011-12-25quote  

Hi,

最近幾天敝人重感冒了(多行不義的關係嗎? XD),patch 送得遲了,請見諒。

這些 patch 請千萬 review 後再 apply。敝人說明如後:

GCIN_gtk3_only.diff: 新 ./configure 編譯參數 --with_gtk3,
可讓 GCIN 在純 GTK3 下編譯。(不會編 gtk2-im-module),
另外有些參數檢查字串我也改成了比較安全的方式。

GCIN_html_editor.diff:未考慮到 $UTF8ED 為空的情況下,可能會發生安全上的疑慮。

GCIN_man_install.diff:manpage 檔案權限設為 444 似乎怪怪的?至少我的系統上都是 644。

Tetralet

joined: 2007-11-27
posted: 255
promoted: 35
bookmarked: 13
2subject: Promote 0 Bookmark 02011-12-25quote  

另外,對於 gen-deb 可能要再加上一些無關緊要的小小修正,如附件。

還有,在 postinst/prerm 裡應該不用再呼叫 gtk-query-immodules-3.0。
(gtk-query-immodules-2.0 應該也不用)
在 Debian 系統裡這些應該是由 trigger 完成了。
但因為不知 eliu 的考量何在,
所以就沒有寫在 patch 裡了。

eliu

joined: 2007-08-09
posted: 11482
promoted: 617
bookmarked: 187
新竹, 台灣
3subject: Promote 0 Bookmark 02011-12-25quote  

Tetralet
還有,在 postinst/prerm 裡應該不用再呼叫 gtk-query-immodules-3.0。

在 ubuntu 11.10 上可以看到 postinst 並沒有執行 gtk-query-immodules-3.0,因為 cache file 沒有 update。 

Tetralet

joined: 2007-11-27
posted: 255
promoted: 35
bookmarked: 13
4subject: Promote 0 Bookmark 02011-12-26quote  

eliu


在 ubuntu 11.10 上可以看到 postinst 並沒有執行 gtk-query-immodules-3.0,因為 cache file 沒有 update。



使用 caleb 原本寫的那個 gen-deb 加上我的 GCIN_gen-deb.diff,
然後試著在 Ubuntu Oneiric 上編譯並安裝 gcin 套件時,
應該會看到以下訊息:(已實測)

Processing triggers for libgtk2.0-0 ...
Processing triggers for libgtk-3-0 ...

所以應該是有執行到 gtk-query-immodules-3.0 的,不需再額外手動執行。

eliu

joined: 2007-08-09
posted: 11482
promoted: 617
bookmarked: 187
新竹, 台灣
5subject: Promote 0 Bookmark 02011-12-26quote  

你有沒有實際去看 gtk3 im module 的 cache file?

重覆執行也不會怎樣,沒有執行到就很麻煩。

Tetralet

joined: 2007-11-27
posted: 255
promoted: 35
bookmarked: 13
6subject: Promote 0 Bookmark 02011-12-26quote  

eliu

你有沒有實際去看 gtk3 im module 的 cache file?

重覆執行也不會怎樣,沒有執行到就很麻煩。

個人實測是沒問題的...

但既然不放心的話,留著也沒差。

eliu

joined: 2007-08-09
posted: 11482
promoted: 617
bookmarked: 187
新竹, 台灣
7subject: Promote 0 Bookmark 02011-12-26quote  

我就是因為在 ubuntu 11.10 發現有問題才加的。

cht電腦資訊gcin
adm Find login register
views:14532