--- web/cgi-bin/cvslog.cgi 2003/08/21 22:22:12 1.27 +++ web/cgi-bin/cvslog.cgi 2004/03/24 20:57:27 1.31 @@ -1,6 +1,6 @@ #!/usr/bin/perl -w -use CGI; +use CGI qw/:standard/; $query=new CGI; my($period)=$query->param('period'); $period = "today" unless defined $period; @@ -44,43 +44,60 @@ $modulelist = `ls $cvsroot` unless defined $modulelist $modulelist =~ s/[\r\n]/ /gm; my($cvs2clpath) = "/usr/local/bin/cvs2cl"; -my($cvs2clargs) = "--stdout --no-wrap -r -t -w -S -U $cvsroot/CVSROOT/users -l \"-d'$firstdate'\" -g \"-d$cvsroot\" -g \"-Q\""; +my($cvs2clargs) = "--stdout --no-wrap --no-common-dir -r -t -w -S -U $cvsroot/CVSROOT/users -l \"-d'$firstdate'\" -g \"-d$cvsroot\" -g \"-Q\""; my($updatecmd) = "/home/iscream/bin/fullcvsupdate.sh $modulelist"; my($logcmd) = "cd /tmp/i-scream/cvstmp && $cvs2clpath $cvs2clargs"; -my ($left) = "../htdocs/left.inc" ; -my ($title) = "../htdocs/title.inc"; -my ($bottom) = "../htdocs/bottom.inc"; +my ($incdir) = "../nwww"; -print "Content-type: text/html\n\n"; +# Include files +my ($doctype) = "$incdir/doctype.inc"; +my ($style) = "$incdir/style.inc"; +my ($header) = "$incdir/header.inc"; +my ($footer) = "$incdir/footer.inc"; +my ($menu) = "$incdir/menu.inc" ; -print <<"END"; - +# Section markers +my ($sec) = 0; +my ($secwho) = 1; +my ($secfiles) = 2; +my ($seclog) = 3; - +# This should be application/xhtml+xml +print "Content-type: text/html\n\n"; - - The i-scream Project Commit Log - +&print_html($doctype); - +print <<"END"; - - - - - -
+ + + i-scream CVS Commit Logs + END -&print_html($left); +&print_html($style); print <<"END"; - + + + +
+
END -&print_html($title); +&print_html($header); print <<"END"; -

$heading

+ +
+

+ i-scream CVS Commit Logs +

+

+ $heading +

+
END print `$updatecmd >/dev/null 2>&1`; @@ -88,41 +105,93 @@ print `$updatecmd >/dev/null 2>&1`; my(@modules) = split (/\s+/, $modulelist); foreach my $module (@modules) { + my($firstentry) = 1; + print <<"END"; +
+

+ $module module +

+END - print "\n\n"; - print "\n\n"; - print "\n"; - - print "
\n$module module\n
\n"; - my(@lines) = `$logcmd $module 2>&1`; if(@lines == 0) { - print "There have been no commits in this module during this period.\n"; + print <<"END"; +

+ There have been no commits in this module during this period. +

+END } else { + my(@log) = (); foreach my $line (@lines) { - if ($line =~ /^([0-9]{4}-[0-9]{2}-[0-9]{2}.*?)\s+([^\s]+)\s+<([^\s]+)>$/) { - print "\n\n"; - print HTML_encode($1); - print "\n"; - print " committed by "; - print HTML_encode("$2"); - print "\n\n" + if ($line =~ /^([0-9]{4}-[0-9]{2}-[0-9]{2}.*?)\s+([^\s]+)(\s+<([^\s]+)>)?$/) { + my($datetime) = escapeHTML($1); + my($committer) = escapeHTML($2); + my($email) = $4; + + if($sec != $secwho) { + if($sec == $seclog) { + @log = &trim_array(@log); + foreach my $logline (@log) { + $logline =~ s/^\t//; + print "$logline\n"; + } + @log = (); + print <<"END"; + + +END + } + if($firstentry) { + $firstentry = 0; + print <<"END"; +
+END + } + else { + print <<"END"; +
+END + } + $sec = $secwho; + } + + print <<"END"; +

+ + $datetime + + committed by +END + if(defined $email) { + print <<"END"; + $committer +END + } + else { + print <<"END"; + $committer +END + } + print <<"END"; +

+END } - elsif ($line =~ /^([0-9]{4}-[0-9]{2}-[0-9]{2}.*?)\s+([^\s]+)$/) { - print "\n\n"; - print HTML_encode($1); - print "\n"; - print " committed by "; - print HTML_encode("$2"); - print "\n\n" - } - elsif($line =~ /(\S+) (\(\S+\))([,:])/ || $line =~ /(\S+) (\(\S+,\s+\S+\))([,:])/) { - my ($file, $rest, $ext) = ($1, $2, $3); - print ""; - print ""; - print HTML_encode($file); - print " "; - if($rest =~ /\((\S+)\.(\d+)(.*)\)/) { + elsif($line =~ /([^\s]+) (\([^,^\)]+(,[^\)]+)?\))([,:])/) { + my ($file, $rev, $tags, $ext) = ($1, $2, $3, $4); + $ext = escapeHTML($ext); + if($sec != $secfiles) { + $sec = $secfiles; + print <<"END"; +

+END + } + $file = escapeHTML($file); + print <<"END"; + $file +END + if($rev =~ /\(([^\s]+)\.(\d+)(.*)\)/) { my $start = $1; my $end = $2; my $other = $3; @@ -138,67 +207,111 @@ foreach my $module (@modules) { chop $oldrev; } if(defined $oldrev) { - my $diff = ".diff?r1=$oldrev&r2=$newrev"; - print "("; - print HTML_encode("$start.$end"); - print ""; - print HTML_encode("$other)"); + my $diff = ".diff?r1=$oldrev&r2=$newrev"; + my $startend = escapeHTML("$start.$end"); + $other = escapeHTML($other); + chomp $other; + print <<"END"; + ($startend$other)$ext +END } else { - print HTML_encode($rest); + $rev = escapeHTML($rev); + print <<"END"; + $rev$ext +END } } else { - print HTML_encode($rest); + $rev = escapeHTML($rev); + print <<"END"; + $rev$ext +END } - print HTML_encode($ext); - print "
\n" + if($ext !~ /:$/) { + print <<"END"; +
+END + } } else { - chop $line; - print ""; - print HTML_encode($line); - print "
\n" + next if $sec == $secwho; # not done secfiles yet + if($sec != $seclog) { + $sec = $seclog; + print <<"END"; +

+
+END
+                }
+                chomp $line;
+                push(@log, $line);
             }
         }
+        if($sec == $seclog) {
+            @log = &trim_array(@log);
+            foreach my $logline (@log) {
+                $logline =~ s/^\t//;
+                print "$logline\n";
+            }
+            @log = ();
+            print <<"END";
+
+
+END + } } - print "

\n\n"; + print <<"END"; +

+END } -&print_html($bottom); +print <<"END"; +
+
+END +&print_html($footer); + print <<"END"; -
+ +END - +&print_html($menu); +print <<"END"; + + + END exit 0; -#------------------------------------------------------ -# sub HTML_encode -# -# escape HTML characters that may cause problems when -# shown either in the or within text fields. -#------------------------------------------------------ -sub HTML_encode ($){ - my ($encoded) = @_; - $encoded =~ s/&/&/g; - $encoded =~ s/"/"/g; - $encoded =~ s//>/g; - return $encoded; +# Print the contents of a file containing html +sub print_html { + my ($filename) = @_; + open(FILE, $filename); + while() { + print; + } + close FILE; } -# Print a file without escaping HTML: - -sub print_html ($) { - my ($filename) = @_; - print `cat $filename 2>&1`; +# Trim an array (remove beginning and end emptry entries) +sub trim_array { + my (@array) = @_; + for(my $i = 0; $i < scalar @array; $i++) { + if($array[$i] ne "") { + splice @array, 0, $i; + last; + } + } + for(my $i = scalar @array - 1; $i >= 0; $i--) { + if($array[$i] ne "") { + splice @array, $i+1; + last; + } + } + return @array; }