[Ping] Port of VTV for Cygwin and MinGW
H.J. Lu
hjl.tools@gmail.com
Thu Jan 29 20:08:00 GMT 2015
On Thu, Jan 29, 2015 at 10:30 AM, Jakub Jelinek <jakub@redhat.com> wrote:
> On Thu, Jan 29, 2015 at 10:28:17AM -0800, H.J. Lu wrote:
>> >> --- acinclude.m4 (revision 220257)
>> >> +++ acinclude.m4 (working copy)
>> >> @@ -2320,8 +2320,6 @@
>> >> AC_MSG_CHECKING([for vtable verify support])
>> >> AC_MSG_RESULT([$enable_vtable_verify])
>> >>
>> >> - AM_CONDITIONAL(VTV_CYGMIN, test $vtv_cygmin = yes)
>> >> -
>> >> if test $enable_vtable_verify = yes; then
>> >> case ${target_os} in
>> >> cygwin*|mingw32*)
>> >> @@ -2341,6 +2339,8 @@
>> >> VTV_CXXLINKFLAGS=
>> >> fi
>> >>
>> >> + AM_CONDITIONAL(VTV_CYGMIN, test $vtv_cygmin = yes)
>> >> +
>> >
>>
>> Or we can just do
>>
>> AM_CONDITIONAL(VTV_CYGMIN, test x$vtv_cygmin = xyes)
>
> Or that, but that is not what you've checked in.
> Some shells will be upset about test $vtv_cygmin = yes when
> $vtv_cygmin expands to empty string.
>
Yes, I got
checking for vtable verify support... no
/export/gnu/import/git/sources/gcc/libstdc++-v3/configure: line 17481:
test: =: unary operator expected
At least it doesn't stop bootstrap. Either fix is fine with me.
--
H.J.
More information about the Gcc-patches
mailing list