Re: Small patch to fix build on Windows - Mailing list pgsql-hackers

From Kyotaro Horiguchi
Subject Re: Small patch to fix build on Windows
Date
Msg-id 20190808.174311.187453731.horikyota.ntt@gmail.com
Whole thread Raw
In response to Re: Small patch to fix build on Windows  (Kyotaro Horiguchi <horikyota.ntt@gmail.com>)
List pgsql-hackers
Hello.

At Thu, 08 Aug 2019 12:15:38 +0900 (Tokyo Standard Time), Kyotaro Horiguchi <horikyota.ntt@gmail.com> wrote in
<20190808.121538.87367461.horikyota.ntt@gmail.com>
> At Wed, 7 Aug 2019 12:14:48 +0300, Dmitry Igrishin <dmitigr@gmail.com> wrote in
<CAAfz9KPVff92Np51DvvCDvqvxVchiuuvJCzz56qtM=N0SUnG8A@mail.gmail.com>
> > > -    if ($lib =~ m/\s/)
> > > -    {
> > > -        $lib = '"' . $lib . """;
> > > -    }
> > > +    # 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 doubt that some older versions of VC might need it. I confirmed
> that the extra " actually harms at least VC2019 and the code
> removal in the patch works. 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).
> 
> Now I'm trying to install VS2015 into my alomost-filled-up disk..

I confirmed that VC2015 works with the above diff. So the patch
is overall fine with me.

regards.

-- 
Kyotaro Horiguchi
NTT Open Source Software Center



pgsql-hackers by date:

Previous
From: Amit Langote
Date:
Subject: Re: Some memory not freed at the exit of RelationBuildPartitionDesc()
Next
From: Markus Winand
Date:
Subject: Re: SQL/JSON path: collation for comparisons, minor typos in docs