--- CVSROOT/cvs2rss.pl 2005/10/11 19:45:18 1.1 +++ CVSROOT/cvs2rss.pl 2005/10/11 22:11:16 1.2 @@ -13,23 +13,18 @@ # # AddType application/rss+xml .rss # -# Important note: -# If you have html files (or files with html in them) in your CVS Repository, look in your RSS.pm for: -# -# $text =~ s/ that -# contain HTML. -# # P.S. The "other" cvs2rss.pl isn't really cvs2rss, it is more like cvs2xml, boo! # P.P.S. Parts of this are stolen from commit2rss.rb and the XML::RSS examples. # # Version 1.0 - 08.24.04 RSS Compliant, no cvs diff capability yet.. # Version 1.1 - 08.31.04 Adding CVS Diff capability +# Edit by Guy Albertelli, -- corrected the packaging of Entities +# - commits now
ed
+#                          - removed html rants
 #
 
 use strict;
+use HTML::Entities();
 use XML::RSS;
 use POSIX;
 
@@ -37,10 +32,10 @@ use POSIX;
 my $rssFeed ="/home/loninst/public_html/loncapa.rss";
 my $emailDomain = "loncapa.org";
 my $channelTitle = "Lon-CAPA RSS Feed";
-my $channelLink = "http://install.loncapa.org/cvs.rss";
+my $channelLink = "http://install.loncapa.org/loncapa.rss";
 my $numEntries = 200;
 
-# Set this to 1 to enable cvs rdiff (is pretty broken if you are diffing html content)
+# Set this to 1 to enable cvs rdiff
 my $cvsDiff = 1;
 
 # Leave everything else alone
@@ -87,17 +82,17 @@ if ($cvsDiff == 1) {
 	# If the old version of the file is not NONE (if it isn't a new file), and if it is a .txt or .java file (that has no html)
 	# This will rdiff it against the previous version, and include that diff in the rss feed
 
-	if (($title[1] != "NONE") && ($title[0]=~/(.*).(txt|java)$/)){
+	if (($title[1] != "NONE") && ($title[0]=~/(.*).(pm)$/)){
 		my $tmpFile = "/tmp/diff.$$";
 		my $cmdLine = "cvs -n rdiff -kk -r " . $title[1] .  " -r " . $title[2] . " " . $title[0] . ">" . $tmpFile;
 		system($cmdLine);
 
-		$description .= "
Differences:
"; + $description .= "
Differences:
";
 		open CVSDIFF, "<" . $tmpFile;
 		foreach my $line () {
-			chomp($line);
-			$description .= $line . "
"; + $description .= &HTML::Entities::encode($line,'<>&"'); } + $description .= "
"; unlink($tmpFile); } } @@ -110,11 +105,8 @@ $rss->add_item( mode => 'insert' ); -# XML::RSS doesn't like HTML in the XML so Escape the description body with -# This is messy and stupid, but both XML::RSS and XML are sort of dumb, and I can't figure out a way around it. -# Certain RSS readers will try and render the HTML regardless of if it is wrapped in a CDATA tag or not so um, deal. foreach my $element (@{$rss->{'items'}}) { - $element->{'description'} = "{'description'} . "]]>"; + $element->{'description'} = &HTML::Entities::encode($element->{'description'},'<>&"'); } $rss->save($rssFeed);