[debian-devel:19026] freewnn: FTBFS: Bad format near line 1 "Binaryfilefull.fsrcmatches". (was: Why does not atod command exist in freewnn package on sid?)

Mitsutoshi NAKANO <[email protected]>
Newsgroups gmane.linux.debian.jp.devel
Message-ID <CANW2+it3b5_ktfZDW-TBY3fbA74yXGMPz_JGeSObUjm1Zdxc0Q@mail.gmail.com>
2015年4月16日 10:12 Nobuhiro Iwamatsu <[email protected]>:
> アップロード依頼の前にパッケージの変更内容レビュー依頼をここで行うのがよいと思います。

freewnnに関してご相談したいことがあります。

先週、以下のようなbug reportがありました。

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=797242

FreeWnnビルドの過程で
EUC-JPで書かれたtext fileを辞書に変換しています。
このときegrepを使っています。

--------
egrep -v '^(# |#line |#pragma |$)'  | ../../Wnn/jutil/atof -h
../../Wnn/jd/hinsi.data kougo.fzk
--------

ところがegrepの新しい版ではEUC-JPをbinaryと判断します。
それで、変換コマンドのatofが

--------
バイナリファイル kougo.fsrc に一致しました
--------

というegrepの出力を読み込んで、失敗終了し、buildが失敗します。


egrepに--binary-files=textオプションを付けて、回避するpatchを
書きました。
debuildできることを確認しました。
(patchを添付しておきます。)

freewnn_1.1.1~a021+cvs20130302-5 を作りたいと思います。

変更箇所はdebian/ディレクトリの
patches/series (パッチ追加)
changelog (履歴追加)
control (メンテナ変更)
だと認識しています。

1) 他に変更しておく所は無いでしょうか?
2) 上記ファイルの変更が完了したら、どこかに掲示しますので
どなたかレビューしていただけますでしょうか?
3) freewnn_1.1.1~a021+cvs20130302-5 が出来たらどこに
置いておけばよいでしょうか?
4) #797242 をpendingに変えておくべきでしょうか?

-- 
Mitsutoshi NAKANO <[email protected]> <[email protected]>
 <https://twitter.com/ItSANgo> <https://launchpad.net/~bkbin005>
0007-add-egrep-binary-files-text-option.patch (application/octet-stream, 3 KB)
From: Mitsutoshi NAKANO <[email protected]>
Date: Sat, 5 Sep 2015 19:30:43 +0900
Subject: add egrep --binary-files=text option

---
 Wnn/pubdicplus/Makefile.in | 2 +-
 cWnn/cdic/Makefile.in      | 4 ++--
 cWnn/tdic/Makefile.in      | 4 ++--
 kWnn/kdic/Makefile.in      | 2 +-
 4 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/Wnn/pubdicplus/Makefile.in b/Wnn/pubdicplus/Makefile.in
index 9a3dd29..017b3ef 100644
--- a/Wnn/pubdicplus/Makefile.in
+++ b/Wnn/pubdicplus/Makefile.in
@@ -353,7 +353,7 @@ $(WNNDICS): pod $(PUBDICPLUSSRC)/pubdic.p
 	$(ATOD) -h $(HINSI) $@ < $<
 
 .fsrc.fzk:
-	egrep -v '^(# |#line |#pragma |$$)' $< | $(ATOF) -h $(HINSI) $@
+	egrep --binary-files=text -v '^(# |#line |#pragma |$$)' $< | $(ATOF) -h $(HINSI) $@
 
 $(DIC): $(HINSI)
 
diff --git a/cWnn/cdic/Makefile.in b/cWnn/cdic/Makefile.in
index 41bfcab..1477411 100644
--- a/cWnn/cdic/Makefile.in
+++ b/cWnn/cdic/Makefile.in
@@ -64,10 +64,10 @@ all:: $(ALLTGT)
 $(DICS): $(HINSI)
 
 $(FULL):: $(FZK_SRCS) con.master-.c
-	$(FZK_PP) -I$(srcdir) $(FZK_FLAG) con.master-.c | egrep -v '^(# |#line |#pragma |$$)' | $(ATOF) -h $(HINSI) $@
+	$(FZK_PP) -I$(srcdir) $(FZK_FLAG) con.master-.c | egrep --binary-files=text -v '^(# |#line |#pragma |$$)' | $(ATOF) -h $(HINSI) $@
 
 $(FULLR):: $(FZK_SRCS) con.masterR-.c
-	$(FZK_PP) -I$(srcdir) $(FZK_FLAG) con.masterR-.c | egrep -v '^(# |#line |#pragma |$$)' | $(ATOF) -h $(HINSI) $@
+	$(FZK_PP) -I$(srcdir) $(FZK_FLAG) con.masterR-.c | egrep --binary-files=text -v '^(# |#line |#pragma |$$)' | $(ATOF) -h $(HINSI) $@
 
 # To avoid circular dependency, we don't use 'con.master?.c'
 con.master-.c: con.master
diff --git a/cWnn/tdic/Makefile.in b/cWnn/tdic/Makefile.in
index 29327f7..62fb767 100644
--- a/cWnn/tdic/Makefile.in
+++ b/cWnn/tdic/Makefile.in
@@ -64,10 +64,10 @@ all:: $(ALLTGT)
 $(DICS): $(HINSI)
 
 $(FULL):: $(FZK_SRCS) con.master-.c
-	$(FZK_PP) -I$(srcdir) $(FZK_FLAG) con.master-.c | egrep -v '^(# |#line |#pragma |$$)' | $(ATOF) -h $(HINSI) $@
+	$(FZK_PP) -I$(srcdir) $(FZK_FLAG) con.master-.c | egrep --binary-files=text -v '^(# |#line |#pragma |$$)' | $(ATOF) -h $(HINSI) $@
 
 $(FULLR):: $(FZK_SRCS) con.masterR-.c
-	$(FZK_PP) -I$(srcdir) $(FZK_FLAG) con.masterR-.c | egrep -v '^(# |#line |#pragma |$$)' | $(ATOF) -h $(HINSI) $@
+	$(FZK_PP) -I$(srcdir) $(FZK_FLAG) con.masterR-.c | egrep --binary-files=text -v '^(# |#line |#pragma |$$)' | $(ATOF) -h $(HINSI) $@
 
 # To avoid circular dependency, we don't use 'con.master?.c'
 con.master-.c: con.master
diff --git a/kWnn/kdic/Makefile.in b/kWnn/kdic/Makefile.in
index e326fdb..0013631 100644
--- a/kWnn/kdic/Makefile.in
+++ b/kWnn/kdic/Makefile.in
@@ -62,7 +62,7 @@ all:: $(ALLTGT)
 $(DICS): $(HINSI)
 
 $(FULL):: $(FZK_SRCS) fzk.master-.c
-	$(FZK_PP) -I$(srcdir) $(FZK_FLAG) fzk.master-.c | egrep -v '^(# |#line |#pragma |$$)' | $(ATOF) -h $(HINSI) $@
+	$(FZK_PP) -I$(srcdir) $(FZK_FLAG) fzk.master-.c | egrep --binary-files=text -v '^(# |#line |#pragma |$$)' | $(ATOF) -h $(HINSI) $@
 
 # To avoid circular dependency, we don't use 'fzk.master.c'
 fzk.master-.c: fzk.master
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.