Quiet clang scan

Strip -Wall when returned from python-config

Fix integer widths in rlm_python.c

Helps to regenerate the configure script

lineno isn't a pointer...

Yet more flag stripping *sigh*

make cf_expand_variables() take lineno instead of *lineno

which was done for who knows what historical reasons

make fr_cond_tokenize() take CONF_SECTION we're parsing

which ends up being easier than the alternative.  That way
instead of passing a bunch of things around, we can just pass
a CONF_SECTION

ensure that we always pass a CONF_SECTION

More filtering of ldflags

More python-config stupidity

-Werror args probably don't contain spaces, but they do contain hyphens

Deal with -Werror where it has args

make process_map() call cf_get_token()

which regularizes the parsing of the various things

we only need 4 buffers

rearrange the user of the buffers, and document what they're for.

simplify process_include()

so that it no longer uses the old APIs

allow unlang expansions in name1 for conf section/pairs

which can occasionally be useful.  And, is preparation for
removing the rest of the use of gettoken()

use 'this' section, not the top-level one

expand variables in cf_get_token

errors are errors

s/Ssize_t/Size_t/ in rlm_python

Use PyLong_FromSsize_t where appropriate

Switch numeric interchange to PyLong, but use PyNumber to check for compatibility with Python v2 and v3

Just use python-config to configure the python module

Python 3 compatibility fixes

Revert "We don't need to clear repeatable anymore it's done by frame_cleanup"

This reverts commit bbc8a8bfe272095cc186d1313b98438d19c65d7b.

fix nested %{...}

allow bare %{...} in conditions

which for now are parsed and printed as double-quoted strings.

But it makes some things much easier

decrement before checks

do less memcpy

make cf_expand_variables() take a length of input

which makies other things simpler, such as the conditional
tokenizer

cleanups and create cf_get_token()

which wraps tmpl_preparse(), and eventually cf_expand_variables()
so that the config file parser *doesn't* use gettoken().

Instead, it should use the new APIs which provide for better
error reporting and more functionality

Add test for return after policy

Policy sets return_point on that stack frame, so we need to make sure it's cleared when the next instruction is executed

We don't need to clear repeatable anymore it's done by frame_cleanup

Clear break_point and return_point when moving to the next instruction or popping things

We really do only need to set top_frame when pushing new things onto the stack

Print unwind at the stack level

Set dependencies correctly for return keyword tests

less code == better code

typo

Fix missing free() in $INCLUDE

fix list for proper nesting

text was already in the partial include file

typo

use ":" instead of "::" everywhere

should be fixed-width

this file was moved

document changed behavior

use standard macro

unescape bare %{...} too

doesn't seem to make any difference, even though it is sometimes set

make some functions static

they are old, and now only used in one place

use tmpl_preparse(), which now does most necessary things

tmpl_preparse() is now capable of parsing %{...}

Create an initial, empty CRL in DER encoding #2964

be a bit more flexible about 'map'

map { ...} is OK in sigtran as a config section

allow , and ; after words

less code is better code

simplify the parser.

it's clearer to just check for characters directly, instead of
calling functions and then checking for token values

move "expand variables" into add_pair() function

move add_pair() to separate function

and start cleaning up the parser to be simpler and more understandable

remove WITH_CONF_WRITE

unused, untested, likely not to work.  If we re-do the parser,
we can make this functionality part of the requirements.

Until then, this code is making refactoring mode difficult

ue correct CA password when not the default "whatever"

without passin and passout, the CA passphrase is taken from ca.cnf where it is "whatever". In passwords.mk it is also "whatever" so that is usually okay.
However when changing the password in passwords.mk, the CA is still generated with "whatever" but subsequent cert generation commands use the custom password -> failure.

Port unit changes from v3.0.x

More unwind tests

Fix multiple issues in return/unwind

Convert the macros to inline functions

Better type warnings, and slightly cleaner use of "unwind_to_*"

jenkins: The 'QA_RPATHS' is already in 'make rpm' target

Fix missing free() in 'decode-tacacs' decode

Consistent case

I guess we just can't use an enum for flag fields anymore...

Quiet the compiler?

Different way of managing where to unwind to

Parent the VPs from the frame state for easy cleanup

this possibly wasn't there before, because there used to be a union in the frame to hold instruction specific data

Switch to using a flags field, and set flags in the correct places, i.e. within the foreach and policy keyword functions

Fix stupid hacks in foreach code

add test for `foo = #`, which shouldn't be allowed

move "map" processing to its own function

move "if / elsif" handling to its own function

try to have most goto's going forward

move $TEMPLATE handling to a separate function

move $INCLUDE handling into its own function

remove unused assignment

clean up and clarify parser

foo = # stuff

is now a parse error.  Previously it did who knows what. :(

and two "switch" statements have been changed to one, with
a subsequent "expand variables" stage.

use "op_token" for clarity

this doesn't change the code, but it makes it easier to read

rename t1, t2, t3 to something more reasonable

because 20 years of t1 t2 and t3 are hurting my brain

Move unwind to be part of the stack, not part of a frame

use condition as talloc_ctx

to avoid leaking memory.  It's leaked only in the tests,
but whatever.

empty strings are OK

with tests

allow $ENV{...} too

filter out sha2 at the make stage

because even if we have `if (0) { ... }` in the unlang policies,
the inner policies are still getting evaluated.

This fix is the right one, but we still need to add tests for
*not* compiling the contents of `if (0)`

nope.  that didn't work

typo

show OpenSSL libs for travis

password_process_sha3() Fix the talloc_list_free() call (#2956)

It expects reference instead of value.

Extra check for returning inside subrequests

nope... try more debugging

Debian packaging: Add builddep on asciidoctor and pandoc for man pages

Minimal fix only for the moment...

Add builddep on asciidoc and pandoc so that we can build the man pages
that were previously source files.

travis is broken

It's been chopping off debug output for months. See if this hack
from https://github.com/travis-ci/travis-ci/issues/8920 helps

Revert "Add openssl-perl as a BuildRequires in the Redhat spec file"

This reverts commit d7cdb717fc336f68ed8faea7278468c4881c4eae.

revert this for now as docker isn't picking up the package.
Which then lets us see / fix other potential build failures

fix patch for new docs