Re: Issue on compiling pgAdmin trunk

Поиск
Список
Период
Сортировка
От Guillaume Lelarge
Тема Re: Issue on compiling pgAdmin trunk
Дата
Msg-id 46AB7D3A.2090101@lelarge.info
обсуждение исходный текст
Ответ на Re: Issue on compiling pgAdmin trunk  (Dave Page <dpage@postgresql.org>)
Список pgadmin-hackers
Dave Page a écrit :
> Guillaume Lelarge wrote:
>> Hi,
>>
>> I had an issue when I tried to compile pgAdmin today. I did a complete
>> svn update (I mean I first did rm -rf on my directory, then svn update,
>> then ./configure, then make). gcc is complaining about duplicate use of
>> DebuggerApiVersion :
>>
>> ../pgadmin/include/debugger/dbgPgConn.h:67: error: declaration of
>> 'DebuggerApiVersion dbgPgConn::DebuggerApiVersion()'
>> ../pgadmin/include/debugger/dbgPgConn.h:37: error: changes meaning of
>> 'DebuggerApiVersion' from 'typedef enum DebuggerApiVersion
>> DebuggerApiVersion'
>>
>> So I changed the Enum's name to DebuggerApiVersion_Enum and everything
>> compiles now. Nevertheless, I'm not sure I did right, even if it
>> compiles fine. So, here is my patch.
>>
>> Comments are more than welcome :)
>
> Yeah, I spotted that in the auto-build last night. The fix seems sound,
> but I'm not keen on the name. Perhaps just DebuggerApiVersions (note the
> s)?
>

You're right. DebuggerApiVersions is a much better name.

> Anyhoo, please commit if you have a minute.
>

Done.

Thanks.


--
Guillaume.
<!-- http://abs.traduc.org/
     http://lfs.traduc.org/
     http://docs.postgresqlfr.org/ -->

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

Предыдущее
От: svn@pgadmin.org
Дата:
Сообщение: SVN Commit by guillaume: r6508 - in trunk/pgadmin3/pgadmin: debugger include/debugger
Следующее
От: Guillaume Lelarge
Дата:
Сообщение: Re: Beta 2 Wednesday?