]> git.donarmstrong.com Git - debbugs.git/blobdiff - cgi/version.cgi
Standardize on using <br> (as opposed to <br/>)
[debbugs.git] / cgi / version.cgi
index 338075f985cea9e87c73d08c80ca0b2e08f6e7dc..e3df1fc1cd2decd453e54c67681a91d07f6b61f2 100755 (executable)
@@ -5,7 +5,7 @@ use strict;
 
 # Hack to work on merkel where suexec is in place
 BEGIN{
-     if ($ENV{HTTP_HOST} eq 'merkel.debian.org') {
+     if (defined $ENV{HTTP_HOST} and $ENV{HTTP_HOST} eq 'merkel.debian.org') {
          unshift @INC, qw(/home/don/perl/usr/share/perl5 /home/don/perl/usr/lib/perl5 /home/don/source);
          $ENV{DEBBUGS_CONFIG_FILE}="/home/don/config_internal";
      }
@@ -14,13 +14,17 @@ BEGIN{
 
 use CGI::Simple;
 
-use CGI::Alert 'don@donarmstrong.com';
+# by default send this message nowhere
+# use CGI::Alert q(nobody@example.com);
 
 use Debbugs::Config qw(:config);
-use Debbugs::CGI qw(htmlize_packagelinks html_escape cgi_parameters);
+
+our $VERSION=1;
+
+use Debbugs::CGI qw(htmlize_packagelinks html_escape cgi_parameters munge_url :cache);
 use Debbugs::Versions;
 use Debbugs::Versions::Dpkg;
-use Debbugs::Packages qw(getversions makesourceversions);
+use Debbugs::Packages qw(get_versions makesourceversions);
 use HTML::Entities qw(encode_entities);
 use File::Temp qw(tempdir);
 use IO::File;
@@ -31,45 +35,138 @@ my %img_types = (svg => 'image/svg+xml',
                 png => 'image/png',
                );
 
-my $q = new CGI::Simple;
+my $q = CGI::Simple->new();
 
 my %cgi_var = cgi_parameters(query   => $q,
-                            single  => [qw(package format ignore_boring)],
-                            default => {package       => 'xterm',
+                            single  => [qw(package format ignore_boring width height collapse info)],
+                            default => {package       => 'spamass-milter',
                                         found         => [],
                                         fixed         => [],
                                         ignore_boring => 1,
+                                        collapse      => 0,
                                         format        => 'png',
+                                        width         => undef,
+                                        height        => undef,
+                                        info          => 0,
                                        },
                            );
+my $this = munge_url('version.cgi?',
+                    %cgi_var,
+                   );
+
+if (defined $cgi_var{width}) {
+     $cgi_var{width} =~ /(\d+)/;
+     $cgi_var{width} = $1;
+}
+if (defined $cgi_var{height}) {
+     $cgi_var{height} =~ /(\d+)/;
+     $cgi_var{height} = $1;
+}
+
+if (defined $cgi_var{format}) {
+     $cgi_var{format} =~ /(png|svg|jpg|gif)/;
+     $cgi_var{format} = $1 || 'png';
+}
+else {
+     $cgi_var{format} = 'png';
+}
+
+my $etag;
+if ($cgi_var{info} and not defined $cgi_var{dot}) {
+    $etag = etag_does_not_match(cgi=>$q,
+                               additional_data=>[grep {defined $_ ? $_ : ()}
+                                                 @cgi_var{(qw(package ignore_boring),
+                                                           qw(collapse))
+                                                      },
+                                                 $this,
+                                                 $VERSION],
+                              );
+    if (not $etag) {
+       print $q->header(-status => 304);
+       print "304: Not modified\n";
+       exit 0;
+    }
+    print $q->header(-status => 200,
+                    -cache_control => 'public, max-age=86400',
+                    -etag => $etag,
+                    -content_type => 'text/html',
+                   );
+    print <<END;
+<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
+<html>
+END
+     print '<head><title>'.html_escape($cgi_var{package}).' Version Graph</title></head>'."\n";
+     print "<body>\n";
+     print '<a href="'.html_escape(munge_url($this,ignore_boring=>$cgi_var{ignore_boring}?0:1)).
+         '">['.($cgi_var{ignore_boring}?"Don't i":'I').'gnore boring]</a> ';
+     print '<a href="'.html_escape(munge_url($this,collapse=>$cgi_var{collapse}?0:1)).
+         '">['.($cgi_var{collapse}?"Don't c":'C').'ollapse]</a> ';
+     print '<a href="'.html_escape(munge_url($this,dot=>1)).
+         '">[Dot]</a><br>';
+     print '<img src="'.html_escape(munge_url($this,info=>0)).'">';
+     print <<END;
+</body>
+</html>
+END
+         exit 0;
+}
 
 # we want to first load the appropriate file,
 # then figure out which versions are there in which architectures,
 my %versions;
 my %version_to_dist;
 for my $dist (@{$config{distributions}}) {
-     $versions{$dist} = [getversions($cgi_var{package},$dist)];
+     $versions{$dist} = [get_versions(package => [split /\s*,\s*/, $cgi_var{package}],
+                                     dist => $dist,
+                                     source => 1,
+                                    )];
      # make version_to_dist
      foreach my $version (@{$versions{$dist}}){
          push @{$version_to_dist{$version}}, $dist;
      }
 }
+
 # then figure out which are affected.
 # turn found and fixed into full versions
-@{$cgi_var{found}} = makesourceversions($cgi_var{package},undef,@{$cgi_var{found}});
-@{$cgi_var{fixed}} = makesourceversions($cgi_var{package},undef,@{$cgi_var{fixed}});
-my @interesting_versions = makesourceversions($cgi_var{package},undef,keys %version_to_dist);
+@{$cgi_var{found}} = map {makesourceversions($_,undef,@{$cgi_var{found}})} split/\s*,\s*/, $cgi_var{package};
+@{$cgi_var{fixed}} = map {makesourceversions($_,undef,@{$cgi_var{fixed}})} split/\s*,\s*/, $cgi_var{package};
+my @interesting_versions = map {makesourceversions($_,undef,keys %version_to_dist)} split/\s*,\s*/, $cgi_var{package};
 
 # We need to be able to rip out leaves which the versions that do not affect the current versions of unstable/testing
 my %sources;
 @sources{map {m{(.+)/}; $1} @{$cgi_var{found}}} = (1) x @{$cgi_var{found}};
 @sources{map {m{(.+)/}; $1} @{$cgi_var{fixed}}} = (1) x @{$cgi_var{fixed}};
 @sources{map {m{(.+)/}; $1} @interesting_versions} = (1) x @interesting_versions;
-my $version = Debbugs::Versions->new(\&Debbugs::Versions::Dpkg::vercmp);
+
+# at this point, we know enough to calculate the etag.
+
+my @versions_files;
 foreach my $source (keys %sources) {
-     my $srchash = substr $source, 0, 1;
-     my $version_fh = new IO::File "$config{version_packages_dir}/$srchash/$source", 'r';
-     $version->load($version_fh);
+    my $srchash = substr $source, 0, 1;
+    next unless -e "$config{version_packages_dir}/$srchash/$source";
+    push @versions_files, "$config{version_packages_dir}/$srchash/$source";
+}
+
+$etag = etag_does_not_match(cgi=>$q,
+                           additional_data=>[@cgi_var{(qw(package ignore_boring),
+                                                       qw(collapse))
+                                                  },
+                                             $this,
+                                             $VERSION],
+                           files => [@versions_files
+                                    ],
+                          );
+if (not $etag) {
+    print $q->header(-status => 304);
+    print "304: Not modified\n";
+    exit 0;
+}
+
+my $version = Debbugs::Versions->new(\&Debbugs::Versions::Dpkg::vercmp);
+for my $version_fn (@versions_files) {
+    my $version_fh = IO::File->new($version_fn, 'r') or
+       warn "Unable to open $version_fn for reading: $!";
+    $version->load($version_fh);
 }
 # Here, we need to generate a short version to full version map
 my %version_map;
@@ -96,30 +193,108 @@ for my $found_fixed (qw(found fixed)) {
 my %all_states = $version->allstates($cgi_var{found},$cgi_var{fixed});
 
 my $dot = "digraph G {\n";
+if (defined $cgi_var{width} and defined $cgi_var{height}) {
+     $dot .= qq(size="$cgi_var{width},$cgi_var{height}";\n);
+}
 my %state = (found  => ['fillcolor="salmon"',
                        'style="filled"',
-                       'shape="diamond"',
+                       'shape="ellipse"',
                       ],
             absent => ['fillcolor="grey"',
                        'style="filled"',
+                       'shape="trapezium"',
                       ],
             fixed  => ['fillcolor="chartreuse"',
                        'style="filled"',
                        'shape="rect"',
                       ],
            );
+# TODO: Allow collapsing versions which are at the same state and not
+# in a suite.
+my %collapsed_nodes;
+my %group_nodes;
+if ($cgi_var{collapse}) {
+     my %reversed_nodes;
+     foreach my $key (keys %{$version->{parent}}) {
+         next if $cgi_var{ignore_boring} and (not defined $all_states{$key}
+                                              or $all_states{$key} eq 'absent');
+         next if $cgi_var{ignore_boring} and not version_relevant($version,$key,\@interesting_versions);
+         if (defined $version->{parent}{$key}) {
+              next if $cgi_var{ignore_boring} and (not defined $all_states{$version->{parent}{$key}}
+                                                   or $all_states{$version->{parent}{$key}} eq 'absent');
+              next if $cgi_var{ignore_boring} and not version_relevant($version,$version->{parent}{$key},\@interesting_versions);
+              push @{$reversed_nodes{$version->{parent}{$key}}},$key;
+         }
+         else {
+              $reversed_nodes{$key} ||=[];
+         }
+     }
+     # nodes that can be collapsed are those that have one child
+     # are in the same state as their parent, and are not in a suite
+     foreach my $key (keys %reversed_nodes) {
+         my ($short_version) = $key =~ m{/(.+)$};
+         if (not exists $version_to_dist{$key}
+             and @{$reversed_nodes{$key}} <= 1
+             and defined $version->{parent}{$key}
+             and $all_states{$key} eq $all_states{$version->{parent}{$key}}
+            ) {
+              # check to see if there is an entry for the parent or child of this node
+              my $group_node;
+              if ((@{$reversed_nodes{$key}} and exists $collapsed_nodes{$reversed_nodes{$key}[0]})) {
+                   $group_node = $collapsed_nodes{$reversed_nodes{$key}[0]};
+                   if ($group_nodes{$group_node}{parent} eq $key) {
+                        $group_nodes{$group_node}{parent} = $version->{parent}{$key};
+                   }
+              }
+              if (defined $version->{parent}{$key} and exists $collapsed_nodes{$version->{parent}{$key}}) {
+                   if (defined $group_node) {
+                        #MWHAHAHAHAHA
+                        my $collapser = $group_nodes{$collapsed_nodes{$version->{parent}{$key}}};
+                        push @{$collapser->{collapsed_nodes}},@{$group_nodes{$group_node}{collapsed_nodes}},$group_node;
+                        foreach (@{$collapser->{collapsed_nodes}}) {
+                             if (exists $group_nodes{$_}) {
+                                  $group_nodes{$_} = $collapser;
+                             }
+                        }
+                   }
+                   $group_node = $collapsed_nodes{$version->{parent}{$key}};
+              }
+              if (not defined $group_node) {
+                   $group_node = "group_$key";
+                   $group_nodes{$group_node} = {attr => qq("$group_node" [).join(',','label="some versions"',
+                                                                                 @{$state{$all_states{$key}}},
+                                                                                 'style="bold,filled"',
+                                                                                ).qq(]\n),
+                                                name => $group_node,
+                                                parent => $version->{parent}{$key},
+                                                collapsed_nodes => [],
+                                               };
+              }
+              $collapsed_nodes{$key} = $group_node;
+         }
+     }
+     my %used_node;
+     foreach my $group (values %group_nodes) {
+         next if $used_node{$group->{name}};
+         $used_node{$group->{name}} = 1;
+         $dot .= $group->{attr};
+     }
+}
+
 foreach my $key (keys %all_states) {
      my ($short_version) = $key =~ m{/(.+)$};
+     next if exists $collapsed_nodes{$key};
      next if $cgi_var{ignore_boring} and (not defined $all_states{$key}
                                          or $all_states{$key} eq 'absent');
      next if $cgi_var{ignore_boring} and not version_relevant($version,$key,\@interesting_versions);
      my @attributes = @{$state{$all_states{$key}}};
-     if (length $short_version and exists $version_to_dist{$short_version}) {
-         push @attributes, 'label="'.$key.'\n'."(".join(', ',@{$version_to_dist{$short_version}}).")\"";
+     if (exists $version_to_dist{$key}) {
+         push @attributes, 'label="'.$key.'\n'."(".join(', ',@{$version_to_dist{$key}}).")\"";
      }
      my $node_attributes = qq("$key" [).join(',',@attributes).qq(]\n);
      $dot .= $node_attributes;
 }
+
 foreach my $key (keys %{$version->{parent}}) {
      next if not defined $version->{parent}{$key};
      next if $cgi_var{ignore_boring} and $all_states{$key} eq 'absent';
@@ -127,37 +302,67 @@ foreach my $key (keys %{$version->{parent}}) {
                                          or $all_states{$version->{parent}{$key}} eq 'absent');
      # Ignore branches which are not ancestors of a currently distributed version
      next if $cgi_var{ignore_boring} and not version_relevant($version,$key,\@interesting_versions);
-     $dot .= qq("$key").'->'.qq("$version->{parent}{$key}" [dir="back"])."\n" if defined $version->{parent}{$key};
+     next if exists $collapsed_nodes{$key};
+     $dot .= qq("$key").'->'.q(").
+         (exists $collapsed_nodes{$version->{parent}{$key}}?
+          $group_nodes{$collapsed_nodes{$version->{parent}{$key}}}{name}:$version->{parent}{$key}).
+               qq(" [dir="back"])."\n" if defined $version->{parent}{$key};
+}
+if ($cgi_var{collapse}) {
+     my %used_node;
+     foreach my $group (values %group_nodes) {
+         next if $used_node{$group->{name}};
+         next if not defined $group->{parent};
+         $used_node{$group->{name}} = 1;
+         $dot .= qq("$group->{name}").'->'.q(").
+              (exists $collapsed_nodes{$group->{parent}}?
+               $group_nodes{$collapsed_nodes{$group->{parent}}}{name}:$group->{parent}).
+                   qq(" [dir="back"])."\n";
+     }
 }
 $dot .= "}\n";
 
 my $temp_dir = tempdir(CLEANUP => 1);
 
 if (not defined $cgi_var{dot}) {
-     my $dot_fh = new IO::File "$temp_dir/temp.dot",'w' or
+     my $dot_fh = IO::File->new("$temp_dir/temp.dot",'w') or
          die "Unable to open $temp_dir/temp.dot for writing: $!";
      print {$dot_fh} $dot or die "Unable to print output to the dot file: $!";
      close $dot_fh or die "Unable to close the dot file: $!";
      system('dot','-T'.$cgi_var{format},"$temp_dir/temp.dot",'-o',"$temp_dir/temp.$cgi_var{format}") == 0
          or print "Content-Type: text\n\nDot failed." and die "Dot failed: $?";
-     my $img_fh = new IO::File "$temp_dir/temp.$cgi_var{format}", 'r' or
+     my $img_fh = IO::File->new("$temp_dir/temp.$cgi_var{format}", 'r') or
          die "Unable to open $temp_dir/temp.$cgi_var{format} for reading: $!";
-     print "Content-Type: $img_types{$cgi_var{format}}\n\n";
+    print $q->header(-status => 200,
+                    -cache_control => 'public, max-age=300',
+                    -etag => $etag,
+                    -content_type => $img_types{$cgi_var{format}},
+                   );
      print <$img_fh>;
      close $img_fh;
 }
 else {
-     print "Content-Type: text\n\n";
+    print $q->header(-status => 200,
+                    -cache_control => 'public, max-age=300',
+                    -etag => $etag,
+                    -content_type => 'text',
+                   );
      print $dot;
 }
 
 
-my %_version_relevant_cache;
+our %_version_relevant_cache;
 sub version_relevant {
      my ($version,$test_version,$relevant_versions) = @_;
      for my $dist_version (@{$relevant_versions}) {
-         print STDERR "Testing $dist_version against $test_version\n";
-         return 1 if $version->isancestor($test_version,$dist_version);
+         if (exists $_version_relevant_cache{$dist_version}{$test_version}{$version}) {
+              return 1 if $_version_relevant_cache{$dist_version}{$test_version}{$version};
+         }
+         else {
+              my $rel = $version->isancestor($test_version,$dist_version);
+              $_version_relevant_cache{$dist_version}{$test_version}{$version} = $rel;
+              return 1 if $rel;
+         }
      }
      return 0;
 }