Re: [HACKERS] Install shared libs in lib/ and bin/ with MSVC (Was: install libpq.dll in bin directory on Windows / Cygwin)

2015-03-18 Thread Alvaro Herrera 
Michael Paquier wrote:
 On Wed, Mar 18, 2015 at 3:13 AM, Alvaro Herrera
 alvhe...@2ndquadrant.com wrote:

  I mean, can't we just do the push unconditionally here?
 
 Why should we install unnecessary stuff? This complicates the
 installation contents, the point being to have only shared libraries's
 dll installed in bin/, and make things consistent with what MinGW
 does.

I was expecting that if no .dll or .lib file is expected to be
installed, then the file would not be present in the first place.
I just wanted to avoid what seems fragile coding.

  Surely if there are no lib/dll files in the subdirectory, nothing will
  happen, right?  (I haven't actually tried.)
 
 No, it fails. And we should actually have a bin/lib that has been
 correctly generated. Do you think this is a problem? Normally we
 *should* fail IMO, meaning that the build process has broken what it
 should have done.

Makes sense.

I have pushed your patch; we'll see what the buildfarm thinks of it.
(Sadly, the number of MSVC members is rather small and they don't run
often.)

-- 
Álvaro Herrerahttp://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training  Services


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] Install shared libs in lib/ and bin/ with MSVC (Was: install libpq.dll in bin directory on Windows / Cygwin)

2015-03-18 Thread Michael Paquier 
On Thu, Mar 19, 2015 at 3:30 AM, Alvaro Herrera wrote:
 Makes sense.

 I have pushed your patch; we'll see what the buildfarm thinks of it.

Thanks.

 (Sadly, the number of MSVC members is rather small and they don't run
 often.)

Once Windows 10 is out, it could be installed on a Raspberry PI 2.
This would make a nice buildfarm member with MSVC 2015.
-- 
Michael


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] Install shared libs in lib/ and bin/ with MSVC (Was: install libpq.dll in bin directory on Windows / Cygwin)

2015-03-17 Thread Alvaro Herrera 
Michael Paquier wrote:

 So I have recoded the patch to use an hash of arrays (makes the code
 more readable IMO) to be able to track more easily what to install
 where, and process now does the following for shared libraries:
 - In lib/, install all .dll and .lib
 - In bin/, install all .dll

I wonder why do we need this part:

 @@ -247,22 +250,47 @@ sub CopySolutionOutput
  
   my $proj = read_file($pf.$vcproj)
 || croak Could not open $pf.$vcproj\n;
 +
 + # Check if this project uses a shared library by looking if
 + # SO_MAJOR_VERSION is defined in its Makefile, whose path
 + # can be found using the resource file of this project.
 + if (($vcproj eq 'vcxproj' 
 +  $proj =~ qr{ResourceCompile\s*Include=([^]+)}) ||
 + ($vcproj eq 'vcproj' 
 +  $proj =~ qr{File\s*RelativePath=([^\]+)\.rc}))
 + {
 + my $projpath = dirname($1);
 + my $mfname = -e $projpath/GNUmakefile ?
 + $projpath/GNUmakefile : $projpath/Makefile;
 + my $mf = read_file($mfname) ||
 + croak Could not open $mfname\n;
 +
 + if ($mf =~ /^SO_MAJOR_VERSION\s*=\s*(.*)$/mg)
 + {
 + $is_sharedlib = 1;
 + }
 + }

I mean, can't we just do the push unconditionally here?

   elsif ($1 == 2)
   {
 - $dir = lib;
 - $ext = dll;
 + push( @{ $install_list { 'lib' } }, dll);
 + if ($is_sharedlib)
 + {
 + push( @{ $install_list { 'bin' } }, 
 dll);
 + push( @{ $install_list { 'lib' } }, 
 lib);
 + }
   }

Surely if there are no lib/dll files in the subdirectory, nothing will
happen, right?  (I haven't actually tried.)

-- 
Álvaro Herrerahttp://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training  Services


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] Install shared libs in lib/ and bin/ with MSVC (Was: install libpq.dll in bin directory on Windows / Cygwin)

2015-03-17 Thread Michael Paquier 
On Wed, Mar 18, 2015 at 3:13 AM, Alvaro Herrera
alvhe...@2ndquadrant.com wrote:
 Michael Paquier wrote:

 So I have recoded the patch to use an hash of arrays (makes the code
 more readable IMO) to be able to track more easily what to install
 where, and process now does the following for shared libraries:
 - In lib/, install all .dll and .lib
 - In bin/, install all .dll

 I wonder why do we need this part:

 @@ -247,22 +250,47 @@ sub CopySolutionOutput

   my $proj = read_file($pf.$vcproj)
 || croak Could not open $pf.$vcproj\n;
 +
 + # Check if this project uses a shared library by looking if
 + # SO_MAJOR_VERSION is defined in its Makefile, whose path
 + # can be found using the resource file of this project.
 + if (($vcproj eq 'vcxproj' 
 +  $proj =~ qr{ResourceCompile\s*Include=([^]+)}) ||
 + ($vcproj eq 'vcproj' 
 +  $proj =~ qr{File\s*RelativePath=([^\]+)\.rc}))
 + {
 + my $projpath = dirname($1);
 + my $mfname = -e $projpath/GNUmakefile ?
 + $projpath/GNUmakefile : $projpath/Makefile;
 + my $mf = read_file($mfname) ||
 + croak Could not open $mfname\n;
 +
 + if ($mf =~ /^SO_MAJOR_VERSION\s*=\s*(.*)$/mg)
 + {
 + $is_sharedlib = 1;
 + }
 + }

 I mean, can't we just do the push unconditionally here?

Why should we install unnecessary stuff? This complicates the
installation contents, the point being to have only shared libraries's
dll installed in bin/, and make things consistent with what MinGW
does.

   elsif ($1 == 2)
   {
 - $dir = lib;
 - $ext = dll;
 + push( @{ $install_list { 'lib' } }, dll);
 + if ($is_sharedlib)
 + {
 + push( @{ $install_list { 'bin' } }, 
 dll);
 + push( @{ $install_list { 'lib' } }, 
 lib);
 + }
   }

 Surely if there are no lib/dll files in the subdirectory, nothing will
 happen, right?  (I haven't actually tried.)

No, it fails. And we should actually have a bin/lib that has been
correctly generated. Do you think this is a problem? Normally we
*should* fail IMO, meaning that the build process has broken what it
should have done.
-- 
Michael


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] Install shared libs in lib/ and bin/ with MSVC (Was: install libpq.dll in bin directory on Windows / Cygwin)

2015-03-12 Thread Asif Naeem 
Thank you Michael overall v3 patch looks good to me, There is one
observation that it is not installing following lib files that are required
for dev work i.e.

 inst\lib\libpq.lib
 inst\lib\libpgtypes.lib
 inst\lib\libecpg_compat.lib
 inst\lib\libecpg.lib

Please do let me know if I missed something but was not there a need to
avoid installing related .dll files in lib (duplicate) along with bin
directory e.g.

src\tools\msvc\Install.pm


 if ($is_sharedlib)
 {
 @dirs = qw(bin);
 }
 else
 {
 @dirs = qw(lib);
 }


Thanks.


On Mon, Mar 9, 2015 at 1:16 PM, Michael Paquier michael.paqu...@gmail.com
wrote:

 On Wed, Mar 4, 2015 at 4:13 AM, Michael Paquier wrote:
  Those entries are removed as well in the patch.

 Please find attached a new version of the patch, fixing a failure for
 plperl installation that contains GNUmakefile instead of Makefile.
 --
 Michael

Re: [HACKERS] Install shared libs in lib/ and bin/ with MSVC (Was: install libpq.dll in bin directory on Windows / Cygwin)

2015-03-12 Thread Michael Paquier 
On Wed, Mar 11, 2015 at 10:03 PM, Asif Naeem anaeem...@gmail.com wrote:
 Thank you Michael overall v3 patch looks good to me, There is one
 observation that it is not installing following lib files that are required
 for dev work i.e.

Thanks for your input.

 inst\lib\libpq.lib
 inst\lib\libpgtypes.lib
 inst\lib\libecpg_compat.lib
 inst\lib\libecpg.lib

 Please do let me know if I missed something but was not there a need to
 avoid installing related .dll files in lib (duplicate) along with bin
 directory e.g?

Yeas, right. Sorry for missing something like that. In HEAD, the
following things are installed regarding the shared libraries:
- In lib/, all .dll and .lib
- In bin/, only libpq.dll

So I have recoded the patch to use an hash of arrays (makes the code
more readable IMO) to be able to track more easily what to install
where, and process now does the following for shared libraries:
- In lib/, install all .dll and .lib
- In bin/, install all .dll

I hope that's fine to you.
Regards,
-- 
Michael
diff --git a/src/tools/msvc/Install.pm b/src/tools/msvc/Install.pm
index eba9aa0..c35a9dd 100644
--- a/src/tools/msvc/Install.pm
+++ b/src/tools/msvc/Install.pm
@@ -91,7 +91,6 @@ sub Install
 	}
 
 	CopySolutionOutput($conf, $target);
-	lcopy($target . '/lib/libpq.dll', $target . '/bin/libpq.dll');
 	my $sample_files = [];
 	my @top_dir  = (src);
 	@top_dir = (src\\bin, src\\interfaces) if ($insttype eq client);
@@ -108,12 +107,8 @@ sub Install
 		$target . '/lib/',
 		$conf\\,
 		postgres\\postgres.lib,
-		libpq\\libpq.lib,
-		libecpg\\libecpg.lib,
 		libpgcommon\\libpgcommon.lib,
-		libpgport\\libpgport.lib,
-		libpgtypes\\libpgtypes.lib,
-		libecpg_compat\\libecpg_compat.lib);
+		libpgport\\libpgport.lib);
 	CopyContribFiles($config, $target);
 	CopyIncludeFiles($target);
 
@@ -236,8 +231,16 @@ sub CopySolutionOutput
 	while ($sln =~ $rem)
 	{
 		my $pf = $1;
-		my $dir;
-		my $ext;
+
+		# Hash of array to list things to install. This has the
+		# shape similar to that for each project to indicate
+		# what should be installed where. The hash key indicates
+		# the installation location, and the array elements define
+		# the file type to install:
+		# 'bin' = [ 'dll', 'lib' ]
+		# 'lib' = [ 'lib' ]
+		my %install_list;
+		my $is_sharedlib = 0;
 
 		$sln =~ s/$rem//;
 
@@ -247,22 +250,47 @@ sub CopySolutionOutput
 
 		my $proj = read_file($pf.$vcproj)
 		  || croak Could not open $pf.$vcproj\n;
+
+		# Check if this project uses a shared library by looking if
+		# SO_MAJOR_VERSION is defined in its Makefile, whose path
+		# can be found using the resource file of this project.
+		if (($vcproj eq 'vcxproj' 
+			 $proj =~ qr{ResourceCompile\s*Include=([^]+)}) ||
+			($vcproj eq 'vcproj' 
+			 $proj =~ qr{File\s*RelativePath=([^\]+)\.rc}))
+		{
+			my $projpath = dirname($1);
+			my $mfname = -e $projpath/GNUmakefile ?
+$projpath/GNUmakefile : $projpath/Makefile;
+			my $mf = read_file($mfname) ||
+croak Could not open $mfname\n;
+
+			if ($mf =~ /^SO_MAJOR_VERSION\s*=\s*(.*)$/mg)
+			{
+$is_sharedlib = 1;
+			}
+		}
+
 		if ($vcproj eq 'vcproj'  $proj =~ qr{ConfigurationType=([^]+)})
 		{
 			if ($1 == 1)
 			{
-$dir = bin;
-$ext = exe;
+push( @{ $install_list { 'bin' } }, exe);
 			}
 			elsif ($1 == 2)
 			{
-$dir = lib;
-$ext = dll;
+push( @{ $install_list { 'lib' } }, dll);
+if ($is_sharedlib)
+{
+	push( @{ $install_list { 'bin' } }, dll);
+	push( @{ $install_list { 'lib' } }, lib);
+}
 			}
 			else
 			{
 
-# Static lib, such as libpgport, only used internally during build, don't install
+# Static libraries, such as libpgport, only used internally
+# during build, don't install.
 next;
 			}
 		}
@@ -271,18 +299,21 @@ sub CopySolutionOutput
 		{
 			if ($1 eq 'Application')
 			{
-$dir = bin;
-$ext = exe;
+push( @{ $install_list { 'bin' } }, exe);
 			}
 			elsif ($1 eq 'DynamicLibrary')
 			{
-$dir = lib;
-$ext = dll;
+push( @{ $install_list { 'lib' } }, dll);
+if ($is_sharedlib)
+{
+	push( @{ $install_list { 'bin' } }, dll);
+	push( @{ $install_list { 'lib' } }, lib);
+}
 			}
 			else# 'StaticLibrary'
 			{
-
-# Static lib, such as libpgport, only used internally during build, don't install
+# Static lib, such as libpgport, only used internally
+# during build, don't install.
 next;
 			}
 		}
@@ -290,8 +321,16 @@ sub CopySolutionOutput
 		{
 			croak Could not parse $pf.$vcproj\n;
 		}
-		lcopy($conf\\$pf\\$pf.$ext, $target\\$dir\\$pf.$ext)
-		  || croak Could not copy $pf.$ext\n;
+
+		# Install each element
+		foreach my $dir ( keys %install_list )
+		{
+			foreach my $ext ( @{ $install_list{ $dir } } )
+			{
+lcopy($conf\\$pf\\$pf.$ext, $target\\$dir\\$pf.$ext)
+	|| croak Could not copy $pf.$ext\n;
+			}
+		}
 		lcopy($conf\\$pf\\$pf.pdb, $target\\symbols\\$pf.pdb)
 		  || croak Could not copy $pf.pdb\n;
 		print .;

-- 
Sent via pgsql-hackers mailing list

Re: [HACKERS] Install shared libs in lib/ and bin/ with MSVC (Was: install libpq.dll in bin directory on Windows / Cygwin)

2015-03-09 Thread Michael Paquier 
On Wed, Mar 4, 2015 at 4:13 AM, Michael Paquier wrote:
 Those entries are removed as well in the patch.

Please find attached a new version of the patch, fixing a failure for
plperl installation that contains GNUmakefile instead of Makefile.
-- 
Michael
diff --git a/src/tools/msvc/Install.pm b/src/tools/msvc/Install.pm
index eba9aa0..fd6080f 100644
--- a/src/tools/msvc/Install.pm
+++ b/src/tools/msvc/Install.pm
@@ -91,7 +91,6 @@ sub Install
 	}
 
 	CopySolutionOutput($conf, $target);
-	lcopy($target . '/lib/libpq.dll', $target . '/bin/libpq.dll');
 	my $sample_files = [];
 	my @top_dir  = (src);
 	@top_dir = (src\\bin, src\\interfaces) if ($insttype eq client);
@@ -108,12 +107,8 @@ sub Install
 		$target . '/lib/',
 		$conf\\,
 		postgres\\postgres.lib,
-		libpq\\libpq.lib,
-		libecpg\\libecpg.lib,
 		libpgcommon\\libpgcommon.lib,
-		libpgport\\libpgport.lib,
-		libpgtypes\\libpgtypes.lib,
-		libecpg_compat\\libecpg_compat.lib);
+		libpgport\\libpgport.lib);
 	CopyContribFiles($config, $target);
 	CopyIncludeFiles($target);
 
@@ -236,8 +231,9 @@ sub CopySolutionOutput
 	while ($sln =~ $rem)
 	{
 		my $pf = $1;
-		my $dir;
+		my @dirs;
 		my $ext;
+		my $is_sharedlib = 0;
 
 		$sln =~ s/$rem//;
 
@@ -247,17 +243,42 @@ sub CopySolutionOutput
 
 		my $proj = read_file($pf.$vcproj)
 		  || croak Could not open $pf.$vcproj\n;
+
+		# Check if this project uses a shared library by looking if
+		# SO_MAJOR_VERSION is defined in its Makefile, whose path
+		# can be found using the resource file of this project.
+		if (($vcproj eq 'vcxproj' 
+			 $proj =~ qr{ResourceCompile\s*Include=([^]+)}) ||
+			($vcproj eq 'vcproj' 
+			 $proj =~ qr{File\s*RelativePath=([^\]+)\.rc}))
+		{
+			my $projpath = dirname($1);
+			my $mfname = -e $projpath/GNUmakefile ?
+$projpath/GNUmakefile : $projpath/Makefile;
+			my $mf = read_file($mfname) ||
+croak Could not open $mfname\n;
+
+			if ($mf =~ /^SO_MAJOR_VERSION\s*=\s*(.*)$/mg)
+			{
+$is_sharedlib = 1;
+			}
+		}
+
 		if ($vcproj eq 'vcproj'  $proj =~ qr{ConfigurationType=([^]+)})
 		{
 			if ($1 == 1)
 			{
-$dir = bin;
+@dirs = qw(bin);
 $ext = exe;
 			}
 			elsif ($1 == 2)
 			{
-$dir = lib;
+@dirs = qw(lib);
 $ext = dll;
+if ($is_sharedlib)
+{
+	push(@dirs, 'bin');
+}
 			}
 			else
 			{
@@ -271,13 +292,17 @@ sub CopySolutionOutput
 		{
 			if ($1 eq 'Application')
 			{
-$dir = bin;
+@dirs = qw(bin);
 $ext = exe;
 			}
 			elsif ($1 eq 'DynamicLibrary')
 			{
-$dir = lib;
+@dirs = qw(lib);
 $ext = dll;
+if ($is_sharedlib)
+{
+	push(@dirs, 'bin');
+}
 			}
 			else# 'StaticLibrary'
 			{
@@ -290,8 +315,11 @@ sub CopySolutionOutput
 		{
 			croak Could not parse $pf.$vcproj\n;
 		}
-		lcopy($conf\\$pf\\$pf.$ext, $target\\$dir\\$pf.$ext)
-		  || croak Could not copy $pf.$ext\n;
+		foreach my $dir (@dirs)
+		{
+			lcopy($conf\\$pf\\$pf.$ext, $target\\$dir\\$pf.$ext)
+|| croak Could not copy $pf.$ext\n;
+		}
 		lcopy($conf\\$pf\\$pf.pdb, $target\\symbols\\$pf.pdb)
 		  || croak Could not copy $pf.pdb\n;
 		print .;

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] Install shared libs in lib/ and bin/ with MSVC (Was: install libpq.dll in bin directory on Windows / Cygwin)

2015-03-04 Thread Michael Paquier 
On Wed, Mar 4, 2015 at 4:43 PM, Michael Paquier wrote:
 On Tue, Mar 3, 2015 at 8:36 PM, Asif Naeem wrote:
 Thank you Michael. I have looked the patch.

 Thanks for the review!

 Overall logic looks good to me,
 I have checked it with MSVC{2013,2008}. It works for MSVC 2013 but fail for
 MSVC 2008, I think the condition if ($proj =~
 qr{ResourceCompile\s*Include=([^]+)}) is not going to work for MSVC2008
 and MSVC2005 i.e.
 [...]

 Thanks for the details, my patch is definitely missing vcproj entries.
 Note that I don't have yet an environment with MSVC 2008 or similar, I
 just got 2010 on my box for now. So you will have to wait until I have
 a fresh environment to get an updated patch...

OK, so I have been able to figure out a regex expression scanning for
win32ver.rc in RelativePath for a vcproj file. For vcxproj files, I am
still using ResourceCompile. Attached is the promised patch.

 Although this patch is going to make MSVC
 build consistent with Cygwin and MinGW build, following files seems
 redundant now, is there any use for them other than backward compatibility ?
 i.e.
 inst\lib\libpq.dll
 inst\lib\libpgtypes.dll
 inst\lib\libecpg_compat.dll
 inst\lib\libecpg.dll

 Indeed, I haven't noticed them. But we can simply remove them as they
 are installed in bin/ as well with this patch, it seems better for
 consistency with MinGW and Cygwin in any case.

Those entries are removed as well in the patch.

Regards,
-- 
Michael
From 257685e441ad940012600202e720de447ae5d6dc Mon Sep 17 00:00:00 2001
From: Michael Paquier mich...@otacoo.com
Date: Tue, 23 Dec 2014 21:58:50 -0800
Subject: [PATCH] Install shared libraries in bin/ and lib/ on MSVC

This is the MSVC part of the previous commit, to ensure that install is
completely consistent with the multiple methods supported.
---
 src/tools/msvc/Install.pm | 52 +++
 1 file changed, 39 insertions(+), 13 deletions(-)

diff --git a/src/tools/msvc/Install.pm b/src/tools/msvc/Install.pm
index eba9aa0..237a0e6 100644
--- a/src/tools/msvc/Install.pm
+++ b/src/tools/msvc/Install.pm
@@ -91,7 +91,6 @@ sub Install
 	}
 
 	CopySolutionOutput($conf, $target);
-	lcopy($target . '/lib/libpq.dll', $target . '/bin/libpq.dll');
 	my $sample_files = [];
 	my @top_dir  = (src);
 	@top_dir = (src\\bin, src\\interfaces) if ($insttype eq client);
@@ -108,12 +107,8 @@ sub Install
 		$target . '/lib/',
 		$conf\\,
 		postgres\\postgres.lib,
-		libpq\\libpq.lib,
-		libecpg\\libecpg.lib,
 		libpgcommon\\libpgcommon.lib,
-		libpgport\\libpgport.lib,
-		libpgtypes\\libpgtypes.lib,
-		libecpg_compat\\libecpg_compat.lib);
+		libpgport\\libpgport.lib);
 	CopyContribFiles($config, $target);
 	CopyIncludeFiles($target);
 
@@ -236,8 +231,9 @@ sub CopySolutionOutput
 	while ($sln =~ $rem)
 	{
 		my $pf = $1;
-		my $dir;
+		my @dirs;
 		my $ext;
+		my $is_sharedlib = 0;
 
 		$sln =~ s/$rem//;
 
@@ -247,17 +243,40 @@ sub CopySolutionOutput
 
 		my $proj = read_file($pf.$vcproj)
 		  || croak Could not open $pf.$vcproj\n;
+
+		# Check if this project uses a shared library by looking if
+		# SO_MAJOR_VERSION is defined in its Makefile, whose path
+		# can be found using the resource file of this project.
+		if (($vcproj eq 'vcxproj' 
+			 $proj =~ qr{ResourceCompile\s*Include=([^]+)}) ||
+			($vcproj eq 'vcproj' 
+			 $proj =~ qr{File\s*RelativePath=([^\]+)\.rc}))
+		{
+			my $projpath = dirname($1);
+			my $mf = read_file($projpath . '/Makefile')
+|| croak Could not open $projpath/Makefile\n;
+
+			if ($mf =~ /^SO_MAJOR_VERSION\s*=\s*(.*)$/mg)
+			{
+$is_sharedlib = 1;
+			}
+		}
+
 		if ($vcproj eq 'vcproj'  $proj =~ qr{ConfigurationType=([^]+)})
 		{
 			if ($1 == 1)
 			{
-$dir = bin;
+@dirs = qw(bin);
 $ext = exe;
 			}
 			elsif ($1 == 2)
 			{
-$dir = lib;
+@dirs = qw(lib);
 $ext = dll;
+if ($is_sharedlib)
+{
+	push(@dirs, 'bin');
+}
 			}
 			else
 			{
@@ -271,13 +290,17 @@ sub CopySolutionOutput
 		{
 			if ($1 eq 'Application')
 			{
-$dir = bin;
+@dirs = qw(bin);
 $ext = exe;
 			}
 			elsif ($1 eq 'DynamicLibrary')
 			{
-$dir = lib;
+@dirs = qw(lib);
 $ext = dll;
+if ($is_sharedlib)
+{
+	push(@dirs, 'bin');
+}
 			}
 			else# 'StaticLibrary'
 			{
@@ -290,8 +313,11 @@ sub CopySolutionOutput
 		{
 			croak Could not parse $pf.$vcproj\n;
 		}
-		lcopy($conf\\$pf\\$pf.$ext, $target\\$dir\\$pf.$ext)
-		  || croak Could not copy $pf.$ext\n;
+		foreach my $dir (@dirs)
+		{
+			lcopy($conf\\$pf\\$pf.$ext, $target\\$dir\\$pf.$ext)
+|| croak Could not copy $pf.$ext\n;
+		}
 		lcopy($conf\\$pf\\$pf.pdb, $target\\symbols\\$pf.pdb)
 		  || croak Could not copy $pf.pdb\n;
 		print .;
-- 
1.9.2.msysgit.0


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] Install shared libs in lib/ and bin/ with MSVC (Was: install libpq.dll in bin directory on Windows / Cygwin)

2015-03-03 Thread Michael Paquier 
On Tue, Mar 3, 2015 at 8:36 PM, Asif Naeem anaeem...@gmail.com wrote:
 Thank you Michael. I have looked the patch.

Thanks for the review!

 Overall logic looks good to me,
 I have checked it with MSVC{2013,2008}. It works for MSVC 2013 but fail for
 MSVC 2008, I think the condition if ($proj =~
 qr{ResourceCompile\s*Include=([^]+)}) is not going to work for MSVC2008
 and MSVC2005 i.e.
 [...]

Thanks for the details, my patch is definitely missing vcproj entries.
Note that I don't have yet an environment with MSVC 2008 or similar, I
just got 2010 on my box for now. So you will have to wait until I have
a fresh environment to get an updated patch...

 It seems that search criteria can be narrowed to skip reading related
 Makefile for SO_MAJOR_VERSION string when VS project file is related to
 StaticLibrary or Application.

Well, agreed, and the patch takes care of that for vcxproj files by
only installing shared libraries if they use DynamicLibrary. Perhaps
you have in mind a better logic?

 Although this patch is going to make MSVC
 build consistent with Cygwin and MinGW build, following files seems
 redundant now, is there any use for them other than backward compatibility ?
 i.e.
 inst\lib\libpq.dll
 inst\lib\libpgtypes.dll
 inst\lib\libecpg_compat.dll
 inst\lib\libecpg.dll

Indeed, I haven't noticed them. But we can simply remove them as they
are installed in bin/ as well with this patch, it seems better for
consistency with MinGW and Cygwin in any case.
-- 
Michael


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] Install shared libs in lib/ and bin/ with MSVC (Was: install libpq.dll in bin directory on Windows / Cygwin)

2015-03-03 Thread Asif Naeem 
Thank you Michael. I have looked the patch. Overall logic looks good to me,
I have checked it with MSVC{2013,2008}. It works for MSVC 2013 but fail for
MSVC 2008, I think the condition if ($proj =~
qr{ResourceCompile\s*Include=([^]+)”})” is not going to work for MSVC2008
and MSVC2005 i.e.

MSVC2013

  ResourceCompile Include=src\interfaces\ecpg\ecpglib\win32ver.rc /


MSVC2008

  File RelativePath=src\interfaces\ecpg\ecpglib\win32ver.rc /


For more details please check i.e.

 src/tools/msvc/MSBuildProject.pm (Visual C++ 2010 or greater)
 src/tools/msvc/VCBuildProject.pm (Visual C++ 2005/2008)


It seems that search criteria can be narrowed to skip reading related
Makefile for SO_MAJOR_VERSION string when VS project file is related to
StaticLibrary or Application. Although this patch is going to make MSVC
build consistent with Cygwin and MinGW build, following files seems
redundant now, is there any use for them other than backward compatibility
? i.e.

inst\lib\libpq.dll
 inst\lib\libpgtypes.dll
 inst\lib\libecpg_compat.dll
 inst\lib\libecpg.dll


Thanks.

Regards,
Muhammad Asif Naeem

On Tue, Jan 20, 2015 at 6:06 AM, Michael Paquier michael.paqu...@gmail.com
wrote:

 Hi all,

 Here is a continuation of the thread which covered $subject for MinGW
 and Cygwin:
 http://www.postgresql.org/message-id/51f19059.7050...@pgexperts.com
 But this time it is for MSVC.

 Just a bit of background... Since commit cb4a3b04 we are installing
 shared libraries in bin/ and lib/ for Cygwin and MinGW on Windows, but
 we are still missing MSVC, so as of now build method is inconsistent.
 Attached is a patch aimed at fixing this inconsistency. What it does
 is simple: when kicking Install.pm to install the deliverables of each
 project, we check if the project Makefile contains SO_MAJOR_VERSION.
 If yes, libraries of this project are installed in bin/ and lib/. The
 path to the Makefile is found by scanning ResourceCompile in the
 vcproj file, this method having the advantage to make the patch
 independent of build process.

 This also removes a wart present in Install.pm installing copying
 manually libpq.dll:
 -   lcopy($target . '/lib/libpq.dll', $target . '/bin/libpq.dll');

 I am adding an entry in the upcoming CF for this patch, and let's use
 this thread for this discussion.
 Regards,
 --
 Michael


 --
 Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
 To make changes to your subscription:
 http://www.postgresql.org/mailpref/pgsql-hackers

[HACKERS] Install shared libs in lib/ and bin/ with MSVC (Was: install libpq.dll in bin directory on Windows / Cygwin)

2015-01-19 Thread Michael Paquier 
Hi all,

Here is a continuation of the thread which covered $subject for MinGW
and Cygwin:
http://www.postgresql.org/message-id/51f19059.7050...@pgexperts.com
But this time it is for MSVC.

Just a bit of background... Since commit cb4a3b04 we are installing
shared libraries in bin/ and lib/ for Cygwin and MinGW on Windows, but
we are still missing MSVC, so as of now build method is inconsistent.
Attached is a patch aimed at fixing this inconsistency. What it does
is simple: when kicking Install.pm to install the deliverables of each
project, we check if the project Makefile contains SO_MAJOR_VERSION.
If yes, libraries of this project are installed in bin/ and lib/. The
path to the Makefile is found by scanning ResourceCompile in the
vcproj file, this method having the advantage to make the patch
independent of build process.

This also removes a wart present in Install.pm installing copying
manually libpq.dll:
-   lcopy($target . '/lib/libpq.dll', $target . '/bin/libpq.dll');

I am adding an entry in the upcoming CF for this patch, and let's use
this thread for this discussion.
Regards,
-- 
Michael
From 13ca64cb5cdae419d6ee1bbf7c7a04f6bd3d2388 Mon Sep 17 00:00:00 2001
From: Michael Paquier michael@otacoo.com
Date: Tue, 23 Dec 2014 21:58:50 -0800
Subject: [PATCH 2/2] Install shared libraries in bin/ and lib/ with MSVC

This is the MSVC part of the previous commit, to ensure that install is
completely consistent with the multiple methods supported.
---
 src/tools/msvc/Install.pm | 43 +++
 1 file changed, 35 insertions(+), 8 deletions(-)

diff --git a/src/tools/msvc/Install.pm b/src/tools/msvc/Install.pm
index eba9aa0..616cd9d 100644
--- a/src/tools/msvc/Install.pm
+++ b/src/tools/msvc/Install.pm
@@ -91,7 +91,6 @@ sub Install
 	}
 
 	CopySolutionOutput($conf, $target);
-	lcopy($target . '/lib/libpq.dll', $target . '/bin/libpq.dll');
 	my $sample_files = [];
 	my @top_dir  = (src);
 	@top_dir = (src\\bin, src\\interfaces) if ($insttype eq client);
@@ -236,8 +235,9 @@ sub CopySolutionOutput
 	while ($sln =~ $rem)
 	{
 		my $pf = $1;
-		my $dir;
+		my @dirs;
 		my $ext;
+		my $is_sharedlib = 0;
 
 		$sln =~ s/$rem//;
 
@@ -247,17 +247,37 @@ sub CopySolutionOutput
 
 		my $proj = read_file($pf.$vcproj)
 		  || croak Could not open $pf.$vcproj\n;
+
+		# Check this project uses a shared library by looking if
+		# SO_MAJOR_VERSION is defined in its Makefile, whose path
+		# can be found using the resource file of this project.
+		if ($proj =~ qr{ResourceCompile\s*Include=([^]+)})
+		{
+			my $projpath = dirname($1);
+			my $mf = read_file($projpath . '/Makefile')
+|| croak Could not open $projpath/Makefile\n;
+
+			if ($mf =~ /^SO_MAJOR_VERSION\s*=\s*(.*)$/mg)
+			{
+$is_sharedlib = 1;
+			}
+		}
+
 		if ($vcproj eq 'vcproj'  $proj =~ qr{ConfigurationType=([^]+)})
 		{
 			if ($1 == 1)
 			{
-$dir = bin;
+@dirs = qw(bin);
 $ext = exe;
 			}
 			elsif ($1 == 2)
 			{
-$dir = lib;
+@dirs = qw(lib);
 $ext = dll;
+if ($is_sharedlib)
+{
+	push(@dirs, 'bin');
+}
 			}
 			else
 			{
@@ -271,13 +291,17 @@ sub CopySolutionOutput
 		{
 			if ($1 eq 'Application')
 			{
-$dir = bin;
+@dirs = qw(bin);
 $ext = exe;
 			}
 			elsif ($1 eq 'DynamicLibrary')
 			{
-$dir = lib;
+@dirs = qw(lib);
 $ext = dll;
+if ($is_sharedlib)
+{
+	push(@dirs, 'bin');
+}
 			}
 			else# 'StaticLibrary'
 			{
@@ -290,8 +314,11 @@ sub CopySolutionOutput
 		{
 			croak Could not parse $pf.$vcproj\n;
 		}
-		lcopy($conf\\$pf\\$pf.$ext, $target\\$dir\\$pf.$ext)
-		  || croak Could not copy $pf.$ext\n;
+		foreach my $dir (@dirs)
+		{
+			lcopy($conf\\$pf\\$pf.$ext, $target\\$dir\\$pf.$ext)
+|| croak Could not copy $pf.$ext\n;
+		}
 		lcopy($conf\\$pf\\$pf.pdb, $target\\symbols\\$pf.pdb)
 		  || croak Could not copy $pf.pdb\n;
 		print .;
-- 
2.2.1


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers