summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* common: fix ssl_verify windows build errorsArnon Gilboa2011-05-122-2/+8
|
* client/windows: add common\ssl_verify.c/h to projectArnon Gilboa2011-05-121-2/+10
| | | | disable WarnAsError, due to c/c++ warnings
* client/windows: inc version to 0,9,0,0Arnon Gilboa2011-05-121-4/+4
|
* client/windows: init PACKAGE_VERSIONArnon Gilboa2011-05-122-3/+5
| | | | | in windows, we set PACKAGE_VERSION to the binary version since we don't have config.h as generated by linux configure
* client: fix return code when missing hostAlon Levy2011-05-121-0/+1
|
* server/red_worker: stat_now cleanupAlon Levy2011-05-091-1/+2
| | | | static, remove inline, add (void), and empty line after function variables.
* server/red_worker: fix typo (lats_send_time)Alon Levy2011-05-091-4/+4
|
* server/tests: show port to connect toAlon Levy2011-05-091-1/+3
|
* server: bump SPICE_SERVER_VERSION to 0.9.0Marc-André Lureau2011-05-081-1/+1
| | | | SASL functions will be released with 0.9.0
* server: Unset executable bit of red_tunnel_worker.hZeeshan Ali (Khattak)2011-05-051-0/+0
|
* build: fix gettimeofday warningMarc-André Lureau2011-05-031-0/+1
| | | | | | CC test_playback.o test_playback.c: In function ‘playback_timer_cb’: test_playback.c:56:5: warning: implicit declaration of function ‘gettimeofday’
* client: make use of ssl_verify.cMarc-André Lureau2011-05-034-385/+36
| | | | | | | Fixed since v1: - don't include C code, rather use the common lib - add missing spice_openssl_verify_free() call - keep the extra-parsing of subject for error reporting
* common: add ssl_verify.c common codeMarc-André Lureau2011-05-033-0/+543
| | | | | | | | | | | | | | | | | | Code adapter from RedPeer::ssl_verify_callback() and used by spice-gtk. Since v1: - fixed Makefile.am - added config.h include - autoconf alloca added in patch series - moved int escape inside for loop - added a failed case when missing assignment - replaced strlen () by -1 - skip spaces after comma - c++ guards I didn't use bool, because openSSL uses int, and it is more future proof for error reporting.
* common: add ring_get_length() for debugging purposesMarc-André Lureau2011-05-031-0/+13
| | | | | Please notice it has a "static" modifier, like the rest of the inlined functions in ring.h, so it won't warn if it isn't used.
* common: mem.h add alloca definitionMarc-André Lureau2011-05-032-0/+31
| | | | | We don't support the autoconf ALLOCA/C_ALLOC fallback. If one day, someone cares for a weird platform, he can fix it.
* common: spice_memdup could accept NULLMarc-André Lureau2011-05-031-0/+4
| | | | (this patch is not to solve a crash fix, but to align with glib API)
* common: add windows.h where required, make gdi_handlers staticMarc-André Lureau2011-05-034-2/+8
| | | | | This patch has not been verified with VS/brew. It should be safe hopefully. Compilation is fine with mingw32/spice-gtk.
* build: require c99Marc-André Lureau2011-05-032-1/+4
| | | | Because we use c99: stdbool.h, inttypes.h, bool, variadic macros, // comments, ...
* move get_time_stamp to main_channel.cChristophe Fergeau2011-05-032-7/+7
| | | | | There is only one user of get_time_stamp from spice_common.h so it's not really useful to keep it there.
* use standard do { } while (0) for spice_common.h macrosChristophe Fergeau2011-05-031-11/+11
|
* move WARN and WARN_ONCE to spice_common.hChristophe Fergeau2011-05-034-15/+7
|
* common: use PANIC from spice_common.hChristophe Fergeau2011-05-033-20/+0
|
* common,server: use ASSERT from spice_common.hChristophe Fergeau2011-05-039-36/+7
| | | | | | | spice_common.h provides an ASSERT macro, no need to duplicate it in many places. For now client/debug.h keeps its own copy since debug.h and spice_common.h have clashes on other macros which are trickier to unify.
* server: use gcc builtin for atomic get/set bitChristophe Fergeau2011-05-031-2/+4
|
* client: remove unused mb() macroChristophe Fergeau2011-05-031-6/+0
|
* common: don't duplicate find_msb implementationChristophe Fergeau2011-05-034-67/+95
|
* remove void * arithmeticChristophe Fergeau2011-05-032-2/+2
| | | | | | | With -Wpointer-arith, gcc complains about void pointer arithmetic. This is not a big deal with gcc, but could be with other compilers, so it's better to cast to char */uint8_t * before doing the arithmetic on such pointers.
* add missing staticChristophe Fergeau2011-05-038-28/+29
|
* configure.ac: remove setting default C(XX)FLAGSChristophe Fergeau2011-05-031-12/+0
| | | | | automake/autoconf already set them for us to -g -O2 if there are no flags defined.
* win32: remove obsolete preprocessor #definesChristophe Fergeau2011-05-031-4/+4
| | | | | | SW_CANVAS_NO_CHUNKS isn't used anywhere but in this file. SW_CANVAS_CACHE is now defined directly in the files where it's needed so we no longer need it in the .vcproj file.
* add comment to beginning of autogenerated filesChristophe Fergeau2011-05-031-0/+2
|
* s/USE_OGL/USE_OPENGLChristophe Fergeau2011-05-0320-110/+110
| | | | This is more explicit about what it does, and not much longer
* use AC_DEFINE instead of hardcoding preprocessor flagsChristophe Fergeau2011-05-031-3/+4
| | | | | | | | | | Now that all source files include config.h, we can use AC_DEFINE to enable/disable optional features (opengl, cegui). This is cleaner than directly appending -Dxxx directives to the preprocessor flags. This also guarantees that everything will be appropriately rebuilt when using the same source tree to build first with opengl support and then without (the config.h timestamp will change, triggering a rebuild of the files including it)
* add config.h to autogenerated files tooChristophe Fergeau2011-05-031-0/+4
| | | | | Modify the python (de)marshaller generator to add #include <config.h> at the beginning of the C files it generates
* add #include <config.h> to all source filesChristophe Fergeau2011-05-03109-14/+330
| | | | | | | | When using config.h, it must be the very first include in all source files since it contains #define that may change the compilation process (eg libc structure layout changes when it's used to enable large file support on 32 bit x86 archs). This commit adds it at the beginning of all .c and .cpp files
* autotools: correctly build canvas-related codeChristophe Fergeau2011-05-0330-34/+218
| | | | | | | | | | | | | | | | | | | | | spice client and spice server shares code from common/{gdi,gl,sw}_canvas.[ch]. However, while most of the code is shared, the server code wants a canvas compiled with SW_CANVAS_IMAGE_CACHE defined while the client code wants a canvas compiled with SW_CANVAS_CACHE. The initial autotools refactoring didn't take that into account, this is now fixed by this commit. After this commit, the canvas files from common/ are no longer compiled as part of the libspice-common.la convenience library. Instead, there are "proxy" canvas source files in client/ and server/ which #include the appropriate C files after defining the relevant #define for the binary that is being built. To prevent misuse of the canvas c files and headers in common/, SPICE_CANVAS_INTERNAL must be set when including the canvas headers from common/ or when building the c files from common/ otherwise the build will error out.
* configure.ac: remove unused AC_SUBSTChristophe Fergeau2011-05-031-4/+0
|
* autotools: refactor the whole build machineryChristophe Fergeau2011-05-038-706/+350
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | spice Makefile.am setup is a bit confusing, with source file names being listed several times in different Makefile.am (generally, once in EXTRA_DIST and another time in another Makefile.am in _SOURCES). The client binaries are built by client/x11/Makefile.am, which means recursing into client, then into x11 to finally build spicec. This Makefile.am is also referencing files from common/ and client/, which is a bit unusual with autotools. This patch attempts to simplify the build process to get something more usual from an autotools point of view. The source from common/ are compiled into a libtool convenience library, which the server and the client links against which avoids referencing source files from common/ when building the server and the client. The client is built in client/Makefile.am and directly builds files from x11/ windows/ and gui/ if needed (without recursing in these subdirectories). This makes the build simpler to understand, and also makes it possible to list source files once, which avoids potential make distcheck breakage when adding new files. There is a regression in this patch with respect to sw_canvas/gl_canvas/gdi_canvas. They should be built with different preprocessor #defines resulting in different behaviour of the canvas for the client and the server. However, this is not currently the case, both the client and the server will use the same code for now (which probably means one of them is broken). This will be fixed in a subsequent commit. make distcheck passes, but compilation on windows using the autotools build system hasn't been tested, which means it's likely to be broken. It shouldn't be too hard ot fix it though, just let me know of any issues with this.
* add missing "LGPLv2.1 or later" header to source filesChristophe Fergeau2011-05-0214-1/+235
|
* common: don't try to redefine PANIC if it already existsChristophe Fergeau2011-05-021-0/+2
| | | | | | | | | | canvas_base.c tries to define PANIC, but it might already be set if eg client/debug.h has been included before. All the other macros in this file are guarded by #ifndef, this commit adds the missing #ifndef to PANIC. Note that this is just a bandaid, ideally common/ would contain a logging frameword, and these macros would only be defined once instead of being defined in several places.
* server: s/desable/disableChristophe Fergeau2011-05-023-4/+4
| | | | | This fixes a typo in some function names, there should be no functional change.
* use foo(void) instead of foo() in prototypesChristophe Fergeau2011-05-0220-40/+40
| | | | | In C, the latter isn't a prototype for a function with no arg, but declares a function with an undefined number of args.
* server: remove extra arg in reds_on_main_agent_start callChristophe Fergeau2011-05-021-1/+1
|
* client: remove c++ wrappersChristophe Fergeau2011-05-0217-338/+36
| | | | | | | | client/ contains several .cpp file which only #include a .c file of the same name. This is unusual and seems to only be done to get C++ name mangling on the symbols defined in the C file. Now that all headers files in common/ use extern "C", these wrappers are no longer useful.
* common: add missing header guards to gl_canvas.hChristophe Fergeau2011-05-021-0/+5
|
* common: add extern "C" guards to headersChristophe Fergeau2011-05-0223-2/+182
| | | | | | Since some spice C++ code is using code from common/, the C functions need to be marked as such for the C++ compiler, otherwise we'll get linkage issues.
* client: make building client optionalChristophe Fergeau2011-05-022-1/+13
|
* configure.ac: replace tab with spacesChristophe Fergeau2011-05-021-1/+1
| | | | | This makes the "C++ Compiler: ...." status output nicely aligned with the other messages.
* configure.ac: remove unused dynamic linkage flagChristophe Fergeau2011-05-021-3/+1
| | | | | | | | configure.ac defines a SPICEC_STATIC_LINKAGE_BDYNAMIC variable when --enable-static-linkage is not set, but it's never used. SPICEC_STATIC_LINKAGE_BSTATIC is used in client/, but since we are using libtool, it might be more appropriate to use 'make LDFLAGS="-all-static"' to achieve static link.
* configure.ac: fix use of AC_ARG_ENABLEChristophe Fergeau2011-05-021-33/+35
| | | | | | | | | | | Most uses of AC_ARG_ENABLE were buggy: - when passing --disable-xxx, configure.ac would behave as if it was passed --enable-xxx - passing --enable-xxx=foo would "leak" into the summary, ie the summary (at the end of configure) would say "xxx: foo" instead of "xxx: yes" This patch fixes these 2 issues.