Re: [PATCH] aclocal.m4: Enable configure to detect the presence ofterminfo database on OpenBSD

2003-02-01 Thread Pavel Tsekov
On Fri, 31 Jan 2003, Pavel Roskin wrote:

> > I admit that I didn't knew how S-Lang interfaces with the terminfo database
> > before I browsed through the code today. However, IMHO, what is
> > incompatible here is the S-Lang library itself - there are a couple of
> > routines which deal with the terminfo database and if S-Lang used them
> > instead of emulating them everything would be fine.
> 
> Are those functions in the curses library?  You can ask in the S-Lang
> mailing list, but I think it will be easier to reimplement those functions
> rather than link to -lcurses.  S-Lang used to depend on -lcurses in the
> past on some UNIX systems, but this dependency was later eliminated.

Yep! I figured that out by myself. I've started a patch but it seems like 
it'll be taking some time before I can complete it. I'll notify you if I 
eventually finish it up.

___
Mc-devel mailing list
[EMAIL PROTECTED]
http://mail.gnome.org/mailman/listinfo/mc-devel



Re: [PATCH] aclocal.m4: Enable configure to detect the presence ofterminfo database on OpenBSD

2003-01-31 Thread Pavel Roskin
Hello!

> > I don't understand what exactly your patch supposed to fix.  What is the
> > problem?  Have you tested your patch?  Does the problem go away?
>
> Yes, the problem goes away. If it's not clear the problem is that
> configure doesn't detect the presence of terminfo database on OpenBSD thus
> thinking that terminfo is not available at all and this is not true.

OK, then it's not a problem, because configure does the right thing.

> I admit that I didn't knew how S-Lang interfaces with the terminfo database
> before I browsed through the code today. However, IMHO, what is
> incompatible here is the S-Lang library itself - there are a couple of
> routines which deal with the terminfo database and if S-Lang used them
> instead of emulating them everything would be fine.

Are those functions in the curses library?  You can ask in the S-Lang
mailing list, but I think it will be easier to reimplement those functions
rather than link to -lcurses.  S-Lang used to depend on -lcurses in the
past on some UNIX systems, but this dependency was later eliminated.

-- 
Regards,
Pavel Roskin
___
Mc-devel mailing list
[EMAIL PROTECTED]
http://mail.gnome.org/mailman/listinfo/mc-devel



Re: [PATCH] aclocal.m4: Enable configure to detect the presence ofterminfo database on OpenBSD

2003-01-30 Thread Pavel Tsekov
On Thu, 30 Jan 2003, Pavel Roskin wrote:

> Hello!
> 
> > Here is another simple patch to improve configure's behaviour on OpenBSD.
> 
> I don't understand what exactly your patch supposed to fix.  What is the
> problem?  Have you tested your patch?  Does the problem go away?

Yes, the problem goes away. If it's not clear the problem is that 
configure doesn't detect the presence of terminfo database on OpenBSD thus 
thinking that terminfo is not available at all and this is not true.

Now, as I comes to testing the patch - yes I did and it worked. However 
reading your post I decided to verify again. This time I used gdb so I can 
be 100% sure - it turns out that the test I did was faked, not 
intentionally of course. The varibale TERMCAP was set in my .profile 
file and I didn't notice it. This and the following code in _SLtt_tigetent
doomed my test:

/* If we are on a termcap based system, use termcap */
if (0 == tcap_getent (term, ti)) return ti;

Please, read below...

> > [..]
> > terminfo(5) on OpenBSD says:
> > /usr/share/terminfo.db  file containing terminal descrip-
> > tions on OpenBSD
> 
> OK, but your patch is for the MC_WITH_MCSLANG macro, which is only called
> when using the included S-Lang library.  S-Lang doesn't understand
> terminfo in a file.

> It seems to me that your patch would force mc to use terminfo on a system
> where terminfo is only available in an incompatible format.

Yes, you're right.

I admit that I didn't knew how S-Lang interfaces with the terminfo database
before I browsed through the code today. However, IMHO, what is 
incompatible here is the S-Lang library itself - there are a couple of 
routines which deal with the terminfo database and if S-Lang used them 
instead of emulating them everything would be fine.

Thanks, for catching this!

___
Mc-devel mailing list
[EMAIL PROTECTED]
http://mail.gnome.org/mailman/listinfo/mc-devel



Re: [PATCH] aclocal.m4: Enable configure to detect the presence ofterminfo database on OpenBSD

2003-01-29 Thread Pavel Roskin
Hello!

> Here is another simple patch to improve configure's behaviour on OpenBSD.

I don't understand what exactly your patch supposed to fix.  What is the
problem?  Have you tested your patch?  Does the problem go away?

> [..]
> terminfo(5) on OpenBSD says:
> /usr/share/terminfo.db  file containing terminal descrip-
> tions on OpenBSD

OK, but your patch is for the MC_WITH_MCSLANG macro, which is only called
when using the included S-Lang library.  S-Lang doesn't understand
terminfo in a file.

It seems to me that your patch would force mc to use terminfo on a system
where terminfo is only available in an incompatible format.

-- 
Regards,
Pavel Roskin
___
Mc-devel mailing list
[EMAIL PROTECTED]
http://mail.gnome.org/mailman/listinfo/mc-devel



[PATCH] aclocal.m4: Enable configure to detect the presence ofterminfo database on OpenBSD

2003-01-29 Thread Pavel Tsekov
Hello,

Here is another simple patch to improve configure's behaviour on OpenBSD. 

[..]
terminfo(5) on OpenBSD says:
/usr/share/terminfo.db  file containing terminal descrip-
tions on OpenBSD
[..]

It appears that on OpenBSD 3.2 (i386) the rigth file to look
for is /usr/share/misc/terminfo.db .

Thanks!


--- aclocal.m4  2003-01-29 18:33:51.0 +0100
+++ aclocal.m4.OpenBSD.patch2   2003-01-29 18:33:43.0 +0100
@@ -792,6 +792,19 @@ AC_DEFUN([MC_WITH_MCSLANG], [
use_terminfo=yes
fi
done
+   # terminfo(5) on OpenBSD say:
+   # /usr/share/terminfo.db  file containing terminal descrip-
+   # tions on OpenBSD
+   #
+   # It appears that on OpenBSD 3.2 (i386) the rigth file to look
+   # for is /usr/share/misc/terminfo.db .
+   #
+   for tifile in "/usr/share/terminfo.db" \
+ "/usr/share/misc/terminfo.db" ; do
+   if test -f "$tifile" ; then
+   use_terminfo=yes
+   fi
+   done
 fi
 
 # If there is no terminfo, use termcap