Simon Riggs wrote: > > This simple patch lets someone specifies the xlog directory at initdb > > time. It uses symlinks to do it, and create and/or set permissions at > > the directory as appropriate. > > Looks good. > > I'm no expert on coding style, so I won't comment on anything there. > > Any test harness changes? Or a test script? > Unfortunately, nope. :( I tested it manually. How did i test it?
1) xlog directory doesn't exist -- OK 2) xlog directory exists -- OK 3) xlog directory exists but doesn't have the right permissions -- FAILED 4) xlog diretory exists but is not empty -- FAILED 5) interrupting initdb execution (Ctrl-C) to test the cleanup code -- OK PS> i didn't test it in a win32 box but it should be ok. -- Euler Taveira de Oliveira http://www.timbira.com/ ---------------------------(end of broadcast)--------------------------- TIP 6: explain analyze is your friend