public inbox for [email protected]
help / color / mirror / Atom feedError building for 64-bit Windows (10)
10+ messages / 5 participants
[nested] [flat]
* Error building for 64-bit Windows (10)
@ 2021-05-17 20:07 PG Doc comments form <[email protected]>
2021-05-18 05:56 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
0 siblings, 1 reply; 10+ messages in thread
From: PG Doc comments form @ 2021-05-17 20:07 UTC (permalink / raw)
To: [email protected]; +Cc: [email protected]
The following documentation comment has been logged on the website:
Page: https://www.postgresql.org/docs/13/install-windows-full.html
Description:
The Solution.pm file has the following lines:
if ($self->{options}->{gss})
{
$proj->AddIncludeDir($self->{options}->{gss} . '\inc\krb5');
$proj->AddLibrary($self->{options}->{gss} . '\lib\i386\krb5_32.lib');
$proj->AddLibrary($self->{options}->{gss} .
'\lib\i386\comerr32.lib');
$proj->AddLibrary($self->{options}->{gss} .
'\lib\i386\gssapi32.lib');
}
I had to change them to the following or the compiling failed:
if ($self->{options}->{gss})
{
$proj->AddIncludeDir($self->{options}->{gss} . '\include');
$proj->AddIncludeDir($self->{options}->{gss} . '\include\krb5');
$proj->AddLibrary($self->{options}->{gss} . '\lib\amd64\krb5_64.lib');
$proj->AddLibrary($self->{options}->{gss} . '\lib\amd64\comerr64.lib');
$proj->AddLibrary($self->{options}->{gss} . '\lib\amd64\gssapi64.lib');
^ permalink raw reply [nested|flat] 10+ messages in thread
* Re: Error building for 64-bit Windows (10)
2021-05-17 20:07 Error building for 64-bit Windows (10) PG Doc comments form <[email protected]>
@ 2021-05-18 05:56 ` Michael Paquier <[email protected]>
2021-05-18 13:15 ` Re: Error building for 64-bit Windows (10) Brian Ye <[email protected]>
0 siblings, 1 reply; 10+ messages in thread
From: Michael Paquier @ 2021-05-18 05:56 UTC (permalink / raw)
To: [email protected]; [email protected]
On Mon, May 17, 2021 at 08:07:02PM +0000, PG Doc comments form wrote:
> The Solution.pm file has the following lines:
> if ($self->{options}->{gss})
> {
> $proj->AddIncludeDir($self->{options}->{gss} . '\inc\krb5');
> $proj->AddLibrary($self->{options}->{gss} . '\lib\i386\krb5_32.lib');
> $proj->AddLibrary($self->{options}->{gss} .
> '\lib\i386\comerr32.lib');
> $proj->AddLibrary($self->{options}->{gss} .
> '\lib\i386\gssapi32.lib');
> }
> I had to change them to the following or the compiling failed:
> if ($self->{options}->{gss})
> {
> $proj->AddIncludeDir($self->{options}->{gss} . '\include');
> $proj->AddIncludeDir($self->{options}->{gss} . '\include\krb5');
> $proj->AddLibrary($self->{options}->{gss} . '\lib\amd64\krb5_64.lib');
> $proj->AddLibrary($self->{options}->{gss} . '\lib\amd64\comerr64.lib');
> $proj->AddLibrary($self->{options}->{gss} . '\lib\amd64\gssapi64.lib');
Yes, you are right. I have been playing with the deliverables we
recommend in the docs as of [1], and there are a couple of gotchas
here:
- For the 32b and 64b MSI installer, the include path is not "inc",
but "include". So I could not get the installation on Win32 to work
either on HEAD.
- There is a sub-path called "include/krb5", which is not really
necessary except if we use krb5.h, but we don't. Upstream code
recommends actually to use krb5/krb5.h, meaning that only "include/"
would be sufficient. Keeping "include/krb5/" around is not a big deal
either.
This has not been changed in ages, so perhaps few have bothered.
Anyway, the attached patch fixes both the 32b and 64b builds for me.
Another interesting thing is that the installation of krb5 for amd64
and i386 cannot co-exist together, so installing one removes the
second automatically.
[1]: https://web.mit.edu/Kerberos/dist/index.html
--
Michael
Attachments:
[text/x-diff] fix-gss-msvc.patch (1.2K, 2-fix-gss-msvc.patch)
download | inline diff:
diff --git a/src/tools/msvc/Solution.pm b/src/tools/msvc/Solution.pm
index 3c5fe5dddc..05866af925 100644
--- a/src/tools/msvc/Solution.pm
+++ b/src/tools/msvc/Solution.pm
@@ -1023,10 +1023,20 @@ sub AddProject
}
if ($self->{options}->{gss})
{
- $proj->AddIncludeDir($self->{options}->{gss} . '\inc\krb5');
- $proj->AddLibrary($self->{options}->{gss} . '\lib\i386\krb5_32.lib');
- $proj->AddLibrary($self->{options}->{gss} . '\lib\i386\comerr32.lib');
- $proj->AddLibrary($self->{options}->{gss} . '\lib\i386\gssapi32.lib');
+ $proj->AddIncludeDir($self->{options}->{gss} . '\include');
+ $proj->AddIncludeDir($self->{options}->{gss} . '\include\krb5');
+ if ($self->{platform} eq 'Win32')
+ {
+ $proj->AddLibrary($self->{options}->{gss} . '\lib\i386\krb5_32.lib');
+ $proj->AddLibrary($self->{options}->{gss} . '\lib\i386\comerr32.lib');
+ $proj->AddLibrary($self->{options}->{gss} . '\lib\i386\gssapi32.lib');
+ }
+ else
+ {
+ $proj->AddLibrary($self->{options}->{gss} . '\lib\amd64\krb5_64.lib');
+ $proj->AddLibrary($self->{options}->{gss} . '\lib\amd64\comerr64.lib');
+ $proj->AddLibrary($self->{options}->{gss} . '\lib\amd64\gssapi64.lib');
+ }
}
if ($self->{options}->{iconv})
{
[application/pgp-signature] signature.asc (833B, 3-signature.asc)
download
^ permalink raw reply [nested|flat] 10+ messages in thread
* Re: Error building for 64-bit Windows (10)
2021-05-17 20:07 Error building for 64-bit Windows (10) PG Doc comments form <[email protected]>
2021-05-18 05:56 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
@ 2021-05-18 13:15 ` Brian Ye <[email protected]>
2021-05-19 00:15 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
0 siblings, 1 reply; 10+ messages in thread
From: Brian Ye @ 2021-05-18 13:15 UTC (permalink / raw)
To: Michael Paquier <[email protected]>; +Cc: [email protected]
Hi Michael,
Thanks for the reply.
Yes I also saw that after installing 64-bit, the 32-bit "bin" and "include"
directories were removed.
I think the content of the "include" are common for both 32- and 64-bit.
Windows can run both 32-bit and
64-bit binaries so removing these 2 directories is probably okay. Just my
guess.
Again, thanks!
Brian Ye
On Tue, May 18, 2021 at 12:57 AM Michael Paquier <[email protected]>
wrote:
> On Mon, May 17, 2021 at 08:07:02PM +0000, PG Doc comments form wrote:
> > The Solution.pm file has the following lines:
> > if ($self->{options}->{gss})
> > {
> > $proj->AddIncludeDir($self->{options}->{gss} . '\inc\krb5');
> > $proj->AddLibrary($self->{options}->{gss} .
> '\lib\i386\krb5_32.lib');
> > $proj->AddLibrary($self->{options}->{gss} .
> > '\lib\i386\comerr32.lib');
> > $proj->AddLibrary($self->{options}->{gss} .
> > '\lib\i386\gssapi32.lib');
> > }
> > I had to change them to the following or the compiling failed:
> > if ($self->{options}->{gss})
> > {
> > $proj->AddIncludeDir($self->{options}->{gss} . '\include');
> > $proj->AddIncludeDir($self->{options}->{gss} .
> '\include\krb5');
> > $proj->AddLibrary($self->{options}->{gss} .
> '\lib\amd64\krb5_64.lib');
> > $proj->AddLibrary($self->{options}->{gss} .
> '\lib\amd64\comerr64.lib');
> > $proj->AddLibrary($self->{options}->{gss} .
> '\lib\amd64\gssapi64.lib');
>
> Yes, you are right. I have been playing with the deliverables we
> recommend in the docs as of [1], and there are a couple of gotchas
> here:
> - For the 32b and 64b MSI installer, the include path is not "inc",
> but "include". So I could not get the installation on Win32 to work
> either on HEAD.
> - There is a sub-path called "include/krb5", which is not really
> necessary except if we use krb5.h, but we don't. Upstream code
> recommends actually to use krb5/krb5.h, meaning that only "include/"
> would be sufficient. Keeping "include/krb5/" around is not a big deal
> either.
>
> This has not been changed in ages, so perhaps few have bothered.
> Anyway, the attached patch fixes both the 32b and 64b builds for me.
> Another interesting thing is that the installation of krb5 for amd64
> and i386 cannot co-exist together, so installing one removes the
> second automatically.
>
> [1]: https://web.mit.edu/Kerberos/dist/index.html
> --
> Michael
>
^ permalink raw reply [nested|flat] 10+ messages in thread
* Re: Error building for 64-bit Windows (10)
2021-05-17 20:07 Error building for 64-bit Windows (10) PG Doc comments form <[email protected]>
2021-05-18 05:56 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
2021-05-18 13:15 ` Re: Error building for 64-bit Windows (10) Brian Ye <[email protected]>
@ 2021-05-19 00:15 ` Michael Paquier <[email protected]>
2021-05-19 14:56 ` Re: Error building for 64-bit Windows (10) Andrew Dunstan <[email protected]>
0 siblings, 1 reply; 10+ messages in thread
From: Michael Paquier @ 2021-05-19 00:15 UTC (permalink / raw)
To: Brian Ye <[email protected]>; +Cc: [email protected]; Andrew Dunstan <[email protected]>
Hi Andrew,
On Tue, May 18, 2021 at 08:15:35AM -0500, Brian Ye wrote:
> Thanks for the reply.
> Yes I also saw that after installing 64-bit, the 32-bit "bin" and "include"
> directories were removed.
> I think the content of the "include" are common for both 32- and 64-bit.
> Windows can run both 32-bit and
> 64-bit binaries so removing these 2 directories is probably okay. Just my
> guess.
> Again, thanks!
> Brian Ye
hamerkop is the only buildfarm member testing krb5 builds with MSVC on
Windows. The path used in this case is c:\Program Files\MIT\Kerberos\
so the patch of this thread is going to break the builds there if this
relies on "inc/" for the include path. Is this the original include
folder used for this installation of krb5?
Thanks,
--
Michael
Attachments:
[application/pgp-signature] signature.asc (833B, 2-signature.asc)
download
^ permalink raw reply [nested|flat] 10+ messages in thread
* Re: Error building for 64-bit Windows (10)
2021-05-17 20:07 Error building for 64-bit Windows (10) PG Doc comments form <[email protected]>
2021-05-18 05:56 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
2021-05-18 13:15 ` Re: Error building for 64-bit Windows (10) Brian Ye <[email protected]>
2021-05-19 00:15 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
@ 2021-05-19 14:56 ` Andrew Dunstan <[email protected]>
2021-05-20 00:05 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
0 siblings, 1 reply; 10+ messages in thread
From: Andrew Dunstan @ 2021-05-19 14:56 UTC (permalink / raw)
To: Michael Paquier <[email protected]>; Brian Ye <[email protected]>; +Cc: [email protected]
On 5/18/21 8:15 PM, Michael Paquier wrote:
> Hi Andrew,
>
> On Tue, May 18, 2021 at 08:15:35AM -0500, Brian Ye wrote:
>> Thanks for the reply.
>> Yes I also saw that after installing 64-bit, the 32-bit "bin" and "include"
>> directories were removed.
>> I think the content of the "include" are common for both 32- and 64-bit.
>> Windows can run both 32-bit and
>> 64-bit binaries so removing these 2 directories is probably okay. Just my
>> guess.
>> Again, thanks!
>> Brian Ye
> hamerkop is the only buildfarm member testing krb5 builds with MSVC on
> Windows. The path used in this case is c:\Program Files\MIT\Kerberos\
> so the patch of this thread is going to break the builds there if this
> relies on "inc/" for the include path. Is this the original include
> folder used for this installation of krb5?
>
I have no idea - it's not my animal. Maybe ask the owner
<[email protected]>
I guess maybe I should look at adding kerberos to one of the animals I
do control.
cheers
andrew
--
Andrew Dunstan
EDB: https://www.enterprisedb.com
^ permalink raw reply [nested|flat] 10+ messages in thread
* Re: Error building for 64-bit Windows (10)
2021-05-17 20:07 Error building for 64-bit Windows (10) PG Doc comments form <[email protected]>
2021-05-18 05:56 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
2021-05-18 13:15 ` Re: Error building for 64-bit Windows (10) Brian Ye <[email protected]>
2021-05-19 00:15 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
2021-05-19 14:56 ` Re: Error building for 64-bit Windows (10) Andrew Dunstan <[email protected]>
@ 2021-05-20 00:05 ` Michael Paquier <[email protected]>
2021-05-20 02:04 ` Re: Error building for 64-bit Windows (10) =?ISO-2022-JP?B?GyRCNmFGI006QkAbKEI=?= <[email protected]>
0 siblings, 1 reply; 10+ messages in thread
From: Michael Paquier @ 2021-05-20 00:05 UTC (permalink / raw)
To: Andrew Dunstan <[email protected]>; +Cc: Brian Ye <[email protected]>; [email protected]; [email protected]
On Wed, May 19, 2021 at 10:56:23AM -0400, Andrew Dunstan wrote:
> I have no idea - it's not my animal. Maybe ask the owner
> <[email protected]>
Oops, I thought that this was yours. I am adding those folks in CC of
this thread.
> I guess maybe I should look at adding kerberos to one of the animals I
> do control.
That would be nice. I think that hamerkop should really update its
installation of krb5 as a first step.
@ Owners of hamerkop, could you look at updating the krb5 installation
on this animal? This include path used seems out of date compared to
the MSI installers upstream provides.
--
Michael
Attachments:
[application/pgp-signature] signature.asc (833B, 2-signature.asc)
download
^ permalink raw reply [nested|flat] 10+ messages in thread
* Re: Error building for 64-bit Windows (10)
2021-05-17 20:07 Error building for 64-bit Windows (10) PG Doc comments form <[email protected]>
2021-05-18 05:56 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
2021-05-18 13:15 ` Re: Error building for 64-bit Windows (10) Brian Ye <[email protected]>
2021-05-19 00:15 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
2021-05-19 14:56 ` Re: Error building for 64-bit Windows (10) Andrew Dunstan <[email protected]>
2021-05-20 00:05 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
@ 2021-05-20 02:04 ` =?ISO-2022-JP?B?GyRCNmFGI006QkAbKEI=?= <[email protected]>
2021-05-20 02:19 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
0 siblings, 1 reply; 10+ messages in thread
From: =?ISO-2022-JP?B?GyRCNmFGI006QkAbKEI=?= @ 2021-05-20 02:04 UTC (permalink / raw)
To: Michael Paquier <[email protected]>; +Cc: Andrew Dunstan <[email protected]>; Brian Ye <[email protected]>; [email protected]; [email protected]
Hi Michael,
> @ Owners of hamerkop, could you look at updating the krb5 installation
> on this animal? This include path used seems out of date compared to
> the MSI installers upstream provides.
OK. We'll have it done in a few days.
--
Kondo Yuta <[email protected]>
^ permalink raw reply [nested|flat] 10+ messages in thread
* Re: Error building for 64-bit Windows (10)
2021-05-17 20:07 Error building for 64-bit Windows (10) PG Doc comments form <[email protected]>
2021-05-18 05:56 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
2021-05-18 13:15 ` Re: Error building for 64-bit Windows (10) Brian Ye <[email protected]>
2021-05-19 00:15 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
2021-05-19 14:56 ` Re: Error building for 64-bit Windows (10) Andrew Dunstan <[email protected]>
2021-05-20 00:05 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
2021-05-20 02:04 ` Re: Error building for 64-bit Windows (10) =?ISO-2022-JP?B?GyRCNmFGI006QkAbKEI=?= <[email protected]>
@ 2021-05-20 02:19 ` Michael Paquier <[email protected]>
2021-05-27 10:21 ` Re: Error building for 64-bit Windows (10) =?ISO-2022-JP?B?GyRCNmFGI006QkAbKEI=?= <[email protected]>
0 siblings, 1 reply; 10+ messages in thread
From: Michael Paquier @ 2021-05-20 02:19 UTC (permalink / raw)
To: 近藤雄太 <[email protected]>; +Cc: Andrew Dunstan <[email protected]>; Brian Ye <[email protected]>; [email protected]; [email protected]
On Thu, May 20, 2021 at 11:04:05AM +0900, 近藤雄太 wrote:
>> @ Owners of hamerkop, could you look at updating the krb5 installation
>> on this animal? This include path used seems out of date compared to
>> the MSI installers upstream provides.
>
> OK. We'll have it done in a few days.
Thanks!
--
Michael
Attachments:
[application/pgp-signature] signature.asc (833B, 2-signature.asc)
download
^ permalink raw reply [nested|flat] 10+ messages in thread
* Re: Error building for 64-bit Windows (10)
2021-05-17 20:07 Error building for 64-bit Windows (10) PG Doc comments form <[email protected]>
2021-05-18 05:56 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
2021-05-18 13:15 ` Re: Error building for 64-bit Windows (10) Brian Ye <[email protected]>
2021-05-19 00:15 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
2021-05-19 14:56 ` Re: Error building for 64-bit Windows (10) Andrew Dunstan <[email protected]>
2021-05-20 00:05 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
2021-05-20 02:04 ` Re: Error building for 64-bit Windows (10) =?ISO-2022-JP?B?GyRCNmFGI006QkAbKEI=?= <[email protected]>
2021-05-20 02:19 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
@ 2021-05-27 10:21 ` =?ISO-2022-JP?B?GyRCNmFGI006QkAbKEI=?= <[email protected]>
2021-05-27 10:55 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
0 siblings, 1 reply; 10+ messages in thread
From: =?ISO-2022-JP?B?GyRCNmFGI006QkAbKEI=?= @ 2021-05-27 10:21 UTC (permalink / raw)
To: Michael Paquier <[email protected]>; +Cc: [email protected]; Brian Ye <[email protected]>; Andrew Dunstan <[email protected]>; [email protected]
Hi,
>>> @ Owners of hamerkop, could you look at updating the krb5 installation
>>> on this animal? This include path used seems out of date compared to
>>> the MSI installers upstream provides.
>>
>> OK. We'll have it done in a few days.
>
>Thanks!
Thanks for your patience.
We found some problems with hamerkop, so we ended up updating two things.
First, the config option that specified the kerberos installation
directory was still "krb5", so we changed it to "gss".
Secondly, kerberos SDK (include, lib) was not installed on hamerkop, so
we installed it. After that, we made sure that the "include" (not "inc")
directory existed under the kerberos installation directory
"C:\Program Files\MIT\Kerberos" which we set to "gss".
Please check.
--
Kondo Yuta <[email protected]>
^ permalink raw reply [nested|flat] 10+ messages in thread
* Re: Error building for 64-bit Windows (10)
2021-05-17 20:07 Error building for 64-bit Windows (10) PG Doc comments form <[email protected]>
2021-05-18 05:56 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
2021-05-18 13:15 ` Re: Error building for 64-bit Windows (10) Brian Ye <[email protected]>
2021-05-19 00:15 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
2021-05-19 14:56 ` Re: Error building for 64-bit Windows (10) Andrew Dunstan <[email protected]>
2021-05-20 00:05 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
2021-05-20 02:04 ` Re: Error building for 64-bit Windows (10) =?ISO-2022-JP?B?GyRCNmFGI006QkAbKEI=?= <[email protected]>
2021-05-20 02:19 ` Re: Error building for 64-bit Windows (10) Michael Paquier <[email protected]>
2021-05-27 10:21 ` Re: Error building for 64-bit Windows (10) =?ISO-2022-JP?B?GyRCNmFGI006QkAbKEI=?= <[email protected]>
@ 2021-05-27 10:55 ` Michael Paquier <[email protected]>
0 siblings, 0 replies; 10+ messages in thread
From: Michael Paquier @ 2021-05-27 10:55 UTC (permalink / raw)
To: 近藤雄太 <[email protected]>; +Cc: [email protected]; Brian Ye <[email protected]>; Andrew Dunstan <[email protected]>; [email protected]
On Thu, May 27, 2021 at 07:21:16PM +0900, 近藤雄太 wrote:
> First, the config option that specified the kerberos installation
> directory was still "krb5", so we changed it to "gss".
Thanks Kondo-san. Wow. So this has been actually untested for many
years. The stuff got renamed in the MSVC scripts as of 74a72ec from
2014.
> Secondly, kerberos SDK (include, lib) was not installed on hamerkop, so
> we installed it. After that, we made sure that the "include" (not "inc")
> directory existed under the kerberos installation directory
> "C:\Program Files\MIT\Kerberos" which we set to "gss".
>
> Please check.
From what I can see, hamerkop now fails to build, which is what I
would expect with the current code. I'll go fix this stuff and
backpatch, that should bring this buildfarm to green.
--
Michael
Attachments:
[application/pgp-signature] signature.asc (833B, 2-signature.asc)
download
^ permalink raw reply [nested|flat] 10+ messages in thread
end of thread, other threads:[~2021-05-27 10:55 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2021-05-17 20:07 Error building for 64-bit Windows (10) PG Doc comments form <[email protected]>
2021-05-18 05:56 ` Michael Paquier <[email protected]>
2021-05-18 13:15 ` Brian Ye <[email protected]>
2021-05-19 00:15 ` Michael Paquier <[email protected]>
2021-05-19 14:56 ` Andrew Dunstan <[email protected]>
2021-05-20 00:05 ` Michael Paquier <[email protected]>
2021-05-20 02:04 ` =?ISO-2022-JP?B?GyRCNmFGI006QkAbKEI=?= <[email protected]>
2021-05-20 02:19 ` Michael Paquier <[email protected]>
2021-05-27 10:21 ` =?ISO-2022-JP?B?GyRCNmFGI006QkAbKEI=?= <[email protected]>
2021-05-27 10:55 ` Michael Paquier <[email protected]>
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox