Re: [PATCH] Replacement for OSSP-UUID for Linux and BSD

Поиск
Список
Период
Сортировка
От Matteo Beccati
Тема Re: [PATCH] Replacement for OSSP-UUID for Linux and BSD
Дата
Msg-id 53843FF6.9030803@beccati.com
обсуждение исходный текст
Ответ на Re: [PATCH] Replacement for OSSP-UUID for Linux and BSD  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: [PATCH] Replacement for OSSP-UUID for Linux and BSD  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
Hi Tom,

On 27/05/2014 03:07, Tom Lane wrote:
> I've verified functionality of this patch on these scenarios:
>
> (1) --with-ossp-uuid on RHEL6, using uuid-1.6.1-10.el6.x86_64
> (2) --with-linux-uuid on RHEL6, using libuuid-2.17.2-12.14.el6_5.x86_64
> (3) --with-linux-uuid on OS X 10.9.3, Intel
> (4) --with-linux-uuid on OS X 10.4.11, PPC (hence, bigendian)
>
> I do not have a machine on which to try --with-bsd-uuid, so it's
> possible I broke that portion of Matteo's patch.  Would someone try
> that case on a FreeBSD box?

I've tested on NetBSD i386 and --with-bsd-uuid worked out of the box. I
could fire up some virtual machines with FreeBSD and other BSD flavours,
but maybe some buildfarm animals could be used for that.

I'm attaching a little patch to be applied on top of yours.

I didn't notice that "buf ? 13 : 0" was raising a warning about the
condition being always true on BSD. I guess it's safe to hardcode 13 as
the argument is ignored anyway with ossp, so I've fixed that.

I've also fixed v1mc generation on "linux" to match the OSSP and BSD
variants and added a regression test for it.


Cheers
--
Matteo Beccati

Development & Consulting - http://www.beccati.com/

Вложения

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

Предыдущее
От: ash
Дата:
Сообщение: Re: Re-create dependent views on ALTER TABLE ALTER COLUMN ... TYPE?
Следующее
От: Fujii Masao
Дата:
Сообщение: Re: Spreading full-page writes