Re: [PATCH i-g-t] tests/gem_concurrent.c: rename to gem_concurrent_all

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Apr 01, 2015 at 04:36:59PM +0100, tim.gore@xxxxxxxxx wrote:
> From: Tim Gore <tim.gore@xxxxxxxxx>
> 
> Recent patch #7763349a9a87.. renamed gem_concurrent_blit.c
> to gem_concurrent.c and then added entries to Makefile.am
> to make two identical executeables (but with different
> names) from this source file. This executeable changes its
> behaviour based on argv[0]. But, this has broken the
> Android build, which does not use autotools.
> 
> This patch instead renames the source file to match
> the name of one executable (gem_concurrent_all.c) and
> creates a second source file which simply #includes the
> first. The Makefile.am entries are also removed.
> This restores the simple test.c -> test executeable
> relationship seen in the rest of IGT and allows the
> Android build system to work without parsing Makfile.am
> or having to incorporate a special workaround for this
> test.
> 
> Signed-off-by: Tim Gore <tim.gore@xxxxxxxxx>

Neat, I am not even going to complain about /* */\n/* */\n/* */

Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx





[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux