But this will duplicate many third party code into beignet repo. IMO, this is not a good manner. Considering what if we will write more examples and depdends on more third party code?
One alternative method is to create a third party submodule in examples directory, and could get libva source code automatically if user want to build libva example. Then we don't need to maintain these duplicate code in beignet. What do you think? On Thu, Jan 08, 2015 at 02:36:13AM +0000, Weng, Chuanbo wrote: > Hi Zhigang, > Yes, these files are referenced from libva directly. But they are not > one part of > libva distribution, just files from libva unit test. I think it's more > convenient for users to > try and build this example if we copy these files to examples directory, or > else users should > download libva source code by themselves. > > -----Original Message----- > From: Zhigang Gong [mailto:[email protected]] > Sent: Wednesday, January 07, 2015 8:49 > To: Weng, Chuanbo > Cc: [email protected]; Zou, Nanhai; Gong, Zhigang > Subject: Re: [Beignet] version 2 of libva buffer sharing patch set > > Are the following files from libva directly: > examples/libva_buffer_sharing/sysdeps.h | 79 ++++ > examples/libva_buffer_sharing/va_display.c | 166 ++++++++ > examples/libva_buffer_sharing/va_display.h | 64 +++ > examples/libva_buffer_sharing/va_display_x11.c | 139 +++++++ > > If so, why not just make this test depends on libva, and link to libva > directly? > > On Mon, Dec 22, 2014 at 10:37:53AM +0000, Weng, Chuanbo wrote: > > Hi all, > > Due to the test image size issue in the patch, I have > > made a new version of patches in order to switch to a smaller test > > image. If the patch is directly sent to mailing-list with git send-email, > > the test image can't be encoded in the correct format. So I put the patch > > on github. > > Please use the following command to fetch the patch set: > > git clone > > https://github.com/Chuanbo-Weng/libva-buffer-sharing-patches.git > > Or visit: > > > > https://github.com/Chuanbo-Weng/libva-buffer-sharing-patches > > > > > > Thanks, > > Chuanbo Weng > > > _______________________________________________ > > Beignet mailing list > > [email protected] > > http://lists.freedesktop.org/mailman/listinfo/beignet > > _______________________________________________ > Beignet mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/beignet _______________________________________________ Beignet mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/beignet
