Applied to master, thanks!
-jon

On 10/30/18 8:05 AM, Matthew R. Trower wrote:
I'm sorry to report that with this commit, installCDE.src:CleanDaemons()
is utterly broken.  It doesn't work at all on any platform, and breaks
installation on non AIX/freebsd/openbsd/netbsd/linux platforms (eg:
sun).  On those platforms it tries to kill *all* running processes (!!).
Fortunately, the pipeline has further problems, and bails with an error.

This code at import was originally very simple.  Somehow over the years
it got long and crazy.  Attached is a patch which returns it to its
simple form, this time with a POSIX pipeline that should run correctly
anywhere (at least any POSIX 2004 platform, BSD, and AIX).


-mrt





Jon Trulson <j...@radscan.com> writes:

Merged to master, thanks!

Sorry for the delay in reviewing these - this week was very busy at $JOB.

-jon

On 08/19/2018 03:02 PM, Chase via cdesktopenv-devel wrote:
1st of four incoming patches

Thank you for your time,
-Chase




------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot



_______________________________________________
cdesktopenv-devel mailing list
cdesktopenv-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/cdesktopenv-devel




_______________________________________________
cdesktopenv-devel mailing list
cdesktopenv-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/cdesktopenv-devel


--
Jon Trulson

"In the game of chess, you can never let your adversary see your pieces."

                              - Zapp Brannigan


_______________________________________________
cdesktopenv-devel mailing list
cdesktopenv-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/cdesktopenv-devel

Reply via email to