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

[debian-devel:18326] Re: xflr5 更新



こんにちは。あかべです。
確認ありがとうございます。

パッケージを修正して再度 mentors にアップしたので、お手数おかけしますが再度確認おねがいします。
http://mentors.debian.net/debian/pool/main/x/xflr5/xflr5_6.07+svn513-1.dsc

On Fri, 29 Jun 2012 07:19:23 +0900
Hideki Yamane <henrich@debian.or.jp> wrote:

> 
> On Thu, 28 Jun 2012 17:53:56 +0900
> Koichi Akabe <vbkaisetsu@xxxxxxxxx> wrote:
> > どなたかスポンサーしていただけませんか?
> > DMUA を yes にして頂けると嬉しいです。
> 
>  それは赤部さんの方で追加下さい。問題なければそのままにします。
> 

追加しました。

> 
>  debian/patches/01_set_targetpath.patch
>  は upstream forward は必要なのでしょうか、それとも debian specific?
>  Origin: タグと Forwarded: をつけてください。
> 
>  あとtarget.path を有効にするのがなぜ必要なのかの why があるといいと思います。
> 

target.path は、生成されたバイナリーのインストール先を指定するものですが、この指定を変更する別の方法が見つからなかったので、パッチを当ててコメントアウトを解除しています。
パッチの説明に「バイナリーのインストール先を設定するため」を追加しました。

このパッチは Debian 向けに自分が作成したので、 Origin などはありませんが、 Author フィールドがあるときは追加する必要があるようですね。[1]
このため、 Author フィールドを削除しました。

> 
> >   * debian/control
> >    - update Standards-Version to 3.9.3
> >    - up debhelper version to 9.20120417
> 
>  debhelper のバージョンが 9.20120417 にした why を書いていただけると
>  なお良いです。
> 

... to add hardening flags for qmake としておきました。

> 
> > 1. ソースファイルが tarball で配布されていないので、debian/rules に get-orig-source ターゲットを追加し、svnからのソースパッケージの作成方法を記しました。[1]
> 
>  素晴らしい、いいですね。ついでに lintian override で debian/watch が無い警告
>  を抑制すると良いと思います。

debian/source/lintian-overrides を追加しました。

6.06-1 からの差分を添付したので、確認おねがいします。
# 今回 diff を使って debian/ 以下の差分を取得しましたが、 debdiff コマンドを使った方法 (debian/以下のみ) について教えて頂けると幸いです。

[1] http://dep.debian.net/deps/dep3/

-- 
Koichi Akabe
  vbkaisetsu at {gmail.com, debian.or.jp}
diff -urN xflr5-6.06/debian/changelog xflr5-6.07+svn513/debian/changelog
--- xflr5-6.06/debian/changelog	2012-02-12 09:21:11.000000000 +0900
+++ xflr5-6.07+svn513/debian/changelog	2012-06-29 18:56:49.815997255 +0900
@@ -1,3 +1,22 @@
+xflr5 (6.07+svn513-1) unstable; urgency=low
+
+  * New upstream release
+  * debian/rules
+   - add get-orig-source target
+  * debian/copyright
+   - update format uri
+  * debian/control
+   - update Standards-Version to 3.9.3
+   - up debhelper version to 9.20120417 to add hardening flags
+     for qmake
+   - enable DM-Upload
+  * debian/compat
+   - up to 9 to add hardening flags
+  * debian/source/lintian-overrides
+   - add debian-watch-file-is-missing because tarball is not privided
+
+ -- Koichi Akabe <vbkaisetsu@xxxxxxxxx>  Fri, 29 Jun 2012 17:22:31 +0900
+
 xflr5 (6.06-1) unstable; urgency=low
 
   * changed and hopefully improved the algorithm which intersects
diff -urN xflr5-6.06/debian/compat xflr5-6.07+svn513/debian/compat
--- xflr5-6.06/debian/compat	2012-01-29 08:58:36.000000000 +0900
+++ xflr5-6.07+svn513/debian/compat	2012-06-28 18:50:24.000000000 +0900
@@ -1 +1 @@
-8
+9
diff -urN xflr5-6.06/debian/control xflr5-6.07+svn513/debian/control
--- xflr5-6.06/debian/control	2012-01-29 10:30:52.000000000 +0900
+++ xflr5-6.07+svn513/debian/control	2012-06-29 08:07:51.000000000 +0900
@@ -2,8 +2,9 @@
 Section: science
 Priority: extra
 Maintainer: Koichi Akabe <vbkaisetsu@xxxxxxxxx>
-Build-Depends: debhelper (>= 8.0.0), docbook-to-man, libqt4-dev, libgl1-mesa-dev, libqt4-opengl-dev (>= 4.6.0), translate-toolkit
-Standards-Version: 3.9.2
+DM-Upload-Allowed: yes
+Build-Depends: debhelper (>= 9.20120417), docbook-to-man, libqt4-dev, libgl1-mesa-dev, libqt4-opengl-dev (>= 4.6.0), translate-toolkit
+Standards-Version: 3.9.3
 Homepage: http://www.xflr5.com/xflr5.htm
 
 Package: xflr5
@@ -13,7 +14,8 @@
 Description: analysis tool for airfoils
  XFLR5 is an analysis tool for airfoils, wings and planes operating at low
  Reynolds Numbers.
- It includes
+ 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
diff -urN xflr5-6.06/debian/copyright xflr5-6.07+svn513/debian/copyright
--- xflr5-6.06/debian/copyright	2012-01-29 08:53:57.000000000 +0900
+++ xflr5-6.07+svn513/debian/copyright	2012-06-21 20:10:32.000000000 +0900
@@ -1,4 +1,4 @@
-Format: http://anonscm.debian.org/viewvc/dep/web/deps/dep5/index.html?revision=248
+Format: http://www.debian.org/doc/packaging-manuals/copyright-format/1.0/
 Upstream-Name: xflr5
 Upstream-Contact: Andre Deperrois <xflr5@xxxxxxxxx>
 Source: http://www.xflr5.com/xflr5.htm
diff -urN xflr5-6.06/debian/patches/01_set_targetpath.patch xflr5-6.07+svn513/debian/patches/01_set_targetpath.patch
--- xflr5-6.06/debian/patches/01_set_targetpath.patch	2012-01-29 09:19:29.000000000 +0900
+++ xflr5-6.07+svn513/debian/patches/01_set_targetpath.patch	2012-06-29 17:14:58.156347166 +0900
@@ -1,11 +1,11 @@
 Description: set target.path on xflr5.pro
- target.path on upstream version is commented out. This patch remove it.
-Author: Koichi Akabe <vbkaisetsu@xxxxxxxxx>
-Last-Update: <2012-01-29>
+ target.path on upstream version is commented out. This patch remove it
+ to set the installation path "/usr/bin" for the binary.
+Last-Update: <2012-06-21>
 
---- xflr5-6.06.orig/xflr5.pro
-+++ xflr5-6.06/xflr5.pro
-@@ -238,7 +238,7 @@ unix {
+--- xflr5-6.07.orig/xflr5_607.pro
++++ xflr5-6.07/xflr5_607.pro
+@@ -244,7 +244,7 @@ unix {
      
      # MAKE INSTALL
      INSTALLS += target
diff -urN xflr5-6.06/debian/rules xflr5-6.07+svn513/debian/rules
--- xflr5-6.06/debian/rules	2011-07-25 16:30:29.000000000 +0900
+++ xflr5-6.07+svn513/debian/rules	2012-06-29 18:58:35.047993539 +0900
@@ -10,5 +10,26 @@
 	lrelease -nounfinished $(CURDIR)/translations/xflr5v6_ja.ts -qm $(CURDIR)/translations/xflr5v6_ja.qm
 	docbook-to-man debian/xflr5.sgml > debian/xflr5.1
 
+override_dh_auto_clean:
+	dh_auto_clean
+	rm -f `find $(CURDIR)/translations -name *.qm` $(CURDIR)/debian/xflr5.1
+
 override_dh_compress:
 	dh_compress -X.pdf
+
+# Adapted from http://wiki.debian.org/SandroTosi/Svn_get-orig-source
+PACKAGE = xflr5
+SRC_VERSION := $(shell dpkg-parsechangelog | sed -ne 's/^Version: \(\([0-9]\+\):\)\?\(.*\)-.*/\3/p')
+SRC_RELEASE := $(shell echo $(SRC_VERSION) | awk -F"+" '{ print $$1 }' )
+SVN_REVISION := $(shell echo $(SRC_VERSION) | awk -F"+" '{ print $$2 }' | sed 's/svn//' )
+TARBALL = $(PACKAGE)_$(SRC_VERSION).orig.tar.bz2
+.PHONY: get-orig-source
+get-orig-source:
+	rm -rf get-orig-source $(TARBALL)
+	mkdir get-orig-source
+	svn export -r $(SVN_REVISION) "https://xflr5.svn.sourceforge.net/svnroot/xflr5/branches/xflr5-$(SRC_RELEASE)" \
+	get-orig-source/$(PACKAGE)-$(SRC_VERSION).orig
+	find get-orig-source/$(PACKAGE)-$(SRC_VERSION).orig -type f -exec chmod -x {} +
+	tar cjf $(TARBALL) -C get-orig-source $(PACKAGE)-$(SRC_VERSION).orig
+	rm -rf get-orig-source
+	echo "  "$(TARBALL)" created; move it to the right destination to build the package"
diff -urN xflr5-6.06/debian/source/lintian-overrides xflr5-6.07+svn513/debian/source/lintian-overrides
--- xflr5-6.06/debian/source/lintian-overrides	1970-01-01 09:00:00.000000000 +0900
+++ xflr5-6.07+svn513/debian/source/lintian-overrides	2012-06-29 17:19:55.480336668 +0900
@@ -0,0 +1,3 @@
+# The upstream doesn't provide source code tar ball. get-orig-source
+# target on debian/rules describes how to get the source.
+debian-watch-file-is-missing