Re: [pgagent] patch: link with -pthread

Поиск
Список
Период
Сортировка
От Dave Page
Тема Re: [pgagent] patch: link with -pthread
Дата
Msg-id CA+OCxow3hxd=gmMF5F+86QyJgn188nAcG-jfxTF9cW5+1txSgQ@mail.gmail.com
обсуждение исходный текст
Ответ на [pgagent] patch: link with -pthread  (Christoph Berg <myon@debian.org>)
Ответы Re: [pgagent] patch: link with -pthread  (Neel Patel <neel.patel@enterprisedb.com>)
Список pgadmin-support
Дерево обсуждения
Hi

On Fri, Jul 20, 2018 at 9:55 AM, Christoph Berg <myon@debian.org> wrote:
All non-x868 architectures need -pthread to link with threads:

https://buildd.debian.org/status/logs.php?pkg=pgagent&ver=4.0.0-1&suite=sid

This patch fixes the problem:

--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -135,7 +135,7 @@ ENDIF(WIN32)

 ADD_EXECUTABLE(pgagent ${_srcs})
 TARGET_LINK_LIBRARIES(
-        pgagent ${PG_LIBRARIES} ${Boost_LIBRARIES}
+        pgagent ${PG_LIBRARIES} ${Boost_LIBRARIES} -pthread
 )

 # Installation

That throws a nice warning on macOS (unused-command-line-argument) as I suspected it might. I'll bet Windows barfs on it too.

Neel, can you look and see what is the best solution here please? I'm guessing you still have build environments for all platforms from the Boost work you did.

Thanks.

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

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

Предыдущее
От: Christoph Berg
Дата:
Сообщение: [pgagent] Instatallation destination?
Следующее
От: Dave Page
Дата:
Сообщение: Re: [pgagent] Instatallation destination?