On Thu, 23 Aug 2012, Pascal Stumpf wrote: > On Thu, 23 Aug 2012 17:49:26 +0200, Marcin Cieslak wrote: > > 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. > > Have you looked at whether ksh is actually required everywhere or if > it's trivial to use just a POSIX-compliant shell? Should be easier to > test on FreeBSD.
For some easier scripts, like "uncomment" and "linksLast" I have changed the #! line to /bin/sh Scripts that have "typeset" or where I could quickly spot some ksh-specific features are getting KORNSHELL. > Also, > > execlp(KORNSHELL, "ksh", "-c", cmd, NULL); > > this will require both a "ksh" and "ksh93" binary on FreeBSD ... As far I understand te first argument is the pathname of the binary to be executed, the second one is the argv[0] delivered to the shell, but is not executed. I have tested this with "dtprintinfo -populate", which at least is able to execute CUPS lpstat commands and get a list of printers. //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
