Bug#764792: x11-apps: xman not rendering any pages (zsoelim: not found)
On Sun, Oct 26, 2014 at 11:08:52PM +0100, Julien Cristau wrote: Thanks. As a quick hack this seems to work; there's probably a way to actually check for man-db... Looks like an improvement. Maybe just check whether mandb is on the path with AC_CHECK_PROG, since other man(1) implementations don't include that program. -- Colin Watson [cjwat...@debian.org] -- To UNSUBSCRIBE, email to debian-x-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org Archive: https://lists.debian.org/20141107160942.gt5...@riva.ucam.org
Bug#764792: x11-apps: xman not rendering any pages (zsoelim: not found)
Control: tag -1 patch
On Sat, Oct 11, 2014 at 14:12:12 +0100, Colin Watson wrote:
On Sat, Oct 11, 2014 at 01:28:34PM +0200, Julien Cristau wrote:
On Sat, Oct 11, 2014 at 02:45:17 -0400, Lee Cremeans wrote:
I was trying out xman on my jessie system, and it turns out it won't
render
any pages at all. I notice that when I start xman from a terminal, I see
sh: 1: zsoelim: not found
in the terminal when I tell it to load a man page. Some checking shows
that
zsoelim was in man-db in wheezy, but that no longer seems to be the case
with man-db in jessie on AMD64.
man-db 2.7.0's changelog says Move zsoelim to /usr/lib/man-db/.
Colin, how should I handle this in xman? Hardcode the path, patch out
the use of zsoelim, something else?
Wow that's an evil mess, and almost certainly broken with anything other
than ISO-8859-1. I would suggest burning it to the ground and using
man -l or similar, at least on systems that have man-db, because xman
appears to be just doing a bad job of reimplementing it. Failing that
it would be OK to hardcode /usr/lib/man-db/zsoelim for now (although you
might want to test it and try both).
Sorry for the inconvenience; I thought this was an internal utility
nobody else was using, so it seemed a no-brainer to move it when it was
conflicting with other packages on other systems ...
Thanks. As a quick hack this seems to work; there's probably a way to
actually check for man-db...
diff --git a/configure.ac b/configure.ac
index 9a80cbb..ced29c7 100644
--- a/configure.ac
+++ b/configure.ac
@@ -44,6 +44,13 @@ if test x$GROFF = xfound ; then
AC_DEFINE([HAS_GROFF], 1, [Define to 1 if you have the groff package.])
fi
+AC_ARG_ENABLE([man-db],
+ AS_HELP_STRING([--enable-man-db], [Use man -l to format pages
instead of our own pipeline (use this if your man implementation is man-db)]),
+ [MANDB=$enableval], [MANDB=no])
+if test x$MANDB = xyes; then
+ AC_DEFINE([HAVE_MANDB], 1, [Define if your man implementation is
man-db])
+fi
+
AC_CHECK_FUNCS([mkstemp])
AC_ARG_WITH(helpdir,
diff --git a/misc.c b/misc.c
index 06891cd..da8744a 100644
--- a/misc.c
+++ b/misc.c
@@ -661,6 +661,13 @@ static Boolean
ConstructCommand(char *cmdbuf, const char *path,
const char *filename, const char *tempfile)
{
+#ifdef HAVE_MANDB
+int used = snprintf(cmdbuf, BUFSIZ, man -l %s %s 2/dev/null,
+filename, tempfile);
+if (used = BUFSIZ - 1)
+ return FALSE;
+return TRUE;
+#else
/* The original code did the following to produce a command line:
* sprintf(cmdbuf,cd %s ; %s %s %s %s %s, path, TBL,
* filename, FORMAT, man_globals-tempfile, 2 /dev/null);
@@ -783,6 +790,7 @@ ConstructCommand(char *cmdbuf, const char *path,
return (FALSE);
return (TRUE);
+#endif /* man-db */
}
#endif /* HANDLE_ROFFSEQ */
Cheers,
Julien
signature.asc
Description: Digital signature
Processed: Re: Bug#764792: x11-apps: xman not rendering any pages (zsoelim: not found)
Processing control commands: tag -1 patch Bug #764792 [x11-apps] x11-apps: xman not rendering any pages (zsoelim: not found) Added tag(s) patch. -- 764792: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=764792 Debian Bug Tracking System Contact ow...@bugs.debian.org with problems -- To UNSUBSCRIBE, email to debian-x-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org Archive: https://lists.debian.org/handler.s.b764792.141436133710417.transcr...@bugs.debian.org
Bug#764792: x11-apps: xman not rendering any pages (zsoelim: not found)
Package: x11-apps Version: 7.7+3 Severity: important Dear Maintainer, I was trying out xman on my jessie system, and it turns out it won't render any pages at all. I notice that when I start xman from a terminal, I see sh: 1: zsoelim: not found in the terminal when I tell it to load a man page. Some checking shows that zsoelim was in man-db in wheezy, but that no longer seems to be the case with man-db in jessie on AMD64. -- System Information: Debian Release: jessie/sid APT prefers testing-updates APT policy: (500, 'testing-updates'), (500, 'testing') Architecture: amd64 (x86_64) Foreign Architectures: i386 Kernel: Linux 3.16-2-amd64 (SMP w/8 CPU cores) Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8) Shell: /bin/sh linked to /bin/dash Versions of packages x11-apps depends on: ii cpp 4:4.9.1-1 ii libc62.19-11 ii libpng12-0 1.2.50-2 ii libsm6 2:1.2.2-1 ii libx11-6 2:1.6.2-3 ii libxaw7 2:1.0.12-2 ii libxcursor1 1:1.1.14-1 ii libxext6 2:1.3.2-1 ii libxft2 2.3.2-1 ii libxkbfile1 1:1.0.8-1 ii libxmu6 2:1.1.2-1 ii libxmuu1 2:1.1.2-1 ii libxrender1 1:0.9.8-1 ii libxt6 1:1.1.4-1 Versions of packages x11-apps recommends: ii xbitmaps 1.1.1-2 Versions of packages x11-apps suggests: ii mesa-utils 8.2.0-1 -- no debconf information
Bug#764792: x11-apps: xman not rendering any pages (zsoelim: not found)
On Sat, Oct 11, 2014 at 02:45:17 -0400, Lee Cremeans wrote: Package: x11-apps Version: 7.7+3 Severity: important Dear Maintainer, I was trying out xman on my jessie system, and it turns out it won't render any pages at all. I notice that when I start xman from a terminal, I see sh: 1: zsoelim: not found in the terminal when I tell it to load a man page. Some checking shows that zsoelim was in man-db in wheezy, but that no longer seems to be the case with man-db in jessie on AMD64. man-db 2.7.0's changelog says Move zsoelim to /usr/lib/man-db/. Colin, how should I handle this in xman? Hardcode the path, patch out the use of zsoelim, something else? Thanks, Julien signature.asc Description: Digital signature
Bug#764792: x11-apps: xman not rendering any pages (zsoelim: not found)
On Sat, Oct 11, 2014 at 01:28:34PM +0200, Julien Cristau wrote: On Sat, Oct 11, 2014 at 02:45:17 -0400, Lee Cremeans wrote: I was trying out xman on my jessie system, and it turns out it won't render any pages at all. I notice that when I start xman from a terminal, I see sh: 1: zsoelim: not found in the terminal when I tell it to load a man page. Some checking shows that zsoelim was in man-db in wheezy, but that no longer seems to be the case with man-db in jessie on AMD64. man-db 2.7.0's changelog says Move zsoelim to /usr/lib/man-db/. Colin, how should I handle this in xman? Hardcode the path, patch out the use of zsoelim, something else? Wow that's an evil mess, and almost certainly broken with anything other than ISO-8859-1. I would suggest burning it to the ground and using man -l or similar, at least on systems that have man-db, because xman appears to be just doing a bad job of reimplementing it. Failing that it would be OK to hardcode /usr/lib/man-db/zsoelim for now (although you might want to test it and try both). Sorry for the inconvenience; I thought this was an internal utility nobody else was using, so it seemed a no-brainer to move it when it was conflicting with other packages on other systems ... -- Colin Watson [cjwat...@debian.org] -- To UNSUBSCRIBE, email to debian-x-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org Archive: https://lists.debian.org/20141011131212.gj5...@riva.ucam.org
