MC 4.7.0.1 on AIX - several build problem

[tammer]

Hello,
I just tried to upgrade mc (from 4.6.2 to 4.7.0.1) on AIX. There are several build problem (with IBM XL C) on AIX.

1. src/tty/tty-ncurses.c

- cur_term->Nttyb.c_cc[VINTR] = CTRL ('g'); /* <sup>g */
+ cur_term->Nttyb.c_cc[VINTR] = unctrl ('g'); /*</sup> g */

CTRL() is not there, I have used unctrl() instead.

2. src/unixcompat.h

#if defined(_AIX) && defined(HAVE_SYS_SYSMACROS_H)
# include <sys/sysmacros.h> /* AIX */

+# include <time.h> /* AIX tm */

#endif

I had to add <time.h> because tm as undefined

3. vfs/direntry.c

-#include <sys/fcntl.h>
+#if defined(_AIX)
+# include <fcntl.h>
+#else
+# include <sys/fcntl.h>
+#endif

The directory for fcntl.h was not correct. This should be changed in the configure system.

4. vfs/vfs.c

+#if defined(_AIX) && !defined(NAME_MAX)
+#define NAME_MAX FILENAME_MAX
+#endif

This is a bit fuzzy but it should work

5. Makefile

CFLAGS = -fdiagnostics-show-option ...

The -fdiagnostics-show-option does not work with cc_r (IBM XL C). Unfortunately the configure test is not catching this.

6. Screen library

The flag --with-screen=mcslang is no longer working because the slang library is no longer bundled. The bundled version was very good on AIX...

7. opendir()

In mc opendir() is used even if large file support is on. This leads to problems as the opendir() can not handle large inodes (> 32 bit). It would be a good idea to use the XXX64() versions if large file support is enabled.

Bye

Rainer

[tammer]

Hello,
one small addition to 5.:

This is not caught by configure as -fXXX is a valid option for XL C during compile time. But the meaning is completely different:

-f<file_name>

Names a file to store a list of object files for
the compiler to pass to the linker. The <file_name>
file should contain only the names of object files.
There should be one object file per line.

Bye

Rianer

[tammer]

Hello,
this is a proposal for an patch for 5.:

m4.include/mc-cflags.m4

dnl Sorted -f options:
AC_MSG_CHECKING([CC is $CC])
case "$CC" in

gcc)

MC_CHECK_ONE_CFLAG([-fdiagnostics-show-option])

dnl MC_CHECK_ONE_CFLAG([-fno-stack-protector])

;;

*)

;;

esac

As this is a gcc option we only need to check if gcc is used.

Bye

Rainer

[andrew_b]

Replying to tammer:

6. Screen library

The flag --with-screen=mcslang is no longer working because the slang library is no longer bundled. The bundled version was very good on AIX...

bundled version of S-Lang was version 1.x. Now MC uses S-Lang of 2.x version.

Would you provide a patch in unified format (diff -rup) for this ticket?

[andrew_b]

Replying to andrew_b:

bundled version of S-Lang was version 1.x. Now MC uses S-Lang of 2.x version.

Oh, sorry. It's my mistake. Bundled version was 2.0.6.

[tammer]

Hello,
so the bundled version should still work. Any change to bundle it again ??

Yes,
I can provide a patch. I first wanted to discuss if my solutions for the problems are acceptable to you.

Bye

Rainer

[andrew_b]

Replying to tammer:

Hello,
so the bundled version should still work. Any change to bundle it again ??

No. MC will use upstream (and installed in system) libraries.

Yes,
I can provide a patch. I first wanted to discuss if my solutions for the problems are acceptable to you.

None of MC developers does not use AIX. We just can trust you.

[tammer]

Hello,
good. Should I open a new ticket for every fix or should I provide a combined fix for the AIX build problems ?

As a base for the fix I will use HEAD from git.

The CTRL() thing was not quite right, ... the CTRL() is only:
#define CTRL(x) ((x) & 0x1f)
So I will go with that version.

Bye

Rainer

[tammer]

Hello,
one addition to ncurses. If slang is installed there is no big problem, but if ncurses is installed the the configure tools will pick up the system curses and kinda ignore the ncurses library. I had to manually tweak two Makefiles to pick up ncurses. To use ncurses on AIX we would need --with-ncurses-includes / --with-ncurses-libs. I think I will go with the slang version in the first round. If I can find some additional spare time I try to add the --withXXX parameters.

Bye

Rainer

[slavazanko]

good. Should I open a new ticket for every fix?

No need. Just attach one-fix patch-files into this ticket, please.

[tammer]

Hello,
I have prepared a first fix for current HEAD.

With this patch you can compile MC 4.7 with the IBM XL C/C++ compiler. It is required that slang is installed.

This is my configure call:

#!/usr/bin/bash

export CC=cc_r
export CXX=xlC_r
export CFLAGS="-I/opt/freeware/include"
export LDFLAGS="-Wl,-brtl"
export CONFIG_SHELL=/usr/bin/bash
export SHELL=/usr/bin/bash
./configure --prefix=/usr/local/mc \

--with-libiconv-prefix=/opt/freeware \
--sysconfdir=/etc \
--with-screen=slang \
--with-slang-includes=/usr/local/slang/include \
--with-slang-libs=/usr/local/slang/lib \
--without-x

Bye

Rainer

[tammer]

0001-MC-4.7-patches-for-AIX.patch

[tammer]

Hello,
as soon as I find some time I will have a look at the ncurses stuff.

Am I right that slang is the preferred screen library ?

Bye

Rainer

[slavazanko]

Am I right that slang is the preferred screen library ?

Yes, S-Lang preferred. But ncurses important too, if in some reason target system totally don't have S-Lang :)

Created branch 1957_build_AIX
Changesets:

• 444ea4dd1bc9bcaeb729b54be7f3f30cfdc1f4e8: MC 4.7.0.1 on AIX - fixed several build problems

[slavazanko]

As andrew_b saw, we can just trust to you...

And thanks for your help.

[angel_il]

test build under tcc passed

[tammer]

Hello,
good to hear :-).

I will try to get a look at the ncurses stuff on the weekend. This is probably a bit more work.

Bye

Rainer

[andrew_b]

Replying to tammer:

I will try to get a look at the ncurses stuff on the weekend. This is probably a bit more work.

OK. Branch is frozen until your result.

[tammer]

Hello,
My ncurses version is 5.2, with TERM=dtterm color support is working. Other TERM settings do not work properly (xterm, vt220, vt420). This would be something for the INSTALL file... The slang library is working better in this regard.

I have prepared a first version which accepts the new two configure parameters --with-ncurses-includes/--with-ncurses-lib.

If you specify these two parameters then configure looks for ncurses/ncurses.h and ncurses/term.h or ncurses.h and term.h in the specified directory.

The problem on AIX is that the system supplies a term.h which is not compatible with the ncurses version of term.h. So you have to be very careful not to pick up the wrong version... :-((

Bye

Rainer

[metux]

Some things (eg. including <time.h> and <fcntl.h>) are
IMHO a generic - not AIX specific - thing, so why the
#ifdef's ?

[tammer]

Hello,
I have already changed this in the first part of the patch.

vfs/direntry.c:
... snip ...
#include <fcntl.h> /* include fcntl.h -> sys/fcntl.h only */

/* includes fcntl.h see IEEE Std 1003.1-2008 */

... snip ...

The second part (--with-ncurses-XX) of the patch is almost ready. I have implemented it in the following way:

1. If the parms are not given then the old logic is used
2. If the parms are given then configure looks in the specified location for the ncurses headers/lib.

It works on AIX but I need someone testing it on Linux. I will post the patch later this day. On AIX slang is the better solution for the screen library. I could get the ncurses build only working with TERM=dtterm.

Bye

Rainer

[tammer]

second patch version

[tammer]

Hello,
this is a combined patch against HEAD (2010-01-17).

With this patch you can:

Compile on AIX with S-Lang 2.0.7 or ncurses 5.2.

Please could someone test this on Linux.

Bye

Rainer

[slavazanko]

Branch rebased. New initial changeset:e8c574f4bd20602c6b80f06936a55195fba947dd

Review, please.

[tammer]

Hello,
if there are any questions about this patch please feel free to ask.
Bye

Rainer

[angel_il]

i have no problem with TCC.

[slavazanko]

merge master: bf951d52edd2fcc35fe4476e4678c4058a43873b
stable: b8ff2cfc1a15b054e56621efaec7471d79a1c497