X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~yarrgweb/git?p=ypp-sc-tools.db-live.git;a=blobdiff_plain;f=yarrg%2FCommods.pm;h=860510e33835e5e1e574ab429862f0563ecf7c2a;hp=13f581241fa1bd770e166e7e1cf4e581e36c473c;hb=70f23c42b325a1463fe8a9d76cb967b7df1260df;hpb=4e2f1b2a9105cd228da52e2e8a173cf6eb4bcb71

diff --git a/yarrg/Commods.pm b/yarrg/Commods.pm
index 13f5812..860510e 100644
--- a/yarrg/Commods.pm
+++ b/yarrg/Commods.pm
@@ -42,12 +42,14 @@ BEGIN {
 		      &pipethrough_run_along &pipethrough_run_finish
 		      &pipethrough_run_gzip
 		      &cgipostform &yarrgpostform &cgi_get_caller
-		      &set_ctype_utf8);
+		      &set_ctype_utf8 $masterinfoversion);
     %EXPORT_TAGS = ( );
 
     @EXPORT_OK   = qw();
 }
 
+our $masterinfoversion= 2; # version we understand
+
 our %oceans; # eg $oceans{'Midnight'}{'Ruby'}{'Eta Island'}= $sources;
 our %clients; # eg $clients{'ypp-sc-tools'}= [ qw(last-page) ];
 our %routes; # eg $routes{'Midnight'}{'Orca'}{'Tinga'}= $sources  NB abbrevs!
@@ -63,7 +65,11 @@ our %commods;
 our (%pctb_commodmap,@pctb_commodmap);
 
 my %colours; # eg $colours{'c'}{'black'}= $sources
-my @rawcm; # eg $rawcm[0]='fine rum'; $rawcm[1]='fine %c cloth'
+my (@rawcm, @nocm); # eg $rawcm[0]='fine rum'; $rawcm[1]='fine %c cloth'
+
+# IMPORTANT
+#  when extending the format of source-info in a non-backward
+#  compatible way, be sure to update update-master-info too.
 
 sub parse_info1 ($$) {
     my ($mmfn,$src)= @_;
@@ -78,12 +84,16 @@ sub parse_info1 ($$) {
 	    @ctx= (sub { $colours{$colourkind}{lc $_} .= $src; });
 	} elsif (m/^commods$/) {
 	    @ctx= (sub { push @rawcm, lc $_; });
+	} elsif (m/^nocommods$/) {
+	    @ctx= (sub { push @nocm, lc $_; });
 	} elsif (m/^ocean (\w+)$/) {
 	    my $ocean= $1;
+	    keys %{ $oceans{$ocean} };
 	    @ctx= (sub {
 		$ocean or die; # ref to $ocean needed to work
 		               # around a perl bug
 		my $arch= $_;
+		keys %{ $oceans{$ocean}{$arch} };
 		$ctx[1]= sub {
 		    $oceans{$ocean}{$arch}{$_} .= $src;
 		};
@@ -122,13 +132,14 @@ sub parse_info1 ($$) {
 #print "ca($s)\n";
 	if ($s !~ m/\%(\w+)/) {
 	    my ($name, $props) = $s =~
-		/^(\S[^\t]*\S)\t+(\S[^\t]*\S)$/
+		/^(\S[^\t]*\S)(?:\t+(\S[^\t]*\S))?$/
 		or die "bad commodspec $s";
+	    return if grep { $name eq $_ } @nocm;
 	    my $ucname= ucfirst $name;
 	    $commods{$ucname}{Srcs} .= $ss;
 	    my $c= $commods{$ucname};
 	    $c->{Volume}= 1000;
-	    foreach my $prop (split /\s+/, $props) {
+	    foreach my $prop (defined $props ? split /\s+/, $props : ()) {
 		if ($prop =~ m/^([1-9]\d*)(k?)g$/) {
 		    $c->{Mass}= $1 * ($2 ? 1000 : 1);
 		} elsif ($prop =~m/^([1-9]\d*)l$/) {
@@ -137,7 +148,6 @@ sub parse_info1 ($$) {
 		    die "unknown property $prop for $ucname";
 		}
 	    }
-	    die "no mass for $ucname" unless defined $c->{Mass};
 	    return;
 	}
 	die "unknown $&" unless defined $colours{$1};
@@ -168,7 +178,7 @@ sub parse_info1 ($$) {
 sub parse_info_clientside () {
     my $yarrg= $ENV{'YPPSC_YARRG_DICT_UPDATE'};
     return unless $yarrg;
-    my $master= fetch_with_rsync('info');
+    my $master= fetch_with_rsync("info-$masterinfoversion");
     parse_info1($master,'s');
     my $local= '_local-info.txt';
     if (stat $local) {
@@ -210,12 +220,12 @@ sub parse_info_maproutes ($$$) {
 }
 
 sub parse_info_serverside () {
-    parse_info1('master-info.txt','s');
+    parse_info1('source-info.txt','s');
 }
 sub parse_info_serverside_ocean ($) {
     my ($oceanname) = @_;
     die "unknown ocean $oceanname ?" unless exists $oceans{$oceanname};
-    parse_info1("ocean-".(lc $oceanname).".txt",'s');
+    parse_info1("_ocean-".(lc $oceanname).".txt",'s');
 }
 
 sub parse_pctb_commodmap () {
@@ -239,7 +249,7 @@ sub parse_pctb_commodmap () {
 sub get_our_version ($$) {
     my ($aref,$prefix) = @_;
     $aref->{"${prefix}name"}= 'ypp-sc-tools yarrg';
-    $aref->{"${prefix}fixes"}= 'lastpage';
+    $aref->{"${prefix}fixes"}= 'lastpage checkpager';
 
     my $version= `git-describe --tags HEAD || echo 0unknown`; $? and die $?;
     chomp($version);
@@ -257,6 +267,7 @@ sub pipethrough_prep () {
 
 sub pipethrough_run_along ($$$@) {
     my ($tf, $childprep, $cmd, @a) = @_;
+    $tf->error and die $!;
     $tf->flush or die $!;
     $tf->seek(0,0) or die $!;
     my $fh= new IO::File;