Re: [801bab95] issues with man pages

2012-09-25 Thread Andrew Borodin 
On Tue, 25 Sep 2012 07:37:14 +0200 (CEST) Werner LEMBERG w...@gnu.org wrote:
 compilation and installation of commit 801bab95 on my GNU/Linux box
 using the standard incantation (./autogen.sh, ./configure; sudo make
 install) was just fine, however, there are three issues, the first of
 them serious.
 
 (1) mc fails to find the resource directory if called for man pages.
 Going to a (uncompressed) man page `foo.man' and pressing F3
 yields
 
   /tmp/mc-wl/mcextQeBmEa: line 14:
 /lib/mc/ext.d/text.sh: No such file or directory
 
 Obviously, it misses the `/usr/local' prefix.

Cannot reproduce. Works find for me.

Please show the output of
/usr/local/bin/mc -V  /usr/local/bin/mc -F

-- 
Аndrew
___
mc mailing list
https://mail.gnome.org/mailman/listinfo/mc

Re: [801bab95] issues with man pages

2012-09-25 Thread Andrew Borodin 
On Tue, 25 Sep 2012 12:36:29 +0200 (CEST) Werner LEMBERG wrote:
 the `exec_prefix' variable doesn't get expanded:

${exec_prefix}/lib/mc/ext.d/text.sh view man lessa

As first step, would you please try build mc with the attached patch.

Thanks!

-- 
Andrew
diff --git a/configure.ac b/configure.ac
index cfa7208..55eecde 100644
--- a/configure.ac
+++ b/configure.ac
@@ -425,7 +425,7 @@ dnl 
 dnl ${prefix} and ${exec_prefix} are undefined here if --prefix is not used in command line
 dnl Let define ${prefix} and ${exec_prefix}
 test x$prefix = xNONE  prefix=$ac_default_prefix
-test x$exec_prefix = xNONE  exec_prefix='${prefix}'
+test x$exec_prefix = xNONE  exec_prefix=${prefix}
 
 if test x${libexecdir} = x'${exec_prefix}/libexec'; then
 EXTHELPERSDIR=${prefix}/libexec/${PACKAGE}/ext.d
___
mc mailing list
https://mail.gnome.org/mailman/listinfo/mc

Re: [801bab95] issues with man pages

2012-09-25 Thread Werner LEMBERG 
 Obviously, it misses the `/usr/local' prefix.
 
 Cannot reproduce. Works find for me.
 
 Please show the output of
 /usr/local/bin/mc -V  /usr/local/bin/mc -F

Below is the output from 

  LANG= /usr/local/bin/mc -V  LANG= /usr/local/bin/mc -F

Evrything looks fine.  However, it seems that in /tmp/mc-wl/mcext
the `exec_prefix' variable doesn't get expanded:

  #! /bin/sh

  MC_EXT_FILENAME=/home/wl/cvs/groff/man/ditroff.man
  export MC_EXT_FILENAME
  MC_EXT_BASENAME=ditroff.man
  export MC_EXT_BASENAME
  MC_EXT_CURRENTDIR=/home/wl/cvs/groff/man
  export MC_EXT_CURRENTDIR
  MC_EXT_SELECTED=ditroff.man
  export MC_EXT_SELECTED
  MC_EXT_ONLYTAGGED=
  export MC_EXT_ONLYTAGGED

   ${exec_prefix}/lib/mc/ext.d/text.sh view man lessa

I've just discovered that F3 doesn't work for other files also,
e.g. pressing this key on a .tgz file I get a similar error message.

For completeness, I've attached my config.log file.


Werner

==

GNU Midnight Commander 4.8.6-2-gee12f0c
Built with GLib 2.31.12
Using the S-Lang library with terminfo database
With builtin Editor
With subshell support as default
With support for background operations
With mouse support on xterm and Linux console
With support for X11 events
With internationalization support
With multiple codepages support
Virtual File Systems: cpiofs, tarfs, sfs, extfs, ext2undelfs, ftpfs, fish
Data types: char: 8; int: 32; long: 32; void *: 32; size_t: 32; off_t: 64;
Root directory: /home/wl

[System data]
Config directory: /usr/local/etc/mc/
Data directory:   /usr/local/share/mc/
VFS plugins and scripts: /usr/local/lib/mc/
extfs.d:/usr/local/lib/mc/extfs.d/
fish:   /usr/local/lib/mc/fish/

[User data]
Config directory: /home/wl/.config/mc/
Data directory:   /home/wl/.local/share/mc/
skins:  /home/wl/.local/share/mc/skins/
extfs.d:/home/wl/.local/share/mc/extfs.d/
fish:   /home/wl/.local/share/mc/fish/
mcedit macros:  /home/wl/.local/share/mc/mc.macros
mcedit external macros: /home/wl/.local/share/mc/mcedit/macros.d/macro.*
Cache directory:  /home/wl/.cache/mc/


config.log.xz
Description: Binary data
___
mc mailing list
https://mail.gnome.org/mailman/listinfo/mc

Re: [801bab95] issues with man pages

2012-09-25 Thread Andrew Borodin 
On Tue, 25 Sep 2012 12:36:29 +0200 (CEST) Werner LEMBERG  wrote:
 the `exec_prefix' variable doesn't get expanded:

${exec_prefix}/lib/mc/ext.d/text.sh view man lessa

Please test patch
https://www.midnight-commander.org/changeset/684eaa426b76a791a54b1bc8f29cc37e9dfbb8e7

Thanks!

-- 
Andrew
___
mc mailing list
https://mail.gnome.org/mailman/listinfo/mc

Re: [801bab95] issues with man pages

2012-09-25 Thread Werner LEMBERG 

 Please test patch
 https://www.midnight-commander.org/changeset/684eaa426b76a791a54b1bc8f29cc37e9dfbb8e7

It works!

Thanks for the quick fix.


Werner
___
mc mailing list
https://mail.gnome.org/mailman/listinfo/mc

[801bab95] issues with man pages

2012-09-24 Thread Werner LEMBERG 

Folks,


compilation and installation of commit 801bab95 on my GNU/Linux box
using the standard incantation (./autogen.sh, ./configure; sudo make
install) was just fine, however, there are three issues, the first of
them serious.

(1) mc fails to find the resource directory if called for man pages.
Going to a (uncompressed) man page `foo.man' and pressing F3
yields

  /tmp/mc-wl/mcextQeBmEa: line 14:
/lib/mc/ext.d/text.sh: No such file or directory

Obviously, it misses the `/usr/local' prefix.

(2) mc doesn't respect first-line comments in man pages like

  '\ t

or

  .\ e

to call `tbl' and `eqn', respectively.  I suggest to always use
the pipe

  ... | tbl | eqn | nroff ...

(in which doesn't do any harm and is fast enough today to stay
unnoticed by the user.

(3) Looking into `ext.d/text.sh' I see that you always call nroff with
`-Tlatin1'.  I strongly suggest to remove that.  Recent versions
of groff's nroff script check the locale to automatically select
the right -T parameter.


Werner
___
mc mailing list
https://mail.gnome.org/mailman/listinfo/mc