BUG #16343: Build problem: Install.pm "Could not find debug or release binaries"
The following bug has been logged on the website:
Bug reference: 16343
Logged by: Arnold Müller
Email address: noeld.mueller@googlemail.com
PostgreSQL version: 12.0
Operating system: Windows 10 Home
Description:
During a build with vcpkg I got above error message during the install phase
after a successfull build phase.
(I tried to install
ulibpq:x64-windows 12.0-1
)
It turned out that I got another, completely unrelated configure-file 7
folders up the hierarchy.
Inserting an elsif fixed it.
So, please consider merging this one into the Install.pm. It worked for me
on Windows 10 Home with vcpkg/MSVC, haven't tried it on other platforms,
though.
Thanks,
Arnold
diff --git a/src/tools/msvc/Install.pm b/src/tools/msvc/Install.pm
index 1a92ed233a..9173d0ee53 100644
--- a/src/tools/msvc/Install.pm
+++ b/src/tools/msvc/Install.pm
@@ -63,8 +63,14 @@ sub Install
do "./config.pl" if (-f "config.pl");
}- chdir("../../..") if (-f "../../../configure");
- chdir("../../../..") if (-f "../../../../configure");
+ if (-f "../../../configure")
+ {
+ chdir("../../..");
+ }
+ elsif (-f "../../../../configure")
+ {
+ chdir("../../../..");
+ }
my $conf = "";
if (-d "debug")
{
On 4 Apr 2020, at 17:08, PG Bug reporting form <noreply@postgresql.org> wrote:
It turned out that I got another, completely unrelated configure-file 7
folders up the hierarchy.
Inserting an elsif fixed it.So, please consider merging this one into the Install.pm. It worked for me
on Windows 10 Home with vcpkg/MSVC, haven't tried it on other platforms,
though.
While I haven't tested it, the proposed change makes perfect sense, we need a
conditional between the two. +1 on this.
cheers ./daniel
On 6 Apr 2020, at 08:36, Daniel Gustafsson <daniel@yesql.se> wrote:
On 4 Apr 2020, at 17:08, PG Bug reporting form <noreply@postgresql.org> wrote:
It turned out that I got another, completely unrelated configure-file 7
folders up the hierarchy.
Inserting an elsif fixed it.So, please consider merging this one into the Install.pm. It worked for me
on Windows 10 Home with vcpkg/MSVC, haven't tried it on other platforms,
though.While I haven't tested it, the proposed change makes perfect sense, we need a
conditional between the two. +1 on this.
Can you please add this patch to the next commitfest to make it's not forgotten
about?
https://commitfest.postgresql.org/28/
cheers ./daniel
On Wed, May 20, 2020 at 02:05:28PM +0200, Daniel Gustafsson wrote:
Can you please add this patch to the next commitfest to make it's not forgotten
about?
Applied and back-patched down to 9.5. Sorry for the delay.
--
Michael