Patchwork [vtv,update] Add documentation for --enable-vtable-verify configure option...

login
register
mail settings
Submitter Paolo Carlini
Date Sept. 7, 2013, 9:06 p.m.
Message ID <522B9562.9020206@oracle.com>
Download mbox | patch
Permalink /patch/273391/
State New
Headers show

Comments

Paolo Carlini - Sept. 7, 2013, 9:06 p.m.
Hi again,

On 09/07/2013 10:41 PM, Paolo Carlini wrote:
> -- Caroline,
>
> something seems wrong with the patch, I can't build anymore. Something 
> in libvtv/testsuite:
>
> make[4]: Entering directory 
> `/home/paolo/Gcc/svn-dirs/trunk-build/x86_64-unknown-linux-gnu/libvtv/testsuite'
> Makefile:369: *** missing separator.  Stop.
basing on your committed patch, I think you simply forgot to commit the 
Makefile.* changes in libvtv/testsuite. Thus I'm going to test and 
commit as obvious the below if an x86_64-linux bootstrap succeeds again 
for me.

Thanks,
Paolo.

/////////////////////

Patch

Index: testsuite/Makefile.am
===================================================================
--- testsuite/Makefile.am	(revision 202345)
+++ testsuite/Makefile.am	(working copy)
@@ -39,12 +39,8 @@  check-script: ${testing_script} stamp-subdir
 	-@(chmod +x ${testing_script}; \
 	${testing_script} ${libvtv_srcdir} ${libvtv_builddir})
 
-if ENABLE_VTABLE_VERIFY
 check-am:
 	$(MAKE) $(AM_MAKEFLAGS) check-script
-else
-check-am:
-endif
 
 .PHONY: check-script