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

[debian-devel:18499] tinkerer_1.0.2-1

前田さん

　やまねです。
　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_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 を呼んで、例外処理だけを書く方が良いと思います。


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


-- 
Regards,

 Hideki Yamane     henrich @ debian.or.jp/org
 http://wiki.debian.org/HidekiYamane