Re: Improving and extending int128.h to more of numeric.c - Mailing list pgsql-hackers

From Andres Freund
Subject Re: Improving and extending int128.h to more of numeric.c
Date
Msg-id c4x5wnpzjnyfobvx5lvlykeaukejkeu6743vxapjqbrvfjphjp@akjb56suyocl
Whole thread Raw
In response to Re: Improving and extending int128.h to more of numeric.c  (Dean Rasheed <dean.a.rasheed@gmail.com>)
Responses Re: Improving and extending int128.h to more of numeric.c
List pgsql-hackers
Hi,

On 2025-07-09 10:38:31 +0100, Dean Rasheed wrote:
> On Wed, 9 Jul 2025 at 07:41, John Naylor <johncnaylorls@gmail.com> wrote:
> >
> > Hi Dean, I went to take a look at this and got stuck at building the
> > test file. The usual pointing gcc to the src and build include
> > directories didn't cut it. How did you get it to work?
> >
>
> Yes, it wasn't immediately obvious how to do it. I first built
> postgres as normal, including the  pg_config tool, and then used that
> to compile the test as follows:
>
> gcc -O3 -g \
>     src/tools/testint128.c \
>     -I$(pg_config --includedir-server) \
>     -o src/tools/testint128 \
>     $(pg_config --libs)
>
> It actually only needs -lpgcommon -lpgport -lm, but it seemed easier
> just to include all of the pg_config --libs.

I think we should wire this up to the buildsystem and our testsuite...  Having
testcode that is not run automatically may be helpful while originally
developing something, but it doesn't do anything to detect portability issues
or regressions.

Greetings,

Andres Freund



pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Re: Adding basic NUMA awareness
Next
From: Jeff Davis
Date:
Subject: Windows question: when is LC_MESSAGES defined?