[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[debian-devel:18316] Re: RFS: hyperestraier



 やまねです。途中ですけど…

On Tue, 5 Jun 2012 20:21:25 +0900
Satoru KURASHIKI <lurdan@xxxxxxxxx> wrote:
>   * Apply hardening build flags.

 bhlc かけたら CFLAGS missing が結構出てきました。
 対応するパッチを添付します。


 あと、debian/rules をみてみました。

JAVA_UNSUPPORTED_CPUS = zhppaz zmipsz

 なのに debian/control ファイルでは
>>Build-Depends: openjdk-6-jdk [!kfreebsd-i386 !kfreebsd-amd64 !s390x] 
 となっています。hppa mips がサポート外のように見えるのですが…?
 
 出来れば rules を書き直した方が後々のメンテナンスに良いと思います。



 debian/control で
>>Build-Depends: debhelper (>= 6.0.7)

 これは古い! dpkg-buildflags を rules で使っているのに、これは適切ではないです。
 (>= 9) にするのが良いと思います。debian/compat も合わせて修正してください。


>   * Drop to build Java binding in some Archs. (Closes: #673164)

 出来れば changelog に workaround であり、もし kfreebsd でビルドできたら再度
 enable にするなどの文言を書いておいて頂けると後から見た人がパッとわかって
 よいのでは、と思いました。


 最後に些細なところ、control で
>>Package: libestraier-java
>>Architecture: linux-amd64 armel linux-i386 ia64 mips mipsel powerpc s390 sparc alpha armhf ppc64

 amd64 i386 で linux- はいらないのでは。


-- 
Regards,

 Hideki Yamane     henrich @ debian.or.jp/org
 http://wiki.debian.org/HidekiYamane

Attachment: Makefile.in
Description: Binary data