On Fri, 2015-02-20 at 12:03 +0000, Jose Fonseca wrote:
> On 12/12/14 19:33, Jan Vesely wrote:
> > Fixes Target links to itself cmake warnings
> >
> > Signed-off-by: Jan Vesely <[email protected]>
> > ---
> >
> > compile tested. no missing symbols or libraries reported for 
> > libpiglitutils_glesX.so
> >
> >   tests/util/CMakeLists.gles1.txt | 1 -
> >   tests/util/CMakeLists.gles2.txt | 1 -
> >   tests/util/CMakeLists.gles3.txt | 1 -
> >   3 files changed, 3 deletions(-)
> >
> > diff --git a/tests/util/CMakeLists.gles1.txt 
> > b/tests/util/CMakeLists.gles1.txt
> > index 4342cf2..cd9e90d 100644
> > --- a/tests/util/CMakeLists.gles1.txt
> > +++ b/tests/util/CMakeLists.gles1.txt
> > @@ -4,7 +4,6 @@ include_directories(
> >
> >   link_libraries(
> >     ${UTIL_GL_LIBS}
> > -   piglitutil_${piglit_target_api}
> >     )
> >
> >   piglit_add_library(piglitutil_${piglit_target_api}
> > diff --git a/tests/util/CMakeLists.gles2.txt 
> > b/tests/util/CMakeLists.gles2.txt
> > index 4342cf2..cd9e90d 100644
> > --- a/tests/util/CMakeLists.gles2.txt
> > +++ b/tests/util/CMakeLists.gles2.txt
> > @@ -4,7 +4,6 @@ include_directories(
> >
> >   link_libraries(
> >     ${UTIL_GL_LIBS}
> > -   piglitutil_${piglit_target_api}
> >     )
> >
> >   piglit_add_library(piglitutil_${piglit_target_api}
> > diff --git a/tests/util/CMakeLists.gles3.txt 
> > b/tests/util/CMakeLists.gles3.txt
> > index fec284a..a23c0c7 100644
> > --- a/tests/util/CMakeLists.gles3.txt
> > +++ b/tests/util/CMakeLists.gles3.txt
> > @@ -4,7 +4,6 @@ include_directories(
> >
> >   link_libraries(
> >     ${UTIL_GL_LIBS}
> > -   piglitutil_${piglit_target_api}
> >     )
> >
> >   piglit_add_library(piglitutil_${piglit_target_api}
> >
> 
> Reviewed-by: Jose Fonseca <[email protected]>

thanks, but this patch went in some time ago.
I pushed the other two and posted the correct warning->error one.
sorry for the hassle, I should have grouped them better.

jan


-- 
Jan Vesely <[email protected]>

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to