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

[debian-devel:18519] Re: tinkerer_1.0.2-1



やまねさん

前田です。
返信が大変遅くなってすみません。

2013年1月28日 16:24 Hideki Yamane <henrich@debian.or.jp>:
>  http://mentors.debian.net/debian/pool/main/t/tinkerer/tinkerer_1.0.2-1.dsc
>  を拝見しました。
>
>  - DH_VERBOSE がonになったままでした。気をつけてください。

修正しました。

>
>>        python2.7 setup.py build --build-base=build2 && \
>>        python3.2 setup.py build --build-base=build3
>
>  2.7 や 3.2 と細かいバージョンまで指定する必要はありますか?
>  3.2 が消えて 3.3 になった場合等に問題になると思います。
>  単に
>
>         python setup.py build --build-base=build2 && \
>         python3 setup.py build --build-base=build3
>
>  で良いのでは無いでしょうか。(X-Python-Versionとの兼ね合いでどうするか
>  がちょっと分かりませんが)

override_auto_test 向けに、pyversions -vr, py3versions -vrを網羅するように修正したので、
こちらも同様にしています。

>> override_dh_python2:
>>         dh_python2 && \
>>         mkdir -p $(CURDIR)/debian/python-tinkerer/usr/share/python-tinkerer/ && \
>>         cp -af $(CURDIR)/debian/python-tinkerer/usr/share/pyshared/tinkerer/themes $(CURDIR)/debian/python-tinkerer/usr/share/python-tinkerer/
>>         rm -f $(CURDIR)/debian/python-tinkerer/usr/share/pyshared/tinkerer/themes/boilerplate/LICENSE.md
>>         rm -f $(CURDIR)/debian/python-tinkerer/usr/share/python-tinkerer/themes/boilerplate/LICENSE.md
>
>  LICENSE.md が不要であれば
>
>  find $(CURDIR)/debian/python-tinkerer/ -name LICENSE.md -delete
>
>  の方が意図が通りやすいと思います。

こちらそのように修正しました。

>> override_dh_auto_clean:
>>         python2.7 setup.py clean --build-temp=build  && \
>>         python2.7 setup.py clean --build-temp=build2 && \
>>         python3.2 setup.py clean --build-temp=build3 && \
>>         find $(CURDIR) -name "*.pyc" -delete && \
>
>  最後の find は必要ですか?
>  最初に dh_clean を呼んで、例外処理だけを書く方が良いと思います。

dh_cleanでは、__init__.pycが削除されないので、追加しており、今回も残してあります。


>  - 次回で構いませんので、debian/patches 以下のパッチについて、DEP3 に
>  したがって情報とdescriptionを追加いただけますか。パッチを drop する際
>  などにパッと判断ができます。

追加しました。


また、tikererer 1.1がリリースされたので、1.1にしてmentorsにアップロードしました。

http://mentors.debian.net/debian/pool/main/t/tinkerer/tinkerer_1.1-1.dsc

よろしくお願いします。
--
Kouhei Maeda <mkouhei at gmail.com>
 B8F2 86F6 2206 360F 7D41  0817 2E81 6254 7E37 CE41