[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[debian-devel:17919] Re: xflr5 スポンサー募集
こんにちは。Koichi Akabeです。
ご指導ありがとうございます。
修正した版をアップロードしました。
http://mentors.debian.net/debian/pool/main/x/xflr5/xflr5_6.03-5.dsc
以下の点を修正しましたので、よろしくお願いします。
・セクションを x11 => science に変更しました。
・install ファイルを利用してコピーするようにしました。
・changelog の余分な空白を無くしました。
・make clean が2つあった件ですが、make 前の make clean を残し、2つめは特
に意味はないので無くしました。
・lrelease コマンドを、 override_dh_auto_install から
override_dh_auto_build に移動しました。
・makefile は qmake によって生成されるので、 qmake したあとに make する
のは問題ないはずです。
・/usr/sbin を削除しました。
watchは便利なようですが、上流のsourceforgeサイトを見ると、ソースコードを
入手する方法はsvn以外に無いようです。
watchを使ってsubversionからソースコードをチェックアウトする方法があれ
ば、ご教授よろしくお願いします。
http://sourceforge.net/projects/xflr5/files/
(2011年03月26日 01:53), Hideki Yamane wrote:
> 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には、Kalibpthread.so.0 could be roliina 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 の変更に追従
> しやすくなります。
>
>
>