[debian-devel:17927] Re: RFS: goldencheetah

[Tatsuya Kinoshita <tats@xxxxxxxxxxxxxx>]

On March 26, 2011, [debian-devel:17918],
lurdan (at gmail.com) wrote:

> サイクルコンピュータのデータをあつかう goldencheetah 
> というアプリケーションを mentors に put しています。
> 
> - URL: http://mentors.debian.net/debian/pool/main/g/goldencheetah
> - Source repository: deb-src http://mentors.debian.net/debian unstable main contrib non-free
> - dget http://mentors.debian.net/debian/pool/main/g/goldencheetah/goldencheetah_2.0.0-1.dsc
> 
> どなたかスポンサーをお願いできませんか。

引き受けようかと思います。ざっと見てみました。以下コメントです。

debian/patches/*:

自動生成されたパッチがそのまま残っており、未整理なのではないで
しょうか。あと、できればDEP-3タグを付けてください。

  DEP-3: Patch Tagging Guidelines
  http://dep.debian.net/deps/dep3/

debian/docs:

| README

READMEにはbuildとMac OS X環境のことしか書かれていないので、
インストール不要ではないでしょうか。

  Debian Policy Manual, 12.3 Additional documentation
  http://www.debian.org/doc/debian-policy/ch-docs.html#s12.3

debian/goldencheetah.doc-base:

| Index: /usr/share/doc/goldencheetah/index.html
| Files: /usr/share/doc/goldencheetah/*.html

このindex.htmlがあると、ウェブサーバ経由でREADME.Debian等へ
アクセスしづらくなるのではないでしょうか。upstreamのdoc/*.html
はサブディレクトリ(たとえば/usr/share/doc/goldencheetah/html)
へインストールしてはどうでしょうか。

debian/control:

| Description: Cycling Performance Software

大文字化は不要です。また、もう少し情報を加えてはどうでしょうか。
(たとえば、"set of analysis tools to boost cycling performance")

  Debian Developer's Reference, 6.2.2. The package synopsis, or short description
  http://www.debian.org/doc/developers-reference/best-pkging-practices.en.html#bpp-pkg-synopsis

*.orig.tar.gz:

upstreamの*.orig.tar.gzそのままではなく、手元で作り直したのなら、
README.sourceにその旨を記載してください。

upstreamのリリース版ソースtarballは見あたらなかったのですが、
githubを使っているようですので、githubからダウンロードした
*.tar.gzがそのまま使えないか検討してみてください。

  4.14 Source package handling: debian/README.source
  http://www.debian.org/doc/debian-policy/ch-source.html#s-readmesource

  Debian Developer's Reference, 6.7.8. Best practices for .orig.tar.{gz,bz2,lzma} files
  http://www.debian.org/doc/developers-reference/best-pkging-practices.en.html#bpp-origtargz

debian/copyright:

| <Copyright (C) 2006 Sean Rhea>

<〜>の囲みは不要です。あと、Qwtのファイルを抱え込んでいるなど、
ほかのcopyright表記が見られます。不要なファイルを除去した上での
*.orig.tar.gz作成も検討してみてください。

| License:
| 
|     /usr/share/common-licenses/GPL-2

src/main.cppを見る限りでは、version 2のみではなく、version 2
or laterのようです。

| The Debian packaging is:
| 
|     Copyright (C) 2010 KURASHIKI Satoru <lurdan@xxxxxxxxx>
| 
| and is licensed under the GPL version 2,
| see "/usr/share/common-licenses/GPL-2".

version 2のみにしているのは意図的なのでしょうか。もしupstreamが
version 3に移行しても、自分のパッチを許可なくマージすることは
許さない、ということでないなら、version 2 or laterにするか、
または無制限に許可するライセンスにしてはどうでしょうか。

-- 
木下達也