stephen.hindmarch at bt.com stephen.hindmarch at bt.com
Thu Nov 1 04:30:13 PDT 2007
I've been using configure-replication.sh to make my life easier
preparing slony installation script and I've fixed up a couple of bugs
along the way. I have been using version 1.2.6 but have just checked CVS
and the bugs are still in trunk.
 
Both bugs are to do with missing new lines, one in the store_paths
function, the other in the code that generates "add sequence" lines.

I have create a patch file which is here.

===BEGIN===
--- slony1-1.2.6/tools/configure-replication.sh	2006-12-14
22:31:19.000000000 +0000
+++ slony1-patch/tools/configure-replication.sh	2007-11-01
10:57:42.000000000 +0000
@@ -55,7 +55,8 @@
           eval buser=\$USER${j}
           eval bport=\$PORT${j}
           if [ -n "${bdb}" -a "${bhost}" -a "${buser}" -a "${bport}" ];
then
-            echo "STORE PATH (SERVER=${i}, CLIENT=${j},
CONNINFO='dbname=${db} host=${host} user=${user} port=${port}');" >>
$mktmp/store_paths.slonik          else
+            echo "STORE PATH (SERVER=${i}, CLIENT=${j},
CONNINFO='dbname=${db} host=${host} user=${user} port=${port}');" >>
$mktmp/store_paths.slonik
+          else
             echo "STORE PATH (SERVER=${i}, CLIENT=${j},
CONNINFO='dbname=${db} host=${host} user=${user} port=${port}');" >>
$mktmp/store_paths.slonik
           fi
         fi
@@ -139,7 +140,6 @@
 echo "create set (id=1, origin=1, comment='${CLUSTER} Tables and
Sequences');" >> $SETUPSET
 
 tnum=1
-
 for table in `echo $TABLES`; do
     echo "set add table (id=${tnum}, set id=1, origin=1, fully
qualified name='${table}', comment='${CLUSTER} table ${table}');" >>
$SETUPSET
     tnum=`expr ${tnum} + 1`
@@ -147,7 +147,8 @@
 
 snum=1
 for seq in `echo $SEQUENCES`; do
-    echo "set add sequence (id=${snum}, set id=1, origin=1, fully
qualified name='${seq}', comment='${CLUSTER} sequence ${seq}');" >>
$SETUPSET    snum=`expr ${snum} + 1`
+    echo "set add sequence (id=${snum}, set id=1, origin=1, fully
qualified name='${seq}', comment='${CLUSTER} sequence ${seq}');" >>
$SETUPSET
+    snum=`expr ${snum} + 1`
 done
 
 NODEINIT=$mktmp/create_nodes.slonik
===END===

configure-replication.sh is a great script BTW. Speeds up installation
of test environments and makes sure each one is reproducable.

Steve Hindmarch
BT Design - iBridge Development




More information about the Slony1-general mailing list