I wrote:
> Hmm, changing it like that seems likely to have a lot of unexpected
> side-effects. What I had in mind was to change the
> if (te->nDeps != 1 ||
> TocIDRequired(AH, te->dependencies[0]) == 0)
> return 0;
> bit so that it would loop through the TE's dependencies to see
> if any one of them is a required non-ACL TE. We have to relax
> the "te->nDeps != 1" restriction, but we still want to check
> that there is a parent object that is being restored.
I improved the patch along those lines and pushed it.
regards, tom lane