Re: [PATCH] Pull general SASL framework out of SCRAM

Поиск
Список
Период
Сортировка
От Jacob Champion
Тема Re: [PATCH] Pull general SASL framework out of SCRAM
Дата
Msg-id bf4c861d6331af782af1ec1639aaa00f18703504.camel@vmware.com
обсуждение исходный текст
Ответ на Re: [PATCH] Pull general SASL framework out of SCRAM  (Michael Paquier <michael@paquier.xyz>)
Ответы Re: [PATCH] Pull general SASL framework out of SCRAM
Re: [PATCH] Pull general SASL framework out of SCRAM
Список pgsql-hackers
On Tue, 2021-07-13 at 19:31 +0900, Michael Paquier wrote:
> On Tue, Jul 13, 2021 at 12:41:27PM +0300, Mikhail Kulagin wrote:
> > I got an error while building one of the extensions.
> > /home/mkulagin/pg-install/postgresql-master/include/internal/libpq-int.h:44:10: fatal error: fe-auth-sasl.h: No
suchfile or directory
 
> >                   #include "fe-auth-sasl.h"
> >                   ^~~~~~~~~~~~~~~~            
> 
> Right.  I overlooked the fact that libpq-int.h is installed.

Thanks for catching that Mikhail.

> > I think the new fe-auth-sasl.h file should be installed too.
> > Correction proposal in the attached file (but I'm not sure that fix
> > of Install.pm is correct). 
> 
> That looks correct to me.  I'll check that tomorrow.

Looks right to me too. I'm currently rebuilding my Windows dev
environment so I haven't been able to double-check that piece of it.

Just to make sure -- do we want to export the fe-auth-sasl.h header as
opposed to forward-declaring the pg_fe_sasl_mech struct? Is the use
case for libpq-int.h just "here, have at the internals, and if you
break it then it's on you"?

--Jacob

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Stephen Frost
Дата:
Сообщение: Re: proposal: possibility to read dumped table's name from file
Следующее
От: Alvaro Herrera
Дата:
Сообщение: Re: proposal: possibility to read dumped table's name from file