Обсуждение: patch for src/backends/catalog/genbki.sh.in

Поиск
Список
Период
Сортировка

patch for src/backends/catalog/genbki.sh.in

От
Achim Gottinger
Дата:
Hi,

I attached a patch to that mail that is required if you use sed 3.02.80
instead of 3.08.
Sed now treats \n as an escape sequence. Old sed ignored the \ so this
patch removes the \.
This issue occures in postgres-7.0.3 and 7.1-beta3 and leads to a
segfault or error in initdb because
global1.bki.source and local1_template1.bki.source are generated wrong (
ame instead of name ).

Bye Achim
--- genbki.sh.orig    Fri Jan 26 18:09:57 2001
+++ genbki.sh.in    Fri Jan 26 18:09:57 2001
@@ -86,7 +86,7 @@
     -e "s/[     ]Oid/\ oid/g" \
     -e "s/[     ]NameData/\ name/g" \
     -e "s/^Oid/oid/g" \
-    -e "s/^NameData/\name/g" \
+    -e "s/^NameData/name/g" \
     -e "s/(NameData/(name/g" \
     -e "s/(Oid/(oid/g" \
     -e "s/NAMEDATALEN/$NAMEDATALEN/g" \

Re: patch for src/backends/catalog/genbki.sh.in

От
Bruce Momjian
Дата:
We discovered this a few weeks ago and fixed it.  You will not see it in
beta4.  Thanks.


> Hi,
>
> I attached a patch to that mail that is required if you use sed 3.02.80
> instead of 3.08.
> Sed now treats \n as an escape sequence. Old sed ignored the \ so this
> patch removes the \.
> This issue occures in postgres-7.0.3 and 7.1-beta3 and leads to a
> segfault or error in initdb because
> global1.bki.source and local1_template1.bki.source are generated wrong (
> ame instead of name ).
>
> Bye Achim

> --- genbki.sh.orig    Fri Jan 26 18:09:57 2001
> +++ genbki.sh.in    Fri Jan 26 18:09:57 2001
> @@ -86,7 +86,7 @@
>      -e "s/[     ]Oid/\ oid/g" \
>      -e "s/[     ]NameData/\ name/g" \
>      -e "s/^Oid/oid/g" \
> -    -e "s/^NameData/\name/g" \
> +    -e "s/^NameData/name/g" \
>      -e "s/(NameData/(name/g" \
>      -e "s/(Oid/(oid/g" \
>      -e "s/NAMEDATALEN/$NAMEDATALEN/g" \


--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026