Re: #200264 and xmessage
On 30.04.04 Frank Küster (frank@debian.org) wrote:
> Hilmar Preusse <hille42@web.de> schrieb:
Hi all,
> > example usage:
> > ./texdoctk-warn "please install the package\n\"perl-tk\" to use texdoctk"
>
> Thank you, I've tested it, works well. This is my suggestion for
> texdoctk itself:
>
> ***************************************************
> @@ -1,6 +1,4 @@
> -#!/usr/bin/env perl
> -# We started with a hack provided by Thomas Esser. This expression replaces
> @@ -28,9 +28,9 @@
> + $message = "$progname: problem loading the Tk module:\n $@\nHave you installed the perl-tk package?\n";
> + }
> +
> -+ if (! -t STDOUT and exists $ENV{DISPLAY}
> -+ and system("command -v xmessage >/dev/null 2>&1")==0) {
> -+ exec("xmessage", $message);
> ++ if (! -t STDOUT and exists $ENV{DISPLAY})
> ++ {
> ++ exec("texdoctk-warn", $message);
> + } else {
> + die $message;
> + }
> ***************************************************
>
Yes, something like that.
> Furthermore, Policy requires a man page, I'd suggest
>
To me it seems to be overkill to write a man page for the small
program. IMHO it is temporary anyway, cause after the big split[TM]
we could texdoctk separate again and then revert the Recommend back to
an Depend. Anyway, before anybody starts whining, that we deliver out
5KB-programs without man page... Thanks for your work!
> ***************************************************
>
> .TH "texdoctk-warn" "1" "April 2004" "teTeX" "teTeX"
^^^^^^^^^^^^^^^^
at least the date is correct.
> I'll check that in with the respective changes to debian/rules, if
> it's o.k. for you.
>
It is.
Thanks again,
H.
--
Nothing is easier than to denounce the evildoer; nothing is more difficult
than to understand him.
- Fyodor Dostoevski
http://hilmarpreusse.forum-rheinland.de/
Reply to: