Re: Small patch to fix build on Windows

чт, 8 авг. 2019 г. в 06:18, Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>:
>
> Hello.
>
> At Wed, 7 Aug 2019 12:14:48 +0300, Dmitry Igrishin <dmitigr(at)gmail(dot)com> wrote in <CAAfz9KPVff92Np51DvvCDvqvxVchiuuvJCzz56qtM=N0SUnG8A(at)mail(dot)gmail(dot)com>
> > > - if ($lib =~ m/\s/)
> > > - {
> > > - $lib = '&quot;' . $lib . "&quot;";
> > > - }
> > > + # Since VC automatically quotes paths specified as the data of
> > > + # <AdditionalDependencies> in VC project file, it's mistakably
> > > + # to quote them here. Thus, it's okay if $lib contains spaces.
> > >
> > > I'm not sure, but it's not likely that someone adds it without
> > > actually stumbling on space-containing paths with the ealier
> > > version. Anyway if we shouldn't touch this unless the existing
> > > code makes actual problem.
> > So, do you think a comment is not needed here?
>
> # Sorry the last phrase above is broken.
>
> I meant "if it ain't broke don't fix it".
>
> I doubt that some older versions of VC might need it. I confirmed
> that the extra &quot; actually harms at least VC2019 and the code
> removal in the patch works.
The code removal is required also to build on VC2017.

> As for the replace comment, I'm not
> sure it is needed since I think quoting is not the task for
> AddLibrary/AddIncludeDir in the first place (and AddIncludeDir
> doesn't have the same comment).
The attached 3rd version of the patch contains no comment in AddLibrary().

>
> Now I'm trying to install VS2015 into my alomost-filled-up disk..
Thank you!