From a0dfb8222afa87d6fabaa8234da89e2a1b57ea5a Mon Sep 17 00:00:00 2001
From: Ian Jackson <ian@liberator.relativity.greenend.org.uk>
Date: Wed, 29 Jul 2009 21:56:26 +0100
Subject: [PATCH] fixes from testing

---
 yarrg/Commods.pm | 16 ++++++++++++----
 yarrg/convert.c  |  4 ++--
 2 files changed, 14 insertions(+), 6 deletions(-)

diff --git a/yarrg/Commods.pm b/yarrg/Commods.pm
index 67277c2..62c1f5e 100644
--- a/yarrg/Commods.pm
+++ b/yarrg/Commods.pm
@@ -23,6 +23,7 @@
 package Commods;
 use IO::File;
 use HTTP::Request::Common ();
+use POSIX;
 
 use strict;
 use warnings;
@@ -32,7 +33,7 @@ BEGIN {
     our ($VERSION, @ISA, @EXPORT, @EXPORT_OK, %EXPORT_TAGS);
     $VERSION     = 1.00;
     @ISA         = qw(Exporter);
-    @EXPORT      = qw(&parse_info_clientside
+    @EXPORT      = qw(&parse_info_clientside &fetch_with_rsync
 		      &parse_info_serverside &parse_info_serverside_ocean
 		      %oceans %commods %clients %routes %route_mysteries
 		      &parse_pctb_commodmap %pctb_commodmap @pctb_commodmap
@@ -144,7 +145,12 @@ sub parse_info_clientside () {
     return unless $yarrg;
     my $master= fetch_with_rsync('info');
     parse_info1($master,'s');
-    parse_info1('_local-info.txt','s');
+    my $local= '_local-info.txt';
+    if (stat $local) {
+	parse_info1($local,'s');
+    } else {
+	die "$local $!" unless $! == &ENOENT;
+    }
 }
 
 sub fetch_with_rsync ($) {
@@ -160,6 +166,7 @@ sub fetch_with_rsync ($) {
 	$!=0; system 'rsync','-Lt','--',$remote,$local;
 	die "$? $!" if $! or $?;
     }
+    return $local;
 }
 
 sub parse_info_maproutes ($$$) {
@@ -256,7 +263,8 @@ sub yarrgpostform ($$) {
     my ($ua, $form) = @_;
     my $dest= $ENV{'YPPSC_YARRG_YARRG'};
     get_our_version($form, 'client');
-    return cgipostform($ua, "$dest/commod-update-receiver", $form);
+    die unless $dest =~ m,/$,;
+    return cgipostform($ua, "${dest}commod-update-receiver", $form);
 }    
 
 sub cgipostform ($$$) {
@@ -290,7 +298,7 @@ sub cgipostform ($$$) {
 	$out =~ m,^Content-Type: text/plain.*\n\n, or die "$out ?";
 	return $';
     } else {
-	my $resp= $ua->request($url,$req);
+	my $resp= $ua->request($req);
 	die $resp->status_line unless $resp->is_success;
 	return $resp->content();
     }
diff --git a/yarrg/convert.c b/yarrg/convert.c
index 49fc3ed..3c9ae9c 100644
--- a/yarrg/convert.c
+++ b/yarrg/convert.c
@@ -336,8 +336,8 @@ int main(int argc, char **argv) {
 	     o_serv_pctb,        o_flags & ff_use_pctb);
 	     
   set_server("YPPSC_YARRG_DICT_UPDATE",
-	     "rsync://",         "rsync.yarrg.chiark.net/pctb",
-	                         "rsync.yarrg.chiark.net/pctb/test",
+	     "rsync://",         "rsync.yarrg.chiark.net/yarrg",
+	                         "rsync.yarrg.chiark.net/yarrg/test",
 	     o_serv_dict_fetch,   o_flags & ff_dict_fetch);
 
   set_server("YPPSC_YARRG_DICT_SUBMIT",
-- 
2.30.2