summaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2010-11-15Xlib13: fix XSetModfierMapping test 1/8.HEADmasterPeter Hutterer1-6/+19
XSetModfierMapping reports FAIL due to a different ordering of the modifiers in the return value. The server keeps the modifiers in a different format than used by the core protocol. The modifier map returned by XGetModifierMapping(3) always has the keycodes in ascending order for each modifier. A straight input_map[i] == output_map[i] comparison is not enough if the input map is not in order. Instead, all values for each modifier need to be compared to find the right modifier. Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15Fix a few "return type defaults to 'int' compiler warnings.Peter Hutterer19-7/+91
Force them to be int, return 0 where necessary. Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15mc: Make main() return an int instead of calling exit(0);Peter Hutterer1-1/+2
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15My NULL is bigger than yours, don't you dare redefine it.Peter Hutterer1-3/+0
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15more void cast removalPeter Hutterer178-1157/+1157
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15Purge a bunch of unused Gen_Good_foo()Peter Hutterer1-194/+6
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15Fix compiler warning - return missing for non-void function.Peter Hutterer1-0/+1
../../../../xts5/src/libproto/DfltVals.c: In function ‘Gen_Good_Event’: ../../../../xts5/src/libproto/DfltVals.c:395: warning: control reaches end of non-void function Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15Fix a bunch of compiler warnings for default return type.Peter Hutterer1-0/+45
../../../../xts5/src/libproto/DfltVals.c:391: warning: return type defaults to ‘int’ Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15xts5: fix compiler warning.Peter Hutterer1-2/+2
../../../../xts5/src/lib/nextvinf.c: In function ‘resetvinf’: ../../../../xts5/src/lib/nextvinf.c:162: warning: suggest parentheses around comparison in operand of ‘&’ Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15Fix one more out-of-tree build error.Peter Hutterer1-2/+2
xts5/bin/ may not exist when we're trying to put xts-config into it. Create it. Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15tcm: move envmsg up into TET_LITE ifndefPeter Hutterer1-1/+4
fixes ../../../../src/tet3/tcm/tcmfuncs.c: In function ‘tet_tcminit’: ../../../../src/tet3/tcm/tcmfuncs.c:137: warning: unused variable ‘envmsg’ Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15tpstartup needs to reset config.alt_screens for single-display setups.Peter Hutterer1-1/+5
tpstartup() is called by all library tests (I think). If ScreenCount for the current display is less than 2, reset alt_screens - we have to skip the multi-display tests then. Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15tet3: if TET_RESFILE is not in the env, guess it from the program name.Peter Hutterer1-1/+10
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15Remove now empty Utils.cPeter Hutterer2-145/+0
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15Purge unused Get_Date().Peter Hutterer2-21/+0
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15Purge wffs - use ffs(3) instead.Peter Hutterer2-24/+1
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15Purge wbcmp - unused. Use memcmp instead anywayPeter Hutterer1-10/+0
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15purge wbcopy, replace with memcpyPeter Hutterer15-73/+49
sed -e "s|wbcopy[ ]*(\(.*\),[ ]*\(.*\),[ ]*\(.*\))|memcpy(\2, \1, \3)|" Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15Purge wbzero - use memset instead.Peter Hutterer3-12/+2
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15Remove EXPECT_NOTHING pointer to int cast.Peter Hutterer1-1/+1
It's not assigned to anything anywhere, so just stop the cast. Fixes warnings like: pPolyArc.c:131: warning: cast from pointer to integer of different size Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-11-15Remove superfluous (void) casts.Peter Hutterer259-1581/+1581
sed -e "s/\([ ]\)(void) \(.*\)/\1\2/" Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-10-18xts5: Bail out with UNTESTED for BAD_LENGTH requests that would have length 0Aaron Plattner1-1/+11
The pBell-2 test sets the test type to BAD_LENGTH, which attempts to create a request with a size that is too small by one long (4 bytes). However, it tries to do this with an xBellReq, which is exactly 4 bytes in size. This results in a request with a length field of 0, which triggers the big request handling in the X server. This makes the server interpret the next request as the size part of the Bell request, and hilarity ensues. Instead, detect this case and bail out with an UNTESTED result. Signed-off-by: Aaron Plattner <aplattner@nvidia.com> Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-10-13[dist] Distribute files necessary to rebuild after distcleanAaron Plattner20-1/+26
The 'distclean' target deletes all of the .c files that were generated during the dist step. Since the .m files are not shipped in the tarball, it becomes impossible to rebuild after that. Fix this by shipping the .m files in the tarball along with the .mc helper libraries that mc needs to rebuild the .c files. Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2010-09-16[dist] Don't clean BUILT_SOURCESAaron Plattner19-19/+19
We distribute the generated .c files, so we shouldn't clean them when the user runs "make clean". They should be cleaned as part of the "distclean" target. Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2010-09-16[dist] Distribute files missing from the tarballAaron Plattner13-10/+28
Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2010-09-16[dist] Don't configure directories that aren't distributed.Aaron Plattner1-4/+0
These four directories are not included in their respective SUBDIRS, which means they aren't built and aren't included in the distribution tarball. configure.ac still tries to generate their Makefiles, though, which causes configure failures when run from the dist tarball: config.status: error: cannot find input file: `src/tet3/inetlib/Makefile.in' config.status: error: cannot find input file: `src/tet3/servlib/Makefile.in' config.status: error: cannot find input file: `xts5/src/libXtaw/Makefile.in' config.status: error: cannot find input file: `xts5/src/libXtmu/Makefile.in' Fix this by just deleting those line. The corresponding Makefiles aren't used. Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2010-09-16[dist] Build mc for distAaron Plattner1-0/+4
mc is used to compile the .m files into .c files at dist time. If dist is run from a clean build directory, mc is not built and the dist fails in xts5/Xproto when it tries to run it. Add a dependency on mc for dist-hook to make sure that mc is built before the tests. Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2010-09-16Merge branch 'master' into for-reviewAaron Plattner0-0/+0
* 'master' of git://anongit.freedesktop.org/git/xorg/test/xts: Clean xts-config, xts-run, and tetexec.cfg Delete the Xt scenarios from 'all' Include libxts5.la in Xproto and XIproto's LDADD lines Build mc using the build architecture compiler Reviewed-by: Joe Kain <jkain@nvidia.com> Reviewed-by: Adam Cheney <acheney@nvidia.com>
2010-09-15Clean xts-config, xts-run, and tetexec.cfgAaron Plattner1-1/+1
xts-run, in particular, has $bindir baked into it. If it's not cleaned, then a build with srcdir == destdir, followed by a distclean, followed by a build with srcdir != destdir will pick up the previously-generated xts-run with the wrong bindir in it. Signed-off-by: Aaron Plattner <aplattner@nvidia.com> Reviewed-by: Joe Kain <jkain@nvidia.com>
2010-09-15Delete the Xt scenarios from 'all'Aaron Plattner1-30/+0
The Xt tests currently aren't built, so including them in the 'all' scenario causes a lot of failures. Signed-off-by: Aaron Plattner <aplattner@nvidia.com> Reviewed-by: Adam Cheney <acheney@nvidia.com>
2010-09-15Include libxts5.la in Xproto and XIproto's LDADD linesAaron Plattner2-0/+2
Normally, the linker picks up libxts5.so from libxts5proto.so's RPATH. However, apparently cross-compilers don't do that, and the linker complains about not being able to find it. To work around that, specify it manually on the libtool command line. Signed-off-by: Aaron Plattner <aplattner@nvidia.com> Reviewed-by: Adam Cheney <acheney@nvidia.com>
2010-09-15Build mc using the build architecture compilerAaron Plattner2-0/+21
When cross-compiling, we need to use the build compiler instead of the host compiler because mc is run during the build step to generate .c files from .m files. Signed-off-by: Aaron Plattner <aplattner@nvidia.com> Reviewed-by: Joe Kain <jkain@nvidia.com>
2010-09-15Clean xts-config, xts-run, and tetexec.cfgAaron Plattner1-1/+1
xts-run, in particular, has $bindir baked into it. If it's not cleaned, then a build with srcdir == destdir, followed by a distclean, followed by a build with srcdir != destdir will pick up the previously-generated xts-run with the wrong bindir in it. Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2010-09-14Delete the Xt scenarios from 'all'Aaron Plattner1-30/+0
The Xt tests currently aren't built, so including them in the 'all' scenario causes a lot of failures. Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2010-09-14Include libxts5.la in Xproto and XIproto's LDADD linesAaron Plattner2-0/+2
Normally, the linker picks up libxts5.so from libxts5proto.so's RPATH. However, apparently cross-compilers don't do that, and the linker complains about not being able to find it. To work around that, specify it manually on the libtool command line. Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2010-09-14Build mc using the build architecture compilerAaron Plattner2-0/+21
When cross-compiling, we need to use the build compiler instead of the host compiler because mc is run during the build step to generate .c files from .m files. Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2010-08-17Merge branch 'master' into for-reviewAaron Plattner0-0/+0
* master: xts-config.in: ":n" is a valid display name Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-08-17xts-config.in: ":n" is a valid display nameAaron Plattner1-2/+2
When determining whether the connection is local and you can connect to the server via TCP, don't try to parse out ":n.m" and then reconstruct it as "hostname:n.m", because ":n" is a valid display name. Instead, check for the optional ".m" part and if it matches, just tack the complete $DISPLAY value (including the leading colon) onto the hostname. Fixes XOpenDisplay-10 when $DISPLAY is missing the optional screen number part. Signed-off-by: Aaron Plattner <aplattner@nvidia.com> Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-08-15xts-config.in: ":n" is a valid display nameAaron Plattner1-2/+2
When determining whether the connection is local and you can connect to the server via TCP, don't try to parse out ":n.m" and then reconstruct it as "hostname:n.m", because ":n" is a valid display name. Instead, check for the optional ".m" part and if it matches, just tack the complete $DISPLAY value (including the leading colon) onto the hostname. Fixes XOpenDisplay-10 when $DISPLAY is missing the optional screen number part. Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2010-08-15Merge branch 'master' into for-reviewAaron Plattner0-0/+0
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-08-15check.sh: Treat TET_EXIT_SKIP as successAaron Plattner1-1/+7
TET_EXIT_SKIP is considered higher priority than TET_EXIT_SUCCESS in tcc, but only TET_EXIT_FAILURE really counts as a failure return code. _SKIP just means that some of the assertions couldn't be reliably tested. Signed-off-by: Aaron Plattner <aplattner@nvidia.com> Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-08-15xts5: Construct an absolute XT_FONTPATHAaron Plattner1-0/+2
This fixes tests such as XCreateGlyphCursor that fail with UNRESOLVED when run while the test suite is not installed because the tests pass "../xts5/fonts" as a font path to the server, which fails because the server doesn't look in the same directory. Signed-off-by: Aaron Plattner <aplattner@nvidia.com> Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-08-15check.sh: Treat TET_EXIT_SKIP as successAaron Plattner1-1/+7
TET_EXIT_SKIP is considered higher priority than TET_EXIT_SUCCESS in tcc, but only TET_EXIT_FAILURE really counts as a failure return code. _SKIP just means that some of the assertions couldn't be reliably tested. Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2010-08-15xts5: Construct an absolute XT_FONTPATHAaron Plattner1-0/+2
This fixes tests such as XCreateGlyphCursor that fail with UNRESOLVED when run while the test suite is not installed because the tests pass "../xts5/fonts" as a font path to the server, which fails because the server doesn't look in the same directory. Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2010-07-29Fixes for building srcdir != builddirJon TURNEY1-2/+4
Ensure $(builddir)/bin directory exists before trying to create xts-run/xts-config scripts in it Run xts-config from $(builddir) to create tetexec.cfg Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk> Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-07-29Place libapi_s after objects which reference it in the link order, so it ↵Jon TURNEY1-1/+1
links successfully for linkers which work -as-needed Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk> Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2010-07-12Fix a typo in the README.Aaron Plattner1-1/+1
"It is not be necessary". I did notice that some of the tests, such as XCreateGlyphCursor, fail with UNRESOLVED because XT_FONTPATH is "../xts5/fonts" so the server can't find the font files. I still need to figure out why that's happening. The test runs fine if the suite is installed, or if I modify tetexec.cfg to contain the absolute path to the fonts. Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2010-06-30README: install step is not necessary.Peter Hutterer1-0/+3
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> Reviewed-by: Dan Nicholson <dbn.lists@gmail.com>
2010-06-30README: Update git URL.Peter Hutterer1-1/+1
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> Reviewed-by: Dan Nicholson <dbn.lists@gmail.com>
2010-06-30README: Note that the tests don't stand alone, can only be used for diffsPeter Hutterer1-0/+6
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> Reviewed-by: Dan Nicholson <dbn.lists@gmail.com>