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

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: