From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Christoph Berg <myon(at)debian(dot)org> |
Cc: | pgAdmin support <pgadmin-support(at)postgresql(dot)org>, Neel Patel <neel(dot)patel(at)enterprisedb(dot)com> |
Subject: | Re: [pgagent] patch: link with -pthread |
Date: | 2018-07-20 09:06:45 |
Message-ID: | CA+OCxow3hxd=gmMF5F+86QyJgn188nAcG-jfxTF9cW5+1txSgQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-support |
Hi
On Fri, Jul 20, 2018 at 9:55 AM, Christoph Berg <myon(at)debian(dot)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
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2018-07-20 09:12:06 | Re: [pgagent] Instatallation destination? |
Previous Message | Christoph Berg | 2018-07-20 09:00:16 | [pgagent] Instatallation destination? |