Gergely Nagy <alger...@madhouse-project.org> writes:

> "Desmond O. Chang" <doch...@gmail.com> writes:
>
>> I am looking for a sponsor for my package "clfswm".
>
> Looks fine, and will upload as-is. A couple of suggestions for the
> future, however:
>
> You could simplify debian/rules by using an .install file to install
> /usr/bin/clfswm: Move debian/clfswm.sh to, say debian/bin/clfswm, and
> put "debian/bin/clfswm /usr/bin/" in debian/clfswm.install. That'll do
> the right thing, and you have one override less!
>
> Also, the manpage could be put into debian/clfswm.manpages, and instead
> of building it in an override target, build it at, well, build time, by
> overriding dh_auto_build, or similar.
>
> I think these would make your debian/rules a tiny bit nicer.

Good ideas!  Thank you for pointing them out!


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/ffc1796e-37a2-4977-af60-ae9f512fe...@mail.gmail.com

Reply via email to