On Wed, May 19, 2010 at 8:57 AM, Norbert Preining <prein...@logic.at> wrote:
> On Di, 18 Mai 2010, Andrew SB wrote:
>> - dget http://mentors.debian.net/debian/pool/main/s/sakura/sakura_2.3.8-1.dsc
>
> Some comments: what about the slave x-terminal-emulator.1 man page
> in the update-alternatives call?
>  update-alternatives \
>    --install /usr/bin/x-terminal-emulator \
>              x-terminal-emulator /usr/bin/sakura 40 \
>    --slave /usr/share/man/man1/x-terminal-emulator.1.gz \
>            x-terminal-emulator.1.gz /usr/share/man/man1/sakura.1.gz
> (in postinst)
>
> and
> update-alternatives --remote x-terminal-emulator /usr/bin/sakura
> in prerm (guarded by the proper if "$1" ...
>
>
> Then I would add a menu file in debian:
> ?package(sakura):needs="X11" section="Applications/Terminal Emulators"\
>  title="Sakura" command="/usr/bin/sakura"

Thanks for the review. I've uploaded a revised package to mentors
incorporating you suggestions.

Thanks!

- Andrew SB



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to