Skip Menu |
 

History Show all quoted text — Show full headers
# Fri Oct 02 16:43:31 2015 ghudson@mit.edu (Greg Hudson) - Ticket created
Subject: Installed krb5.conf files can affect test suite
Download (untitled) / with headers
text/plain 269B
We generally try to set KRB5_CONFIG for every C program which uses
krb5_init_context(), but currently there are a number of exceptions:

* The tests in lib/krad
* Some of the tests in lib/krb5/krb
* The check-unix-trace target in lib/krb5/os
* The tests in tests/misc
# Fri Oct 23 11:56:29 2015 ghudson@mit.edu (Greg Hudson) - Given to ghudson@mit.edu (Greg Hudson)
# Fri Oct 23 11:56:29 2015 ghudson@mit.edu (Greg Hudson) - Status changed from 'open' to 'review'
# Fri Oct 23 11:56:29 2015 ghudson@mit.edu (Greg Hudson) - Correspondence added
From: ghudson@mit.edu
Subject: git commit
Download (untitled) / with headers
text/plain 371B

Add RUN_TEST make variable

Add a pre.in variable to simplify running C test programs from
Makefile rules.

https://github.com/krb5/krb5/commit/45c6e285be8042ffccdab13b5f78cd5ada8e7973
Author: Greg Hudson <ghudson@mit.edu>
Commit: 45c6e285be8042ffccdab13b5f78cd5ada8e7973
Branch: master
src/config/pre.in | 6 ++++++
1 files changed, 6 insertions(+), 0 deletions(-)
# Fri Oct 23 11:56:30 2015 ghudson@mit.edu (Greg Hudson) - Target_Version 1.14 added
# Fri Oct 23 11:56:30 2015 ghudson@mit.edu (Greg Hudson) - Tags pullup added
# Fri Oct 23 11:56:30 2015 ghudson@mit.edu (Greg Hudson) - Correspondence added
From: ghudson@mit.edu
Subject: git commit
Download (untitled) / with headers
text/plain 1.9KiB

Use RUN_TEST and fix installed krb5.conf uses

Use $(RUN_TEST) to run most C test programs, for simplicity and to fix
accidental uses of the installed krb5.conf. Where a particular
krb5.conf must be used instead of the one in src/config-files, use a
locally defined variant like RUN_TEST_LOCAL_CONF.

Accidental references to the installed krb5.conf were present when
running t_pac, t_princ, t_etypes, t_trace, t_attr, t_attrset,
t_packet, t_remote, t_client, pkinit_kdf_test, test_chpw_message,
text_cxx_krb5, and test_cxx_k5int.

Based on a patch from Robbie Harwood.

https://github.com/krb5/krb5/commit/4eea9c287e43ab40936e25094cd093f2f3f32bd9
Author: Greg Hudson <ghudson@mit.edu>
Commit: 4eea9c287e43ab40936e25094cd093f2f3f32bd9
Branch: master
src/kdc/Makefile.in | 3 +-
src/lib/crypto/builtin/des/Makefile.in | 8 ++--
src/lib/crypto/builtin/sha1/Makefile.in | 4 +-
src/lib/crypto/builtin/sha2/Makefile.in | 2 +-
src/lib/crypto/crypto_tests/Makefile.in | 48 +++++++++++++++---------------
src/lib/crypto/krb/Makefile.in | 2 +-
src/lib/gssapi/generic/Makefile.in | 2 +-
src/lib/kdb/Makefile.in | 2 +-
src/lib/krad/Makefile.in | 13 ++++----
src/lib/krb5/ccache/Makefile.in | 6 +--
src/lib/krb5/keytab/Makefile.in | 3 +-
src/lib/krb5/krb/Makefile.in | 31 +++++++++----------
src/lib/krb5/os/Makefile.in | 18 ++++++-----
src/plugins/preauth/pkinit/Makefile.in | 2 +-
src/tests/asn.1/Makefile.in | 16 ++--------
src/tests/gssapi/Makefile.in | 6 ++--
src/tests/misc/Makefile.in | 14 ++++----
src/tests/resolve/Makefile.in | 6 ++--
src/tests/shlib/Makefile.in | 2 +-
src/tests/threads/Makefile.in | 2 +-
src/util/et/Makefile.in | 4 +-
src/util/profile/Makefile.in | 6 ++--
22 files changed, 94 insertions(+), 106 deletions(-)
# Mon Oct 26 15:39:34 2015 tlyu (Taylor Yu) - Correspondence added
From: tlyu@mit.edu
Subject: git commit
Download (untitled) / with headers
text/plain 477B

Add RUN_TEST make variable

Add a pre.in variable to simplify running C test programs from
Makefile rules.

(cherry picked from commit 45c6e285be8042ffccdab13b5f78cd5ada8e7973)

https://github.com/krb5/krb5/commit/b5b98e1acb3d72f0398a9046d220512936941b5d
Author: Greg Hudson <ghudson@mit.edu>
Committer: Tom Yu <tlyu@mit.edu>
Commit: b5b98e1acb3d72f0398a9046d220512936941b5d
Branch: krb5-1.14
src/config/pre.in | 6 ++++++
1 files changed, 6 insertions(+), 0 deletions(-)
# Mon Oct 26 15:39:35 2015 tlyu (Taylor Yu) - Status changed from 'review' to 'resolved'
# Mon Oct 26 15:39:35 2015 tlyu (Taylor Yu) - Version_Fixed 1.14 added
# Mon Oct 26 15:39:35 2015 tlyu (Taylor Yu) - Correspondence added
From: tlyu@mit.edu
Subject: git commit
Download (untitled) / with headers
text/plain 2KiB

Use RUN_TEST and fix installed krb5.conf uses

Use $(RUN_TEST) to run most C test programs, for simplicity and to fix
accidental uses of the installed krb5.conf. Where a particular
krb5.conf must be used instead of the one in src/config-files, use a
locally defined variant like RUN_TEST_LOCAL_CONF.

Accidental references to the installed krb5.conf were present when
running t_pac, t_princ, t_etypes, t_trace, t_attr, t_attrset,
t_packet, t_remote, t_client, pkinit_kdf_test, test_chpw_message,
text_cxx_krb5, and test_cxx_k5int.

Based on a patch from Robbie Harwood.

(cherry picked from commit 4eea9c287e43ab40936e25094cd093f2f3f32bd9)

https://github.com/krb5/krb5/commit/962a14661fc5b16aea4725f67eff22f9f9179918
Author: Greg Hudson <ghudson@mit.edu>
Committer: Tom Yu <tlyu@mit.edu>
Commit: 962a14661fc5b16aea4725f67eff22f9f9179918
Branch: krb5-1.14
src/kdc/Makefile.in | 3 +-
src/lib/crypto/builtin/des/Makefile.in | 8 ++--
src/lib/crypto/builtin/sha1/Makefile.in | 4 +-
src/lib/crypto/builtin/sha2/Makefile.in | 2 +-
src/lib/crypto/crypto_tests/Makefile.in | 48 +++++++++++++++---------------
src/lib/crypto/krb/Makefile.in | 2 +-
src/lib/gssapi/generic/Makefile.in | 2 +-
src/lib/kdb/Makefile.in | 2 +-
src/lib/krad/Makefile.in | 13 ++++----
src/lib/krb5/ccache/Makefile.in | 6 +--
src/lib/krb5/keytab/Makefile.in | 3 +-
src/lib/krb5/krb/Makefile.in | 31 +++++++++----------
src/lib/krb5/os/Makefile.in | 18 ++++++-----
src/plugins/preauth/pkinit/Makefile.in | 2 +-
src/tests/asn.1/Makefile.in | 16 ++--------
src/tests/gssapi/Makefile.in | 6 ++--
src/tests/misc/Makefile.in | 14 ++++----
src/tests/resolve/Makefile.in | 6 ++--
src/tests/shlib/Makefile.in | 2 +-
src/tests/threads/Makefile.in | 2 +-
src/util/et/Makefile.in | 4 +-
src/util/profile/Makefile.in | 6 ++--
22 files changed, 94 insertions(+), 106 deletions(-)
# Wed Dec 16 18:03:04 2015 tlyu (Taylor Yu) - CustomField pullup deleted