[debian-devel:17917] Re: xflr5 スポンサー募集

[Hideki Yamane <henrich@debian.or.jp>]

On Fri, 25 Mar 2011 14:22:04 +0900
Koichi Akabe <vbkaisetsu@xxxxxxxxx> wrote:
> 修正を加えたので、再度よろしくお願いします。
> http://mentors.debian.net/debian/pool/main/x/xflr5/xflr5_6.03-4.dsc
> 
> changelogには、Karoliina Salminenさんの(過去の)変更分を追加しておきました。
> 他はchangelogの通りです。

　debian/control
　　section は x11 以外に適当なものってないですかねぇ

　debian/rules
　　install コマンドでインストールしている部分がありますね。ファイル名が
　　変わらないものは、debian/install ファイルに記述した方が良いです。
　　xflr5.desktop.in は処理されないので、xflr5.desktop にして install ファイル
　　に放り込んでも良いかも。このファイル見ると Comment=an analysis tool for airfoils
　　とあるけど、an 要りますかね?
　　あと、install ターゲットなのに build にあった方が良いのがあります。

　　$(MAKE) がありますが、これって Makefile 無いですよね? では make コマンドを
　　実行されても意味が無いのでは…
　　あと $(MAKE) clean が2ヶ所ありますが、これの意味は？

　　最初の .pro ファイルの指定も他に無ければ * で指定しても良いのでは？

　　あと ./ とカレントディレクトリ指定するのは、自分だとどこを指されるか
　　分からないので $(CURDIR)/ のようにしますね。


　試しにビルドしてみました。

drwxr-xr-x root/root         0 2011-03-26 00:35 ./
drwxr-xr-x root/root         0 2011-03-26 00:35 ./usr/
drwxr-xr-x root/root         0 2011-03-26 00:36 ./usr/bin/
-rwxr-xr-x root/root   3167184 2011-03-26 00:36 ./usr/bin/xflr5
drwxr-xr-x root/root         0 2011-03-26 00:35 ./usr/sbin/

　/usr/sbin が空ですが、これは大丈夫?　大丈夫なら debian/dirs から削る方が良いです。


　debian/changelog
　　あまり空行を使わないでうまく圧縮してください
　　
　* hogefuga
    + hogehoge
      - fugafuga

　のようにして。よっぽど目立たせたい時以外は不要な空行はなしで。

　
　あと、debian/watch ファイルをできれば追加しましょう。upstream の変更に追従
　しやすくなります。



-- 
Regards,

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