Missing lookup in msvcr120 for pgwin32_putenv

Started by Michael Paquieralmost 10 years ago2 messageshackers
Jump to latest
#1Michael Paquier
michael@paquier.xyz

Hi all,

While looking at e545281 I bumped into the following thing that has
visibly been forgotten since VS2013 support has been added:
--- a/src/port/win32env.c
+++ b/src/port/win32env.c
@@ -67,6 +67,9 @@ pgwin32_putenv(const char *envval)
            "msvcr110", 0, NULL
        },                      /* Visual Studio 2012 */
        {
+           "msvcr120", 0, NULL
+       },                      /* Visual Studio 2013 */
+       {
            NULL, 0, NULL
        }
    };

Attached is a patch. This should be backpatched to 9.3 where, if I
recall correctly, support for VS2013 has been added.
Regards,
--
Michael

Attachments:

vs-2013-putenv.patchapplication/x-download; name=vs-2013-putenv.patchDownload+3-0
#2Magnus Hagander
magnus@hagander.net
In reply to: Michael Paquier (#1)
Re: Missing lookup in msvcr120 for pgwin32_putenv

On Thu, Apr 21, 2016 at 11:16 PM, Michael Paquier <michael.paquier@gmail.com

wrote:

Hi all,

While looking at e545281 I bumped into the following thing that has
visibly been forgotten since VS2013 support has been added:
--- a/src/port/win32env.c
+++ b/src/port/win32env.c
@@ -67,6 +67,9 @@ pgwin32_putenv(const char *envval)
"msvcr110", 0, NULL
},                      /* Visual Studio 2012 */
{
+           "msvcr120", 0, NULL
+       },                      /* Visual Studio 2013 */
+       {
NULL, 0, NULL
}
};

Attached is a patch. This should be backpatched to 9.3 where, if I
recall correctly, support for VS2013 has been added.

Applied and backpatched. Thanks!

--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/