Re: PFA: Patch to generate universal(arm64, x86_64) binaries for macOS

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Semab Tariq <semab(dot)tariq(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: PFA: Patch to generate universal(arm64, x86_64) binaries for macOS
Date: 2021-08-02 08:11:50
Message-ID: CA+OCxow3Ki8ZqhcVV0M+CewgX33sBg_kdv204AqPBAoZDnZjCQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Thanks, patch applied.

On Mon, Aug 2, 2021 at 2:44 AM Semab Tariq <semab(dot)tariq(at)enterprisedb(dot)com>
wrote:

> Hi Team
>
> In my previous email, I attached a link to the JIRA ticket and I was
> hoping that you can access that ticket and see the details about my patch
> but I was unaware that JIRA is not accessible to the community so now I am
> attaching the details here
>
> *Details about my patch*
>
> While trying to generate *pgAgent-4.2.1* universal binaries on *macOS
> BigSur x86_64* architecture via following command
> 1BOOST_ROOT=/tmp/PG-pgAgent PGDIR=/tmp/PG-pgAgent /Applications/CMake.app/
> Contents/bin/cmake -DCMAKE_INSTALL_PREFIX=/tmp/PG-pgAgent -
> DCMAKE_OSX_ARCHITECTURES:STRING="x86_64;arm64" -
> DCMAKE_OSX_DEPLOYMENT_TARGET:STRING=10.14 -DSTATIC_BUILD=NO -D
> CMAKE_OSX_SYSROOT:FILEPATH=/Library/Developer/CommandLineTools/SDKs/
> MacOSX11.1.sdk CMakeLists.txt
>
> I got this error in configure log
> 1CMake Error at CMakeLists.txt:43 (IF): 2 if given arguments: 3 "NOT"
> "x86_64" "arm64" "STREQUAL" "" 4 Unknown arguments specified 5--
> Configuring incomplete, errors occurred!
>
> After some debugging, I found out that the
> *pgAgent-4.2.1-Source/CMakeLists.txt* file doesn't have proper double
> quotes around *CMAKE_OSX_ARCHITECTURES* variable inside *if* condition so
> I added that, and now I am able to build pgAgent universal binaries
> successfully. If you don't have any concerns with my patch can someone
> please push the changes to the source? Thanks & Regards,
> Semab
>
> On Fri, Jul 30, 2021 at 3:40 PM Semab Tariq <semab(dot)tariq(at)enterprisedb(dot)com>
> wrote:
>
>> Hi Team
>>
>> Can someone please push this patch to the *master* branch of *pgAgent*
>> repo?
>> For more details Please refer to JIRA PPS-310
>> <https://enterprisedb.atlassian.net/browse/PPS-310>
>>
>> --
>> Thanks & Regards,
>> Semab
>>
>
>
> --
>

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

EDB: https://www.enterprisedb.com

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Semab Tariq 2021-08-02 08:27:17 Re: PFA: Patch to generate universal(arm64, x86_64) binaries for macOS
Previous Message Dave Page 2021-08-02 08:11:40 pgAgent commit: Properly quote CMAKE_OSX_ARCHITECTURES.