Merge branch 'master' of ssh://git.code.sf.net/p/cdesktopenv/code

dtfile: Add missing prototypes

Add missing prototypes and header files to the dtfile
code in order to move closer towards 64-bit compatibility.

Extract the following functions from Motif internal headers:

_XmGetWidgetExtData
_XmRecordEvent
_XmStringUngenerate
_XmTextFieldSetDestination
_XmGetActiveTopLevelMenu

Extract manually prototypes of the obsolete Motif interface:
_XmHighlightBorder
_XmUnhighlightBorder

Remove XmPrivate.h if extractprototype.awk fails

Make the following header files available via -I:
codelibs/boolean.h
codelibs/pathutils.h
codelibs/shellutils.h
and remove shellutils.h from dtwm directory.

dtudcexch/dtudcfonted: Resolve 78 compiler warnings.

Merge branch 'master' of ssh://localhost:3333/p/cdesktopenv/code

CDE-MAN.udb: add /usr/dt/man symlink for linux, like freebsd

Merge branch 'master' of ssh://git.code.sf.net/p/cdesktopenv/code

Improve MANPATH control

* Revert changes to programs/dtsearchpath/libCliSrv/UnixEnv.C
  introduced by c3cb5b8aa6997138dfa06da39142bf793baeae60
  that could have produced disappearing Application Manager
  icons on FreeBSD
* Introduce SearchPath:useSystemPath() virtual method to tell
  dtsearchpath to leave some environment variables alone.

  It is currently overriden for FreeBSD only if the MANPATH
  is empty (system default). Other operating systems
  might want to override it if they prefer to have distribution
  specific control of a search path in effect.

* Symlink /usr/dt/share/man to /usr/dt/man for FreeBSD
  This allows dtsearchpath to actually include /usr/dt/man
  in the MANPATH when MANPATH override is in effect.

dtterm: Resolve 4 compiler warnings

dtstyle: Resolve 28 compiler warnings.

BSD: Add /usr/local/bin to the default PATH on login

FreeBSD: Don't set MANPATH, use OS configuration

- make installer to set up man(1) paths using /usr/local/etc/man.d/*
- don't override empty/unset MANPATH with compiled in default

Don't use fstat for readable pipe chars in dtexec.

The dtexec code assumes that fstat reports pipe's readable chars.
Linux always reports 0 for st_size of a pipe.
Instead read one character when select reports readable.
Note EOF when select says readable but read returns 0.

FreeBSD: Fix dtlogin failsafe session

We need to have /usr/local as the proper
X11 prefix to start mwm and xterm

add HISTORY file

dtlogin: enable building on FreeBSD

Update README file

Allow building on FreeBSD

- build shared libraries with major number only (libtt.so.2)
- don't build dtlogin and dtinfo

FreeBSD support for the installer:
- work around awk issue
- create post_install FreeBSD scripts
- install only cmsd on /etc/inetd.conf
  skip dtspc and ttdbserver for now

Note to users: please check
if you are affected by awk bug
in udbParseLib.awk if you can.

Update CONTRIBUTORS file.

Fix for __va_copy on FreeBSD i386.

Fixes for dtlogin and dtcm on FreeBSD.

linux.cf: add libtirpc support.  Disabled by default.

Currently on Linux, you must run rpcbind in insecure mode (-i) in
order for ttsession to register with rpcbind (the portmapper).  This
is because, on most systems, libc contains an older sun-based rpc
library embedded within it.  The sun-rpc code does not know how to
authenticate with rpcbind when run as a non-privileged user.

Using libtirpc, ttsession can register with rpcbind without requiring
it to be run in insecure mode.  You must have the libtirpc-dev, or
equivalent package installed to use it.

If you want to try this:

- install libtirpc-dev or equivalent
- if your rpcbind process is already running in insecure mode (-i
  option), remove that option and restart it.
- edit config/host.def, (create if it doesn't exist) and add:

  #define HasTIRPCLib  YES

- rebuild CDE (make World).  It's probably a good idea to remove
  /usr/dt/* beforehand to avoid contamination.

Some linux systems seem to incorporate tirpc directly into libc (as
all of the BSD's do AFAIK) so this may not be needed.

I know that at least on Ubuntu systems defining HasTIRPCLib to YES is
required in order to run rpcbind without -i.

message catalogs: fix comment lines, also remove linux hack in merge.c

According to the spec, blank lines in message catalogs or lines
beginning with '$ ' are valid comments.

However, there were many cases where lines in the message catalogs
contained just a single '$', without the required space after it.

Under linux, this caused 126766 error lines (in my builds) of the
form:

... unknown directive `': line ignored

This also causes gencat to exit with a non-0 exit code.  Even though
gencat says it ignores the line, it really doesn't.

An early porting change to programs/localized/util/merge.c was made to
ignore this return value on linux.  This hack has now been removed.

Build logs are a lot smaller and cleaner now.

libDtSvc: Resolve 89 compiler warnings.

libDtSearch: Resolve 26 compiler warnings.

dtpdmd: Resolving warnings.

Merge branch 'master' of ssh://git.code.sf.net/p/cdesktopenv/code

installCDE: remove \c from installing... Log lines

linux.cf: add some gcc system include paths to makedepend.  Shaves about 5150 more warnings off (makedepend) :)

This is on ubuntu.  You may wany to add additional -I... entries for
other linux systems, if the ones I added for i386 and x86_64 don't
help.

dtpdm: Resolve 4 compiler warnings

build: linux.cf, remove the space in the -isystem include of the X11 headers

this was causing build failures on the make depend step.

build: on linux include the X11 and Xm headers with -isystem rather than -I,
this means the compiler treats them as system headers and does not give you
excessive warnings from them. This is used because X11 does not like the
-ansi and -pedantic warnings that CDE is compiled with.

include: remove includes:: XmPrivate.h from includes Imakefile

Having this dependancy in here is a problem.  Depending on how the src
is packaged, or, unpacked, an attempt could be made to regenerate this
file, which cannot succeed unless imports/motif is setup to point to a
compiled motif tree.

This file can be manually regenerated by removing XmPrivate.h, and then
doing a 'make XmPrivate.h' in the include/Xm dir.

dthelp: fix build failure introduced by commit 71f8af6943b9f0ea6d290c1497bf1b3a78be18c0

Tsk Tsk Peter :)

dtcm: fix typo introduced by commit ca9da25d44b258d8ca933808d24668cddd727449

DtWidget: Improve portability of Motif interface

- Add missing includes and prototypes
- Improve type compatibility
- Use <Xm/XmPrivate.h> for unofficial libXm headers

With this patch, dtpad no longer crashes on startup
on FreeBSD/amd64 because of a truncated 64-bit pointer.

Fix warnings in dtwm

- Improve pointer/int compatibility
- Include unpublished Dt headers in Dt
- Use <Xm/XmPrivate.h> for unpublished Motif functions

There are still warnings left generated because
ElementValue.parsed_value should really be a union.

There are also some warnings left because of XtPointer
casting and some unused variables and functions.

Introduce <Xm/XmPrivate.h>

We need ANSI C prototypes of certain Motif
functions that are not published in the official
header files.

<Xm/XmPrivate.h> header file contains
the prototypes extracted from the Motif source.

To re-create <Xm/XmPrivate.h>:

1) Make sure you have sources of libXm
(lib/Xm directory of the Motif distribution)
accessible via imports/motif/lib/Xm

2) rm include/Xm/XmPrivate.h

3) make includes

dtpad: Resolve 6 compiler warnings.

dtlogin: Resolve 7 compiler warnings.

dtlogin: Resolve all -Wformat-security warnings

dtimsstart: Resolve 3 compiler warnings.

Merge branch 'master' of ssh://git.code.sf.net/p/cdesktopenv/code

dthelp: Resolve 106 compiler warnings.

Define AMD64Architecture for FreeBSD

Rudimentary LSBBitOrder for FreeBSD

Merge branch 'master' of ssh://git.code.sf.net/p/cdesktopenv/code

dthello: Resolve 5 compiler warnings.

dtfile: Remove 97 compiler warnings

dthelp/text.c: use snprintf

dtfile: Resolve implicit imcompatible declaration warnings.

dtexec: On Linux use the variation of signal handlers that takes an int arg

Warning Prevention.

dtdspmsg: Warning prevention

dtcreate: warning prevention.

dsdm: Include ansi C headers to prevent implicit incompatible declaration warnings.

dtaction: main() returns an int and -Wformat-security fixes

dticon: Fix some warnings related to mixing NULL, 0 and '\0' randomly.

dtcalc: Fix some warnings related to mixing NULL, 0 and '\0' randomly.

Two fixes for dtmail

Correction of paths for BSDs

dtcm: Resolve all -Wformat-security warnings.

dtsr: include stdlib.h to resolve 20 incompaible implicit declaration warnings.

dtsr: Resolve all -Wformat-security warnings.

util/dttypes: Resolve -Wformat-security warnings

dtterm: Resolve -Wformat-security warnings

libDtTerm: Resolve all -Wformat-security warnings.

Merge branch 'master' of ssh://git.code.sf.net/p/cdesktopenv/code

libDtSearch: resolve -Wformat-secuirty warning

Add pax files generated by dtksh to .gitignore

It seems that after changing -O2 to -O for FreeBSD
pax started to build successfully. Interesting.

libtt: Resolve all remaining format-security warnings.

libtt: Warning prevention

One missing stdlib include and one security warning about not
using varargs func correctly.

Add missing includes [for modern systems] to lndir.c

Fix udbParseLib.awk so it doesn't generate warnings with gawk.

Tested this change with both gawk and nawk and it worked fine.  If
the extra escape character is present and gawk is used, you'll get
warnings from gawk telling you it's ignoring the escape sequence
and just treating it as the character to begin with.

FreeBSD: Use -O instead of -O2

Use KORNSHELL variable instead of /bin/ksh

This patch removes instances of hardcoded
invocation of /bin/ksh and allows to
replace it with, for, example,
/usr/local/bin/ksh93

Also "ksh93" is accepted whenever "ksh" is.

Tested using the following /bin/ksh:

----8<----
WHAT=`ps -o command= -p $PPID`
msg="Something tried to call /bin/ksh: $PPID: $WHAT"
print -u2 "$msg"
logger user.warn "$msg"
exit 99
----8<----
(Warning: first two lines are FreeBSD specific)

Scripts from Makefiles should now be executed either
with

$(KORNSHELL) korn-shell-script

or

$(SHELL) bourne-shell-script

therefore #!/bin/ksh has not been changed everywhere.

/usr/dt/bin/ scripts have been converted (e.g. Xsession)

Whenever possible Imake and CPP facilities have been used.

For C and C++ programs KORNSHELL needs to be defined to
"/path/to/your/ksh" (with quotes) so that it can make
a valid C constant.

Therefore, when adding KORNSHELL to Imakefile for C files,
you have to add

CXXEXTRA_DEFINES = -DKORNSHELL=\"$(KORNSHELL)\"

or similar (for example, see programs/dtprintinfo)

But for simple shell script substitution we usually change

 LOCAL_CPP_DEFINES = -DCDE_CONFIGURATION_TOP=$(CDE_CONFIGURATION_TOP) \
                     -DCDE_INSTALLATION_TOP=$(CDE_INSTALLATION_TOP) \
                     -DCDE_LOGFILES_TOP=$(CDE_LOGFILES_TOP)

to:

 LOCAL_CPP_DEFINES = -DCDE_CONFIGURATION_TOP=$(CDE_CONFIGURATION_TOP) \
                     -DCDE_INSTALLATION_TOP=$(CDE_INSTALLATION_TOP) \
                     -DCDE_LOGFILES_TOP=$(CDE_LOGFILES_TOP) \
                     -DKORNSHELL=$(KORNSHELL) \
                     -DXPROJECTROOT=X11ProjectRoot

since we don't want quotes for shell scripts.

64bit: Fix crash in dtexec

No ANSI prototype for malloc() caused
the returned pointer to be truncated
to 32 bits.

FreeBSD: scan /usr/local/lib/X11/fonts/ for fontpath

Use FreeBSD-specific invocation of /bin/ps

examples/tt now runs on FreeBSD

libtt: resolve implicit delaration of free() and exit() warnings

lib/DtHelp: Include <stdio.h> to prevent warnings about implicit declartion of free()

libcsa: Warning prevention

Remove 'implicit declaration' warnings by included correct ANSI C headers.
Correct some format strings for long ints.

dtcalc: Resolve "format not a string literal and no format arguments [-Wformat-security]" warnings.

Fix warnings related to secruity concerns on varargs functions. By specifying
"%s" on single string calls to sprintf() (and related) it's not possible to
have a % in the input string causing random data to be read off the stack.

dtcalc: Resolve "missing sentinel in function call [-Wformat]" warnings

Caused by XtVaGetValues() and XtVaSetValues() variable arguments lists
needing to be terminated by NULL not 0.

Simple compiler warning fixes in programs/dthelp.

Use mkstemp() and handle /usr/lib64 in imake.c

tmpnam() usage replaced with mkstemp().  Find a suitable tmp directory
checking the TMPDIR environment variable first, then the P_tmpdir
macro and finally /tmp directly.

On 64-bit Linux platforms, check to see if libc.so exists in /usr/lib64.
If found, use it over /usr/lib/libc.so.

Fix basic compiler warnings in ccimake.c

main() always returns an int.

Account for multilib Linux installations in imake.

The libc.so file is not always in /usr/lib.  On multilib systems, the
file we care about could be in /usr/lib64.  Likewise, common Linux
conventions call for 64-bit libraries to go in lib64 directories, so
check there first when on a Linux 64-bit system.

Include stdlib.h in cde/config/imake/ccimake.c

stdlib.h defines exit(3) on Linux systems.

Fix localized build on OpenBSD.

Use the same set of langs as on Linux and FreeBSD (no Japanese), don't
redefine a needed macro as no-op, and unset LC_CTYPE in the environment
when building cat files.

Add mkcatdefs and book.* to .gitignore

Build dtksh on OpenBSD.

This needs ksh93 to bootstrap, available in the openbsd-wip for now as a port,
due to be committed some time soon ...

OpenBSD still uses GNU iconv, so the second argument to iconv(3) is not const.

Fix dttypes for BSD systems

Find correct system includes and libs on FreeBSD.

Add mkcatdefs program and make the imake changes so it can be used.

Build missing Xm.cat Mrm.cat Uil.cat for LANG=C

We need to use mkcatdefs to build those.

mkcatdefs needs to be built from the open motif
source tree (localized/util/mkcatdefs.c)
and installed as:

cde/imports/motif/localized/util/mkcatdefs

dtcreate: Replace broken GetBaseName with basename

GetBaseName causes segfaults, because when the pathname
ends in a "/" it returns NULL. This happens when trying
to give a valid filename to dtcreate for an icon.

dtcreate: Fix some memory management issues

ProcessExecString thought it was returning an array of size 3; however
in C arrays are second-class and there is no direct way to return an
array like this; GCC warning triggered because it was actually
just returning a pointer to local storage. Fixed using malloc.

Also fix some obviously wrong usages of sizeof, although they were
relatively harmless. A little other warning quieting using 0 instead of
NULL.

dtcreate: Fix another buffer overflow issue

Merge branch 'EnvControl-fixes'