xflr5 スポンサー募集
Koichi Akabe
Koichi Akabeと申します。
飛行機の翼型解析ソフト「XFLR5」のパッケージを作成しました。
http://mentors.debian.net/debian/pool/main/x/xflr5/
XFLR5(upstream)
http://xflr5.sourceforge.net/xflr5.htm
どなたかスポンサーになっていただけないでしょうか。
よろしくお願いします。
赤部晃一
xflr5のdscへのリンクです。よろしくお願いします。
http://mentors.debian.net/debian/pool/main/x/xflr5/xflr5_6.03-3.dsc
2011/03/24 15:00 "Koichi Akabe" <vbkai...@gmail.com>:
初めまして。
Koichi Akabeと申します。
飛行機の翼型解析ソフト「XFLR5」のパッケージを作成しました。
Hideki Yamane
やまねです。
On Thu, 24 Mar 2011 14:59:53 +0900
Koichi Akabe <vbkai...@gmail.com> wrote:
> 飛行機の翼型解析ソフト「XFLR5」のパッケージを作成しました。
> http://mentors.debian.net/debian/pool/main/x/xflr5/
まだざっとしか見てないのですが、いくつか確認点。
debian/control
Package: xflr5
Architecture: any
Depends: ${shlibs:Depends}, ${misc:Depends}
Description: XFLR5 is an analysis tool for airfoils,
wings and planes operating at low Reynolds Numbers.
It includes
1. XFoil's Direct and Inverse analysis capabilities
2. Wing design analysis based on the Lifiting Line
Theory and the Vortex Lattice Method
description の書き方が間違っています。
short は1行だけ、その後に long description が続きます。
http://www.debian.or.jp/community/devel/debian-policy-ja/policy.ja.html/ch-controlfields.html#s-f-Description
Package: xflr5-language-de
Architecture: all
Depends: xflr5, ${misc:Depends}
Description: language data for XFLR5 (de)
Package: xflr5-language-fr
Architecture: all
Depends: xflr5, ${misc:Depends}
Description: language data for XFLR5 (fr)
short description だけで long がありません。
debian/changelog の方
> xflr5 (6.03-3) unstable; urgency=low
>
> * Changed build depends
何から何に？
debian/copyright
This package was debianized by Karoliina Salminen <karoliina....@gmail.com>
とありますが、changelog に記載が無いのはなぜ？
あと、可能であれば今からだと DEP5 形式にするのが良いでしょう
http://dep.debian.net/deps/dep5/ や、developer news などの記事その他を参考。
debian/README.Debian
これは必要?
debian/rules
Build-Depends で debhelper (>= 7.0) なのに dh7 が使っていないので dh7 形式にする
のをお勧めします。
debian/convts
bash 依存する必要ある?
--
Regards,
Hideki Yamane henrich @ debian.or.jp/org
http://wiki.debian.org/HidekiYamane
Koichi Akabe
修正を加えたので、再度よろしくお願いします。
http://mentors.debian.net/debian/pool/main/x/xflr5/xflr5_6.03-4.dsc
changelogには、Karoliina Salminenさんの(過去の)変更分を追加しておきました。
他はchangelogの通りです。
Hideki Yamane
Koichi Akabe <vbkai...@gmail.com> 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 の変更に追従
しやすくなります。
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 <vbkai...@gmail.com> wrote:
>> 修正を加えたので、再度よろしくお願いします。
>> http://mentors.debian.net/debian/pool/main/x/xflr5/xflr5_6.03-4.dsc
>>
>> changelogには、Kalibpthread.so.0 could be roliina Salminenさんの(過去の)変更分を追加しておきました。
Hideki Yamane
Koichi Akabe <vbkai...@gmail.com> wrote:
> 修正した版をアップロードしました。
> http://mentors.debian.net/debian/pool/main/x/xflr5/xflr5_6.03-5.dsc
changelog が -2 から -5 にいきなり飛ぶのは良くないですねぇ。
-3,-4 とかは残しておくべきだと思います。
debian/rules 中の2つの変数は参照しなくなるので不要かと
APPNAME := xflr5
VERSION := 603
$(MAKE) clean は特段必要ないのではないかなぁ、と思うのですが (特に削除する
必要のあるファイルが見当たらないのと、必要であれば dh_cleanターゲット
がそれをしてくれるはずなので)
watchファイルの件は了解しました。upstream に行って tarball を作ってもらう
などを検討するのが良いでしょうか。
あと、6.04 が出てるようですね。こちらにあげてもいいのでは無いでしょうか。
Hideki Yamane
Koichi Akabe <vbkai...@gmail.com> wrote:
あと description は適当なところで改行しないように
Description: analysis tool for airfoils
xflr5 is an analysis tool for airfoils, wings and planes operating at low
Reynolds Numbers.
.
It includes
1. XFoil's Direct and Inverse analysis capabilities
2. Wing design analysis based on the Lifiting Line Theory and the Vortex
Lattice Method
てなかんじでしょうか。
Koichi Akabe
修正したものをアップロードしました。おねがいします。
http://mentors.debian.net/debian/pool/main/x/xflr5/xflr5_6.03-6.dsc
tarballの件を開発者のAndreさんに伝えておきました。
debianパッケージ作成について事前に連絡を入れておきましたが、
その時の話によると6.04は4月か5月にリリースされるとのことです。
(実際に今現在、6.04の所には頻繁にコミットが行われているようです。)
Hideki Yamane
やまねです。
On Sun, 27 Mar 2011 09:35:00 +0900
Koichi Akabe <vbkai...@gmail.com> wrote:
> 修正したものをアップロードしました。おねがいします。
> http://mentors.debian.net/debian/pool/main/x/xflr5/xflr5_6.03-6.dsc
あと一息ですね。すごーく細かいですが typo が changelog にあります。
revision を上げるほどでもないので、次の機会にでも直しておいてください。
もう1点、このソフトの有用性とか、どんな方が使うのか、など、背景を
教えて頂けますか? スポンサーするとしたら、どのような物かを知って
おきたいです。
＃私には「飛行機の翼型解析」だけではちょっと分からないので…
> tarballの件を開発者のAndreさんに伝えておきました。
> debianパッケージ作成について事前に連絡を入れておきましたが、
> その時の話によると6.04は4月か5月にリリースされるとのことです。
> (実際に今現在、6.04の所には頻繁にコミットが行われているようです。)
了解です。あとは initial パッケージングをした方にも一応断りと
パッケージの精査を頼んでみてはいかがでしょうか。可能であれば、
co-maintainer になってもらうのが良いでしょう。
＃その連絡の際にはCCに当方を入れておいてくださいませ。
Koichi Akabe
お世話になります。
typo:
remobed => removed
分かりました。
精査・co-maintainerの件、了解しました。
Karoliina Salminenさんに連絡を入れておきます。
XFLR5について:
XFLR5は、低レイノルズ数の飛行機(ラジコン機、滑空機、人力飛行機など)の翼
を設計するときに使うソフトです。翼型(翼の断面図)等の機体 データを入力す
ることにより、揚力や空気の流れ、安定性などを計算できます。
元々コマンドベースのxfoilという翼型解析ソフトがあるのですが、XFLR5は
xfoilの解析アルゴリズムをそのままGUI環境に移植し、 グラフ表示機能などを
追加したものです。
XFLR5による解析と、実際に飛んでいる様子:
http://xflr5.sourceforge.net/Screenshots/T-Joe.htm
Hideki Yamane
Koichi Akabe <vbkai...@gmail.com> wrote:
> XFLR5による解析と、実際に飛んでいる様子:
> http://xflr5.sourceforge.net/Screenshots/T-Joe.htm
飛んでいる様子がいいですね :)
後出しですいません、lintian でいくつか引っかかっていました。
少なくとも Warning は修正しておきましょう。
W: xflr5: binary-without-manpage usr/bin/xflr5
N:
N: Each binary in /usr/bin, /usr/sbin, /bin, /sbin or /usr/games should
N: have a manual page
N:
N: Note that though the man program has the capability to check for several
N: program names in the NAMES section, each of these programs should have
N: its own manual page (a symbolic link to the appropriate manual page is
N: sufficient) because other manual page viewers such as xman or tkman
N: don't support this.
N:
N: If the name of the man page differs from the binary by case, man may be
N: able to find it anyway; however, it is still best practice to make the
N: case of the man page match the case of the binary.
N:
N: If the man pages are provided by another package on which this package
N: depends, lintian may not be able to determine that man pages are
N: available. In this case, after confirming that all binaries do have man
N: pages after this package and its dependencies are installed, please add
N: a lintian override.
N:
N: Refer to Debian Policy Manual section 12.1 (Manual pages) for details.
N:
N: Severity: normal, Certainty: possible
N:
man が無いので書きましょう。
W: xflr5: executable-not-elf-or-script ./usr/share/pixmaps/xflr5.png
N:
N: This executable file is not an ELF format binary, and does not start
N: with the #! sequence that marks interpreted scripts. It might be a sh
N: script that fails to name /bin/sh as its shell, or it may be incorrectly
N: marked as executable. Sometimes upstream files developed on Windows are
N: marked unnecessarily as executable on other systems.
N:
N: If you are using debhelper to build your package, running dh_fixperms
N: will often correct this problem for you.
N:
N: Refer to Debian Policy Manual section 10.4 (Scripts) for details.
N:
N: Severity: normal, Certainty: certain
N:
+x されている？
I: xflr5: spelling-error-in-binary ./usr/bin/xflr5 Perfoming Performing
N:
N: Lintian found a spelling error in the given binary. Lintian has a list
N: of common misspellings that it looks for. It does not have a dictionary
N: like a spelling checker does.
N:
N: If the string containing the spelling error is translated with the help
N: of gettext or a similar tool, please fix the error in the translations
N: as well as the English text to avoid making the translations fuzzy. With
N: gettext, for example, this means you should also fix the spelling
N: mistake in the corresponding msgids in the *.po files.
N:
N: You can often find the word in the source code by running:
N:
N: grep -rw <word> <source-tree>
N:
N: This tag may produce false positives for words that contain non-ASCII
N: characters due to limitations in strings.
N:
N: Severity: minor, Certainty: wild-guess
N:
I: xflr5: spelling-error-in-binary ./usr/bin/xflr5 Continous Continuous
I: xflr5: spelling-error-in-binary ./usr/bin/xflr5 developped developed
元々のバイナリに spelling の間違いがある。パッチ作って、upstream に
投げておくのが良いかと。
Koichi Akabe
ご指摘有難うございます。
修正版をアップしました。確認宜しくお願いします。
http://mentors.debian.net/debian/pool/main/x/xflr5/xflr5_6.03-7.dsc
アップロード後に気づいたのですが、changelogスペルミスの訂正がうまく出来
ていませんでした。次の版までに修正します。
修正点:
manページを作成しました。特に引数指定などはないので、debian/controlの
descriptionsからの引用と、ライセンス表 示のみを書きました。
xflr5.png のパーミッションですが、sourceforgeからsubversionでダウンロー
ドしたソースファイルの恐らく全てに実行権限が付与されていたの で、開発者
の方にスペルミスの件とともに確認のメールをしました。
こちらでの一時的な対応として、rulesファイルにinstallコマンドを設置して
パーミッションを指定した上でコピーする方法を採用しよう と思うのですが、
ご意見お願いします
スペルミスを訂正するパッチをdebian/patchesを使って当てようと思ったのです
が、元のソースコードの改行コードがCRLFのため、 パッチを当てる際にエラー
が発生してしまいます。調べたところ、パッチを当てる前にtrコマンドでCRを消
去する方法を見つけました。しかし、こ れを適用するとrulesが煩雑化する上、
翻訳データの該当個所にも影響が出てくるので、こちらでのパッチは保留としま
した。
一週間前に、initial debianizeされたKaroliina Salminenさんにメールを送っ
たのですが返信がありません。返信を待つ必要があるでしょうか。それとも確認
のメールをもう一度入れて作業を進めてし まっていいのでしょうか。
よろしくお願いします。
Hideki Yamane
Koichi Akabe <vbkai...@gmail.com> wrote:
> 修正版をアップしました。確認宜しくお願いします。
> http://mentors.debian.net/debian/pool/main/x/xflr5/xflr5_6.03-7.dsc
間が開いてすいません。確認したのでアップロードしました。