summaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2011-12-22Remove old files from the TET3 import.HEADmasterCyril Brulebois14-2197/+0
CHANGES, README.old and RELEASE come from the previous millenium and probably don't help much. Autotooling the package totally obsoleted makefile.old files. Clean this up. Signed-off-by: Cyril Brulebois <kibi@debian.org>
2011-12-22xts-config: Fix “Use of qw(...) as parentheses” warning.Cyril Brulebois1-2/+2
In Perl before 5.14, the following syntax was happily accepted: foreach my $foo qw(bar baz) but that's been deprecated and that now generates a warning: Use of qw(...) as parentheses is deprecated at ./xts5/bin/xts-config line 118. The correct form has always been: foreach my $foo (qw(bar baz)) and that's compatible with older Perl releases, so let's use that. Reference: http://perldoc.perl.org/perl5140delta.html Signed-off-by: Cyril Brulebois <kibi@debian.org>
2011-12-22README: Improve two sentences.Cyril Brulebois1-2/+2
Signed-off-by: Cyril Brulebois <kibi@debian.org>
2011-12-22README: Document libXau as a dependency.Cyril Brulebois1-1/+1
Signed-off-by: Cyril Brulebois <kibi@debian.org>
2011-12-22configure.ac: Require libXau.Cyril Brulebois1-1/+1
It is used in several places, without an explicit dependency. Signed-off-by: Cyril Brulebois <kibi@debian.org>
2011-12-22configure.ac: Quote argument to m4_pattern_forbid.Cyril Brulebois1-1/+1
Without this, configure spits out: ./configure: line 12801: ac_fn_c_check_member: command not found ./configure: line 12815: ac_fn_c_check_type: command not found Also anchor the pattern to make it stricter. Signed-off-by: Cyril Brulebois <kibi@debian.org>
2011-09-22xts5: XSubImage-1: Use left shift by '31' instead of 'depth' for depth >= 32Nalluri Harikrishna1-2/+11
15|692 3.3-lite 1|TCM Start 400|692 1 1 20:00:19|IC Start 200|692 1 20:00:19|TP Start 520|692 1 00023801 1 1|VSW5TESTSUITE PURPOSE 1 520|692 1 00023801 1 2|Assertion XSubImage-1.(A) 520|692 1 00023801 1 3|When the rectangle specified by the x, y, subimage_width 520|692 1 00023801 1 4|and subimage_height arguments is contained in the image 520|692 1 00023801 1 5|ximage, then a call to XSubImage creates a copy of the 520|692 1 00023801 1 6|subimage and returns a pointer to the newly created 520|692 1 00023801 1 7|subimage. 520|692 1 00023801 1 8|METH: For all supported drawables types: 520|692 1 00023801 1 9|METH: Create a drawable. 520|692 1 00023801 1 10|METH: Initialise the drawable. 520|692 1 00023801 1 11|METH: For XYPixmap and ZPixmap: 520|692 1 00023801 1 12|METH: Create an image using XGetImage. 520|692 1 00023801 1 13|METH: Obtain a subimage using XSubImage. 520|692 1 00023801 1 14|METH: Verify that the pixels in the subimage are correctly set using XGetPixel. 520|692 1 00023801 1 15|unexpected signal 8 (SIGFPE) received 220|692 1 2 20:00:19|UNRESOLVED 410|692 1 1 20:00:19|IC End This test fills pixel color values by using the function mpattern() and checks the returned image by using the function checksubimage(). Both these functions use color values from the range 0 – 2^depth. The value 2^depth is stored in an int (32 bit) variable by using ‘mod = 1 << depth’. If ‘depth’ is 32 then ‘mod’ ends up being 0. This variable ‘mod’ is later used in a modulo operation resulting in a SIGFPE. This fix corrects the test by using ‘mod = 1 << 31’ instead of 'mod = 1 << depth' for depth >= 32. Reported-by: Nalluri Harikrishna <nharikrishna@nvidia.com> Signed-off-by: Nalluri Harikrishna <nharikrishna@nvidia.com> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-08-29xts5: XCreateColormap-11: Bogus visuals return BadMatch, not BadValueAaron Plattner1-4/+4
According to the X11 protocol specification, "The visual type must be one supported by the screen (or a Match error results)." It does not say anything about invalid visual IDs causing Value errors. Reported-by: Nalluri Harikrishna <nharikrishna@nvidia.com> Reported-by: Rahul Singhal <rasinghal@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-08-28xts5: XWriteBitmapFile-3: Report UNTESTED instead of FAIL when run as rootAaron Plattner1-0/+7
This test creates a directory, makes it non-writable, then calls XWriteBitmapFile to attempt to write to it, expecting it to fail with BitmapOpenFailed. This works fine when run as a normal user, but when run as root, writing the file succeeds anyway. Check the euid and bail out with UNTESTED if it's root. Reported-by: Abhinav Shukla <abhinavs@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com> Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
2011-08-16xts5: Define X11_t and TRANS_CLIENT to get Xtrans function prototypesAaron Plattner1-0/+3
ConnDis.c uses various Xtrans functions. Even though the Xtrans code itself is compiled in x11_trans.c, we need to define these preprocessor macros in ConnDis.c also to get prototypes for various functions. Otherwise, this causes warnings and a sign-extension bug on 64-bit platforms that can cause crashes: ConnDis.c: In function ‘_X11TransConnectDisplay’: ConnDis.c:351:2: warning: implicit declaration of function ‘_X11TransOpenCOTSClient’ ConnDis.c:351:19: warning: assignment makes pointer from integer without a cast ConnDis.c:355:2: warning: implicit declaration of function ‘_X11TransConnect’ ConnDis.c:357:6: warning: implicit declaration of function ‘_X11TransClose’ ConnDis.c:366:2: warning: implicit declaration of function ‘_X11TransGetPeerAddr’ ConnDis.c:374:2: warning: implicit declaration of function ‘_X11TransConvertAddress’ ConnDis.c:399:5: warning: implicit declaration of function ‘_X11TransSetOption’ ConnDis.c: In function ‘_XConnectDisplay’: ConnDis.c:521:8: warning: implicit declaration of function ‘_X11TransGetConnectionNumber’ ConnDis.c:522:8: warning: implicit declaration of function ‘_X11TransFreeConnInfo’ ConnDis.c: In function ‘_XDisconnectDisplay’: ConnDis.c:545:5: warning: implicit declaration of function ‘_X11TransDisconnect’ ConnDis.c: In function ‘_XSendClientPrefix’: ConnDis.c:588:5: warning: implicit declaration of function ‘_X11TransWritev’ pAllocColorCell[27592]: segfault at 58fe7f0 ip 000000000016670e sp 00007fff0da0c110 error 4 in libxts5proto.so[110000+6a000] Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-08-15tet: Ignore SIGWINCH (terminal window size change)Aaron Plattner1-0/+8
This signal is asserted when the controlling terminal changes size, for example if you're running xts from an xterm and you resize the window. This causes tests to fail: 70||"VSW5TESTSUITE CASE FillPoly 3" 10|34 /Xproto/pFillPoly 15:28:48|TC Start, scenario ref 71-0 15|34 3.3-lite 3|TCM Start 510|34|system 0: Abandoning testset: caught unexpected signal 28 (unknown signal) 80|34 1 15:28:49|TC End, scenario ref 71-0 These signals can't affect the test environment, so just ignore them all the time. Signed-off-by: Aaron Plattner <aplattner@nvidia.com> Acked-by: Peter Hutterer <peter.hutterer@who-t.net> Reviewed-by: Dan Nicholson <dbn.lists@gmail.com>
2011-06-24xts5: Switch from K&R to ANSI prototypes.Cyril Brulebois1-5/+3
Conversion performed through: cproto -a -I include -I xts5/include/ xts5/src/lib/checkevent.c (after which trailing spaces were removed.) Suggested-by: Walter Harms <wharms@bfs.de> Signed-off-by: Cyril Brulebois <kibi@debian.org> [aplattner@nvidia.com: fixed whitespace errors] Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-06-10xts5: Try to improve readability.Cyril Brulebois1-1/+2
Suggested-by: Walter Harms <wharms@bfs.de> Signed-off-by: Cyril Brulebois <kibi@debian.org> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-05-26xts5: Fix implicit declaration.Cyril Brulebois1-0/+4
Use a forward declaration to avoid moving the function body across the file. The following warning goes away accordingly: | CC checkevent.lo | checkevent.c:647:10: warning: implicit declaration of function 'check_ext_event' is invalid in C99 [-Wimplicit-function-declaration] | fail = check_ext_event(good,ev); | ^ Signed-off-by: Cyril Brulebois <kibi@debian.org> [aplattner@nvidia.com: Removed unnecessary comment with a typo in it; it's clear what forward declarations are for.] Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Tested-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-05-26xts5: Add parentheses around assignment in an if.Cyril Brulebois1-1/+1
Tell compilers we really meant to assign and then check. The following warning goes away accordingly: | CC nextvclass.lo | nextvclass.c:224:14: warning: using the result of an assignment as a condition without parentheses [-Wparentheses] | if (comma = strchr(++s, COMMA)) | ~~~~~~^~~~~~~~~~~~~~~~~~~~ | nextvclass.c:224:14: note: use '==' to turn this assignment into an equality comparison | if (comma = strchr(++s, COMMA)) | ^ | == | nextvclass.c:224:14: note: place parentheses around the assignment to silence this warning | if (comma = strchr(++s, COMMA)) | ^ | ( ) Signed-off-by: Cyril Brulebois <kibi@debian.org> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Tested-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-05-26xts5: Return a dummy value.Cyril Brulebois1-0/+1
Since it's too late already when this function is called, let's return something to match the prototype. This warning goes away accordingly: | CC err.lo | err.c:180:1: warning: control reaches end of non-void function [-Wreturn-type] | } | ^ Signed-off-by: Cyril Brulebois <kibi@debian.org> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Tested-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-05-26xts5: Fix missing unistd.h includes.Cyril Brulebois5-0/+5
It's needed for sleep(): | CC gettime.lo | gettime.c:159:23: warning: implicit declaration of function 'sleep' is invalid in C99 [-Wimplicit-function-declaration] | for (i=0; i<10; i++, sleep(2)) { | ^ It is needed for alarm(): | CC settimeout.lo | settimeout.c:155:2: warning: implicit declaration of function 'alarm' is invalid in C99 [-Wimplicit-function-declaration] | alarm((unsigned long)to); | ^ It is needed for unlink(): | CC checkarea.lo | checkarea.c:296:2: warning: implicit declaration of function 'unlink' is invalid in C99 [-Wimplicit-function-declaration] | unlink(name); | ^ | CC verimage.lo | verimage.c:307:3: warning: implicit declaration of function 'unlink' is invalid in C99 [-Wimplicit-function-declaration] | unlink(errfile); | ^ | CC savimage.lo | savimage.c:187:6: waning: implicit declaration of function 'unlink' is invalid in C99 [-Wimplicit-function-declaration] | unlink(name); | ^ Signed-off-by: Cyril Brulebois <kibi@debian.org> [aplattner@nvidia.com: squashed several similar commits] Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Tested-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-05-26xts5: Fix missing string.h include.Cyril Brulebois1-0/+1
It is needed for strcmp(): | CC checkfont.lo | checkfont.c:492:16: warning: implicit declaration of function 'strcmp' is invalid in C99 [-Wimplicit-function-declaration] | if (str && strcmp(teststr, str) != 0) { | ^ Signed-off-by: Cyril Brulebois <kibi@debian.org> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Tested-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-05-26xts5: Fix missing type specifiers.Cyril Brulebois7-29/+29
Use void when nothing is actually returned, use int otherwise. Also update the appropriate header for the void ones. Even though a comment claims this header is generated automatically from the source files, that doesn't seem to be the case. The following warning go away accordingly: | CC checkevent.lo | checkevent.c:664:1: warning: type specifier missing, defaults to 'int' [-Wimplicit-int] | check_ext_event(good,ev) | ^~~~~~~~~~~~~~~ | CC devcntl.lo | devcntl.c:401:1: warning: type specifier missing, defaults to 'int' [-Wimplicit-int] | devicerelkeys(dev) | ^~~~~~~~~~~~~ | devcntl.c:537:1: warning: type specifier missing, defaults to 'int' [-Wimplicit-int] | modpress(disp, mask) | ^~~~~~~~ | devcntl.c:545:1: warning: type specifier missing, defaults to 'int' [-Wimplicit-int] | devmodpress(disp, dev, mask) | ^~~~~~~~~~~ | devcntl.c:559:1: warning: type specifier missing, defaults to 'int' [-Wimplicit-int] | modrel(disp, mask) | ^~~~~~ | devcntl.c:567:1: warning: type specifier missing, defaults to 'int' [-Wimplicit-int] | devmodrel(disp, dev, mask) | ^~~~~~~~~ | devcntl.c:644:1: warning: type specifier missing, defaults to 'int' [-Wimplicit-int] | ismodkey(mask, kc) | ^~~~~~~~ | devcntl.c:669:1: warning: type specifier missing, defaults to 'int' [-Wimplicit-int] | restoredevstate() | ^~~~~~~~~~~~~~~ | devcntl.c:681:1: warning: type specifier missing, defaults to 'int' [-Wimplicit-int] | noext(needbutton) | ^~~~~ | devcntl.c:703:1: warning: type specifier missing, defaults to 'int' [-Wimplicit-int] | nbuttons() | ^~~~~~~~ | devcntl.c:718:1: warning: type specifier missing, defaults to 'int' [-Wimplicit-int] | getkeycode(display) | ^~~~~~~~~~ | devcntl.c:741:1: warning: type specifier missing, defaults to 'int' [-Wimplicit-int] | getdevkeycode(display,dev) | ^~~~~~~~~~~~~ | CC pfcount.lo | pfcount.c:118:1: waning: type specifier missing, defaults to 'int' [-Wimplicit-int] | pfcount(pass, fail) | ^~~~~~~ | pfcount.c:126:1: waning: type specifier missing, defaults to 'int' [-Wimplicit-int] | rptcounts() | ^~~~~~~~~ | CC regid.lo | regid.c:312:1: waning: type specifier missing, defaults to 'int' [-Wimplicit-int] | regenable() | ^~~~~~~~~ | regid.c:316:1: waning: type specifier missing, defaults to 'int' [-Wimplicit-int] | regdisable() | ^~~~~~~~~~ | CC checkfont.lo | checkfont.c:155:8: warning: type specifier missing, defaults to 'int' [-Wimplicit-int] | static check1prop(XFontStruct *fsp, XFontProp *fp, char *str); | ~~~~~~ ^ | checkfont.c:451:1: warning: type specifier missing, defaults to 'int' [-Wimplicit-int] | check1prop(fsp, fp, str) | ^ | CC cursor.lo | cursor.c:123:1: warning: type specifier missing, defaults to 'int' [-Wimplicit-int] | curofwin(display, cursor, window) | ^~~~~~~~ | cursor.c:133:1: warning: type specifier missing, defaults to 'int' [-Wimplicit-int] | defcur(display, window) | ^~~~~~ | cursor.c:165:1: warning: type specifier missing, defaults to 'int' [-Wimplicit-int] | spriteiswin(display, window) | ^~~~~~~~~~~ Signed-off-by: Cyril Brulebois <kibi@debian.org> [aplattner@nvidia.com: Squashed a few similar commits] Tested-by: Aaron Plattner <aplattner@nvidia.com> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-05-26configure: Quote argument to m4_pattern_forbid.Cyril Brulebois1-1/+1
Without this, configure spits out: | ./configure: line 12748: ac_fn_c_check_member: command not found | ./configure: line 12762: ac_fn_c_check_type: command not found Signed-off-by: Cyril Brulebois <kibi@debian.org> Tested-by: Aaron Plattner <aplattner@nvidia.com> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-05-24Bump version to 0.99.1Aaron Plattner1-1/+1
Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-05-24tet3: 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> [aplattner@nvidia.com: fixed whitespace to match surrounding code] Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-05-24tet: tet_getvar needs to take a const char* argumentAaron Plattner4-10/+9
config.c: In function ‘initconfig’: config.c:394: error: passing argument 1 of ‘_initconfig’ from incompatible pointer type config.c:328: note: expected ‘char * (*)(const char *)’ but argument is of type ‘char * (*)(char *)’ Signed-off-by: Aaron Plattner <aplattner@nvidia.com> Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com> Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
2011-05-24xts5: BadDevice takes an int*, not an XID*Aaron Plattner1-1/+1
startup.c: In function ‘init_xinput’: startup.c:505: error: passing argument 2 of ‘_xibaddevice’ from incompatible pointer type /usr/include/X11/extensions/XInput.h:162: note: expected ‘int *’ but argument is of type ‘XID *’ Signed-off-by: Aaron Plattner <aplattner@nvidia.com> Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com> Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
2011-05-24Xlib13: fix XSetModifierMapping test 1/8.Peter Hutterer1-6/+24
XSetModifierMapping 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> [aplattner@nvidia.com: clarification comment added] Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-04-12mc: Make main() return an int instead of calling exit(0);Peter Hutterer1-1/+2
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-04-12My 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> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-04-12more void cast removalPeter Hutterer178-1157/+1157
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> Acked-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-04-12Purge a bunch of unused Gen_Good_foo()Peter Hutterer1-194/+6
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-04-12Fix 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> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-04-12Fix 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> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-04-12xts5: 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> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-04-12Fix 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> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-04-12tcm: 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> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-04-12Purge unused Get_Date().Peter Hutterer2-22/+1
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-04-12Purge wffs - use ffs(3) instead.Peter Hutterer2-25/+1
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-04-12Purge wbcmp - unused. Use memcmp instead anywayPeter Hutterer1-10/+0
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-04-12Purge wbzero - use memset instead.Peter Hutterer3-12/+2
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-04-12Remove 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> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-04-12Remove superfluous (void) casts.Peter Hutterer259-1581/+1581
sed -e "s/\([ ]\)(void) \(.*\)/\1\2/" Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> Reviewed-by: Aaron Plattner <aplattner@nvidia.com> Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
2011-04-12Decouple the test suite from "make check"Aaron Plattner2-6/+8
The "check" target is run during "distcheck" after the test suite is built, so having it run the test suite itself means that the suite has to pass before a tarball can be distributed. Also, an unsuspecting developer attempting to create a tarball might accidentally run the test suite against his host X server, which is bad because some of the tests can have adverse effects. Signed-off-by: Aaron Plattner <aplattner@nvidia.com> Acked-by: Peter Hutterer <peter.hutterer@who-t.net> Reviewed-by: Dan Nicholson <dbn.lists@gmail.com>
2011-02-23Unbreak the .m.c ruleAdam Jackson1-0/+4
Some builds of gnu make apparently know about .m.o: rules, and will attempt to use them directly rather than going through .m.c: on the way to .c.o:. Which is great, if your .m files happen to be Objective C source; but here, they're not. Acked-by: Peter Hutterer <peter.hutterer@who-t.net> Tested-by: Paulo Zanoni <pzanoni@mandriva.com> Reviewed-by: Dan Nicholson <dbn.lists@gmail.com> Signed-off-by: Adam Jackson <ajax@redhat.com>
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>
generated by cgit v1.2.3 (git 2.39.1) at 2024-05-16 00:57:44 +0000