On Mon, 13 Aug 2012, Joshuah Hurst wrote:

> On Mon, Aug 13, 2012 at 12:59 PM, Marcin Cieslak <sa...@saper.info> wrote:
> > On Mon, 13 Aug 2012, Irek Szczesniak wrote:
> >
> >> On Mon, Aug 13, 2012 at 4:01 AM, Marcin Cieslak <sa...@saper.info> wrote:
> >> > ---
> >> >  cde/doc/C/Imakefile    |    2 +-
> >> >  cde/programs/Imakefile |    2 +-
> >> >  2 files changed, 2 insertions(+), 2 deletions(-)
> >
> >> Why are you disabling dtksh? It's an integral part of CDE and required
> >> for functionality like DtBuilder.
> >
> > It does not compile right now; the build of dtksh is temporarily
> > disabled for Linux so it seems we need to do for FreeBSD
> > for the build to continue.
> >
> > I fully agree we need to get it working.
> 
> At some point in the future we need to look at updating the ksh93 base
> of dtksh. dtksh is currently based on ksh93d-, which is an alpha
> version. The current AT&T ksh stable upstream version is ksh93u+, ~15
> years newer, much more stable and much more portable. Runs on Linux
> (Suse, Fedora, embedded, etc), FreeBSD, NetBSD, Solaris, Illumos,
> HP/UX, too.
> The license issues (if there are actually any, so far I haven't seen
> proof by a lawyer) are solvable by using libshell.so.1, which is ksh93
> as shared library.

Additionally, we need some Korn-compatible shell for building,
to run admin/IntegTools/dbTools/udbToAny.sh script.

For Linux distributions using bash not much is needed,
but I would propose to introduce KSHELL variable
that points to the Korn-compatible shell (the 
default should be /bin/ksh). 

Because BSD make does not run scripts from
the current directory (it uses $PATH just
like the shell) and ./ construct do not 
work, I would propose changing script
invocations like this:

KSHELL should really be defined to config/cf/...
(this is just an example that works
for me on FreeBSD):

diff --git a/cde/admin/IntegTools/post_install/Imakefile 
b/cde/admin/IntegTools/post_install/Imakefile
index f8e3d8d..4c8a6ec 100644
--- a/cde/admin/IntegTools/post_install/Imakefile
+++ b/cde/admin/IntegTools/post_install/Imakefile
@@ -9,8 +9,10 @@ XCOMM $TOG: Imakefile /main/13 1998/03/11 16:20:10 mgreess $
 #endif
 
 TOOL_DIR = ../dbTools
+KSHELL = /usr/local/bin/ksh93
 DATABASE_DIR = ../../../databases
-BUILD_UDB_TOOL = ./build_udb_list
+BUILD_UDB_TOOL = $(SHELL) build_udb_list
+UDB_TOOL = $(KSHELL) $(TOOL_DIR)/udbToAny.ksh
 
 #ifdef HPArchitecture
 PLATFORM = hp-ux
@@ -50,62 +52,62 @@ MakeSubdirs($(SUBDIRS))
 all::
 
 configHelp:: $(DATABASE_DIR)/CDE-HELP-C.udb
-       $(TOOL_DIR)/udbToAny.ksh  -toLst -ReleaseStream $(PLATFORM) \
+       $(UDB_TOOL)  -toLst -ReleaseStream $(PLATFORM) \
             $(DATABASE_DIR)/CDE-HELP-C.udb > CDE-HELP-C.lst
        $(BUILD_UDB_TOOL) CDE-HELP-C ../../..
 
 configHelpPrg:: $(DATABASE_DIR)/CDE-HELP-PRG.udb
-       $(TOOL_DIR)/udbToAny.ksh  -toLst -ReleaseStream $(PLATFORM) \
+       $(UDB_TOOL)  -toLst -ReleaseStream $(PLATFORM) \
             $(DATABASE_DIR)/CDE-HELP-PRG.udb > CDE-HELP-PRG.lst
        $(BUILD_UDB_TOOL) CDE-HELP-PRG ../../..
 
 configDemos:: $(DATABASE_DIR)/CDE-DEMOS.udb
-       $(TOOL_DIR)/udbToAny.ksh  -toLst -ReleaseStream $(PLATFORM) \
+       $(UDB_TOOL)  -toLst -ReleaseStream $(PLATFORM) \
             $(DATABASE_DIR)/CDE-DEMOS.udb > CDE-DEMOS.lst
        $(BUILD_UDB_TOOL) CDE-DEMOS ../../..
 
 configFonts:: $(DATABASE_DIR)/CDE-FONTS.udb
-       $(TOOL_DIR)/udbToAny.ksh  -toLst -ReleaseStream $(PLATFORM) \
+       $(UDB_TOOL)  -toLst -ReleaseStream $(PLATFORM) \
             $(DATABASE_DIR)/CDE-FONTS.udb > CDE-FONTS.lst
        $(BUILD_UDB_TOOL) CDE-FONTS ../../..
 
 configHelpRun:: $(DATABASE_DIR)/CDE-HELP-RUN.udb
-       $(TOOL_DIR)/udbToAny.ksh  -toLst -ReleaseStream $(PLATFORM) \
+       $(UDB_TOOL)  -toLst -ReleaseStream $(PLATFORM) \
             $(DATABASE_DIR)/CDE-HELP-RUN.udb > CDE-HELP-RUN.lst
        $(BUILD_UDB_TOOL) CDE-HELP-RUN ../../..
 
 configIcons:: $(DATABASE_DIR)/CDE-ICONS.udb
-       $(TOOL_DIR)/udbToAny.ksh  -toLst -ReleaseStream $(PLATFORM) \
+       $(UDB_TOOL)  -toLst -ReleaseStream $(PLATFORM) \
             $(DATABASE_DIR)/CDE-ICONS.udb > CDE-ICONS.lst
        $(BUILD_UDB_TOOL) CDE-ICONS ../../..
 
 configInc:: $(DATABASE_DIR)/CDE-INC.udb
-       $(TOOL_DIR)/udbToAny.ksh  -toLst -ReleaseStream $(PLATFORM) \
+       $(UDB_TOOL)  -toLst -ReleaseStream $(PLATFORM) \
             $(DATABASE_DIR)/CDE-INC.udb > CDE-INC.lst
        $(BUILD_UDB_TOOL) CDE-INC ../../..
 
 configManDev:: $(DATABASE_DIR)/CDE-MAN-DEV.udb
-       $(TOOL_DIR)/udbToAny.ksh  -toLst -ReleaseStream $(PLATFORM) \
+       $(UDB_TOOL)  -toLst -ReleaseStream $(PLATFORM) \
             $(DATABASE_DIR)/CDE-MAN-DEV.udb > CDE-MAN-DEV.lst
        $(BUILD_UDB_TOOL) CDE-MAN-DEV ../../..
 
 configMan:: $(DATABASE_DIR)/CDE-MAN.udb
-       $(TOOL_DIR)/udbToAny.ksh  -toLst -ReleaseStream $(PLATFORM) \
+       $(UDB_TOOL)  -toLst -ReleaseStream $(PLATFORM) \
             $(DATABASE_DIR)/CDE-MAN.udb > CDE-MAN.lst
        $(BUILD_UDB_TOOL) CDE-MAN ../../..
 
 configMsgCat:: $(DATABASE_DIR)/CDE-MSG-C.udb
-       $(TOOL_DIR)/udbToAny.ksh  -toLst -ReleaseStream $(PLATFORM) \
+       $(UDB_TOOL)  -toLst -ReleaseStream $(PLATFORM) \
             $(DATABASE_DIR)/CDE-MSG-C.udb > CDE-MSG-C.lst
        $(BUILD_UDB_TOOL) CDE-MSG-C ../../..
 
 configPrg:: $(DATABASE_DIR)/CDE-PRG.udb
-       $(TOOL_DIR)/udbToAny.ksh  -toLst -ReleaseStream $(PLATFORM) \
+       $(UDB_TOOL)  -toLst -ReleaseStream $(PLATFORM) \
             $(DATABASE_DIR)/CDE-PRG.udb > CDE-PRG.lst
        $(BUILD_UDB_TOOL) CDE-PRG ../../..
 
 configShlibs:: $(DATABASE_DIR)/CDE-SHLIBS.udb
-       $(TOOL_DIR)/udbToAny.ksh  -toLst -ReleaseStream $(PLATFORM) \
+       $(UDB_TOOL)  -toLst -ReleaseStream $(PLATFORM) \
             $(DATABASE_DIR)/CDE-SHLIBS.udb > CDE-SHLIBS.lst
        $(BUILD_UDB_TOOL) CDE-SHLIBS ../../..
 
diff --git a/cde/databases/Imakefile b/cde/databases/Imakefile
index 2db62e4..2fde9f8 100644
--- a/cde/databases/Imakefile
+++ b/cde/databases/Imakefile
@@ -91,13 +91,14 @@ CDE-INFOLIB-Locale.udb:: InfoDB                             
                @@\
 .SUFFIXES: .db .udb
 
 TOOLSRC=../
+KSHELL=/usr/local/bin/ksh93 -x
 DBTOOLS=admin/IntegTools/dbTools
 CONVERSION= -toDB
 
 .udb.db:
        @echo "making $*.db"
        @$(RM) $*.db
-       $(TOOLSRC)/$(DBTOOLS)/udbToAny.ksh $(CONVERSION) $(UDB_RELEASE_OPT) $< 
> $*.db
+       $(KSHELL) $(TOOLSRC)/$(DBTOOLS)/udbToAny.ksh $(CONVERSION) 
$(UDB_RELEASE_OPT) $< > $*.db
 
 all::
 
diff --git a/cde/programs/dtprintinfo/Imakefile 
b/cde/programs/dtprintinfo/Imakefile
index ed42f6e..82ead70 100644
--- a/cde/programs/dtprintinfo/Imakefile
+++ b/cde/programs/dtprintinfo/Imakefile
@@ -68,7 +68,7 @@ DEFINES = -D_TOOLTALK
 #endif
 
 /* special processing for message files */
-SYM2NUM_CMD = ./sym2num
+SYM2NUM_CMD = $(SHELL) sym2num
 
 SRCS = DtPrintinfo.C
 OBJS = libUI/BaseUI.o objects/BaseObj.o DtPrintinfo.o
diff --git a/cde/programs/dtterm/Imakefile b/cde/programs/dtterm/Imakefile
index 847bf14..0b168a2 100644
--- a/cde/programs/dtterm/Imakefile
+++ b/cde/programs/dtterm/Imakefile
@@ -87,7 +87,7 @@ clean::
 
 dtterm.ti: terminfoChecklist
        $(RM) $@
-       CPP="$(CPP) $(TIC_DEFINES)" ./terminfoCreate < terminfoChecklist > $@
+       CPP="$(CPP) $(TIC_DEFINES)" $(SHELL) terminfoCreate < terminfoChecklist 
> $@
 includes:: dtterm.ti
 depend:: dtterm.ti
 all:: dtterm.ti


//Marcin

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
cdesktopenv-devel mailing list
cdesktopenv-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/cdesktopenv-devel

Reply via email to