Re: [PATCHES] ISBN/ISSN/ISMN/EAN13 module

Поиск
Список
Период
Сортировка
От Jeremy Kronuz
Тема Re: [PATCHES] ISBN/ISSN/ISMN/EAN13 module
Дата
Msg-id BAY107-F164BD56B7816AF30DE6A05D8350@phx.gbl
обсуждение исходный текст
Ответы Re: [PATCHES] ISBN/ISSN/ISMN/EAN13 module  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
> > ... Tom, were you able to apply the patch without problems
> > ('cause this is one of the first times I post a patch in a mailing
>list.)
>
>Um, no, doesn't work at all unfortunately:
>
>$ patch --dry <~/ean.patch
>patching file isn.c
>patch: **** malformed patch at line 6: enum isn_type { INVALID, ANY, EAN13,
>ISBN, ISMN, ISSN, UPC };
>
>$
>
>It looks like the patch got whitespace-mangled in transit.  With many
>mail programs it works better to add a patch as an attachment than
>to try to include it directly in the message text.  Also, we usually
>ask for diff -c rather than diff -u format ... I for one find it easier
>to read.
>
>Please sync the text-conversion issues and resubmit.
>

Tom, I'm attaching the new patch now... it's now using the GET_TEXT(), I've
updated the README and I've used the -c format as you suggested.
Please, let me know how it goes for you.

Regards,
Kronuz.


Вложения

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

Предыдущее
От: "Joshua D. Drake"
Дата:
Сообщение: Re: contrib uninstall scripts need some love
Следующее
От: Martijn van Oosterhout
Дата:
Сообщение: Re: Fixed length data types issue