On Fri, Dec 10, 2010 at 03:43:04PM +0100, Pierre Habouzit wrote:
> On Thu, Dec 09, 2010 at 12:38:26PM -0500, Decklin Foster wrote:
> > Excerpts from Marc Lehmann's message of Thu Dec 09 06:30:03 -0500 2010:
> > > The original urxvt uses "rxvt-unicode" and "rxvt-unicode-256color"
> > > TERM values, respectively, which are the correct values, and portable.
> > 
> > By "respectively", what are you referring to?
> > 
> > > as added "benefit" debian wouldn't be needlessly incompatible to the
> > > rest of the world (which uses the correct terminfo files).
> > 
> > There is no rxvt-unicode-256color terminfo entry in Debian or the
> > rxvt-unicode distribution that I can see, but (see above) I don't know
> > exactly what you mean here.
> 
> I know why, this is because the build-system doesn't use "install" but
> "tic" (from ncurses), which is deceptive. tic writes the entry in
> /usr/share/terminfo if root, and in ~/.terminfo/ if user.
> 
> If you want it to be built in the debian package properly, you have to
> do something like that in the debian/rules (I'll let you adapt
> properly):
> 
> TERMINFO=debian/tmp/usr/share/terminfo
> 
> [...]
> install:
>       mkdir $(TERMINFO)
>       TERMINFO=$(TERMINFO) make install prefix=...
> 
> It will create a debian/tmp/usr/share/terminfo/r/rxvt-unicode{,-256color}
> 
> 
> FWIW rxvt-unicode is unfriendly to package systems because it doesn't
> install the terminfo database respecting the "prefix" env variable
> (which is why you, Decklin, don't find this file).
> 
> A better patch on urxvt is to change (untested) the doc/Makefile.in, this way:
> 
> diff --git a/doc/Makefile.in b/doc/Makefile.in
> index 6a7d223..77b8ee6 100644
> --- a/doc/Makefile.in
> +++ b/doc/Makefile.in
> @@ -117,7 +117,7 @@ install: all
>         $(INSTALL_DATA) rxvt.7.man  
> $(DESTDIR)$(man7dir)/$(RXVTNAME).$(man7ext)
>  @IF_PERL@      $(INSTALL) -d $(DESTDIR)$(man3dir)
>  @IF_PERL@      $(INSTALL_DATA) rxvtperl.3.man 
> $(DESTDIR)$(man3dir)/$(RXVTNAME)perl.$(man3ext)
> -       @TIC@ $(srcdir)/etc/rxvt-unicode.terminfo
> +       TERMINFO=$(prefix)/usr/share/terminfo @TIC@ 
> $(srcdir)/etc/rxvt-unicode.terminfo
>  
>  distdepend: alldoc

okay it's slightly wrong, because you have to mkdir that path before.
And in fact, DESTDIR should be used rather than a straight $(prefix).
But I said it was untested. It likely more looks like:

-       @TIC@ $(srcdir)/etc/rxvt-unicode.terminfo
+       $(INSTALL) -d $(DESTDIR)/usr/share/terminfo
+       TERMINFO=$(DESTDIR)/usr/share/terminfo @TIC@ 
$(srcdir)/etc/rxvt-unicode.terminfo

Or even better, using a terminfodir variable that is configureable from
configure and defaults to /usr/share/terminfo. But that exceeds my autoconf
skills by far ;)

Anyways Decklin, you have the rought idea now, I'll let you fix the last 
details.
-- 
·O·  Pierre Habouzit
··O                                                madco...@debian.org
OOO                                                http://www.madism.org



--
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