Small issue with kerberos tests

Started by Maxim Orlov8 months ago2 messageshackers
Jump to latest
#1Maxim Orlov
orlovmg@gmail.com

Hi!

Recently, I attempted to run the entire set of pg tests with
PG_TEST_EXTRA="kerberos ..." on NetBSD but encountered
a problem. The kerberos/001_auth and
libpq/005_negotiate_encryption.pl test failed with an incomprehensible
message:
[01:55:38.737](3.698s) # setting up Kerberos
# Running: krb5-config --version
Undefined subroutine &PostgreSQL::Test::Kerberos::BAIL_OUT called at
/home/omg/proj/postgres/src/test/perl/PostgreSQL/Test/Kerberos.pm line 109.

The root cause of the problem was in using
------
$ /usr/bin/krb5-config --version
heimdal 7.8.0
------

...insted of..

------
$ /usr/pkg/bin/krb5-config --version
Kerberos 5 release 1.21.3
------

In other words, PATH must be properly set to use Kerberos 5 version.
But, AFIACS, "use Test::More" is missing in Kerberos.pm resulting
in "Undefined subroutine" error instead of "Heimdal is not supported".

--
Best regards,
Maxim Orlov.

Attachments:

v1-0001-BAIL_OUT-properly-if-krb5-config-is-not-Kerberos-.patchapplication/octet-stream; name=v1-0001-BAIL_OUT-properly-if-krb5-config-is-not-Kerberos-.patchDownload+1-1
#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Maxim Orlov (#1)
Re: Small issue with kerberos tests

Maxim Orlov <orlovmg@gmail.com> writes:

Recently, I attempted to run the entire set of pg tests with
PG_TEST_EXTRA="kerberos ..." on NetBSD but encountered
a problem. The kerberos/001_auth and
libpq/005_negotiate_encryption.pl test failed with an incomprehensible
message:
[01:55:38.737](3.698s) # setting up Kerberos
# Running: krb5-config --version
Undefined subroutine &PostgreSQL::Test::Kerberos::BAIL_OUT called at
/home/omg/proj/postgres/src/test/perl/PostgreSQL/Test/Kerberos.pm line 109.

Replicated here on FreeBSD (after hot-wiring the script to not know it
should use /usr/local/bin/krb5-config).

It looks like every other use of BAIL_OUT is in a file that does
have "use Test::More", so this is the only such oversight.
Will push, thanks for the report!

regards, tom lane