Re: generating function default settings from pg_proc.dat

I wrote:
> However, I'm not sure I love it in this form, because what it
> asks developers to do is populate proacl using aclitemin's
> notation, eg
> + proacl => '{postgres=X/postgres,pg_read_all_stats=X/postgres}' },

After sleeping on it, I decided that much of the cognitive dissonance
here comes from the fact that elsewhere in the catalog data files
we represent the bootstrap user's name as POSTGRES, making it more
obvious that that's a placeholder and not the real value. If we do
that here and also drop the redundant grantor specifications, we get

+ proacl => '{POSTGRES=X,pg_read_all_stats=X}' },

which is still not the world's prettiest notation, but it feels
manageable for this purpose.

I tried to write it that way and found that it failed miserably,
because initdb tried to substitute the real superuser's name
for POSTGRES, and we don't want that here (the bootstrap backend
can't possibly know what that is yet). So 0001 attached is a
preliminary patch to tighten initdb so it only replaces complete
tokens bounded by whitespace, rather than any arbitrary substrings
of postgres.bki lines. (Note this depends on 2f248ad57.) Then
0002 is the same patch as before plus this notational adjustment.

regards, tom lane