On 03/20/2017 02:32 AM, Peter Eisentraut wrote:
On 3/17/17 05:37, Heikki Linnakangas wrote:
Add TAP tests for password-based authentication methods.

Tests all combinations of users with MD5, plaintext and SCRAM verifiers
stored in pg_authid, with plain 'password', 'md5' and 'scram'
authentication methods.

This is missing an entry for tmp_check/ in .gitignore.  But maybe we can
do that globally instead of repeating it in every directory?

Hmm. That would be nice. However, many of the test suites have other files and directories in their .gitignores, so handling tmp_check globally wouldn't move the needle much. And handling some generated subdirectories globally and others locally seems confusing. For example, src/test/regress/.gitignore currently looks like this:

# Local binaries

# Generated subdirectories

# Note: regreesion.* are only left behind on a failure; that's why they're not 

Not having /tmp_check/ in the above would look like an omission, when /results/ and /log/ are still listed.

If we could also handle results and log globally, that would be nice. But IMHO those names are too generic to put to a global .gitignore. We could rename them, but this starts to feel like more trouble than it's worth. I'll just go and create a .gitignore for /tmp_check/ to src/test/authentication.

- Heikki

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

Reply via email to