This is the mail archive of the cygwin-patches@cygwin.com mailing list for the Cygwin project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
Rob, I've put together another patch from the cygwin_daemon branch, excluding the ipcs(8) changes. In fact, this patch doesn't include any changes to <sys/ipc.h> or <sys/shm.h>, so a couple of other bug fixes and enhancements are also excluded, but since the overall patch doesn't affect the shm code, this seems not unreasonable. So, it contains the following list of changes, as you requested: > * Conditionalize the security code so that cygserver works on non-NT > platforms. > * Add definitions of the strace XXX_printf macros to allow code to use > these whether it's compiled for the DLL or for the daemon. > * Several minor C++ related changes: for example, making some methods > pure virtual, and adding virtual destructors throughout as required. > * Add --version and --help options. > * Add checking for an existing instance of the daemon to avoid having > multiple copies running. > * Some more error checking throughout. > * Refactor the client request classes for greater encapsulation and to > support variable length requests. Two notes: I'm assuming that you wanted the client request changes in the patch so that you could review them; I've also added a newer version of the "checking for another running daemon" code that improves on the previous one somewhat (once I found the lightly documented flag you need to use on the first create of a named pipe to get such checking). I hope this is all appropriate. Note that I generated this code from a plain "cvs diff" rather than from you cvsmkpatch, since I was excluding some changes on that branch and couldn't see how to achieve that and use your script. As a side-effect of this, I've had to include the added file, "woutsup.h" separately, as it's not otherwise included. (And I didn't want to risk a temporary "cvs add" in HEAD since I'm only meant to be playing on the branch; obviously if you wanted and it was okay I could do such a temporary add for this purpose but it doesn't seem too important.) Best wishes, // Conrad
Attachment:
ChangeLog.txt
Description: Text document
Attachment:
cygwin_daemon.patch.bz2
Description: Binary data
Attachment:
woutsup.h
Description: Binary data
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |