Assorted whitespace, stdout, and comment bikeshedding.

1 files changed, 4 insertions, 4 deletions

diff --git a/usr/local/sbin/newrepo b/usr/local/sbin/newrepo
index b903064..2ad8ada 100755
--- a/usr/local/sbin/newrepo
+++ b/usr/local/sbin/newrepo
@@ -56,7 +56,7 @@ if( @errors ) {
   exit( 2 );
 };
 
-while ( 1 ) {
+while( 1 ) {
 
   print( "Provide a one-line description to be used in repo listings, the shorter the better:\n" );
   chomp( $description = <STDIN> );
@@ -73,7 +73,7 @@ while ( 1 ) {
 
 # CREATE USER/GROUPS
 unless( system( "mkdir /srv/git/$reponame.git" ) == 0 ) { # We need a locking or atomic operation as our first to check against simultaneous execution.
-  print( "Somebody else has created the same repo as you in the course of executing this program!" );
+  print( "Somebody else has created the same repo as you in the course of executing this program!\n" );
   exit( 3 );
 };
 system( "/usr/sbin/addgroup --quiet git-$reponame" );
@@ -222,13 +222,13 @@ system( "/bin/cat /etc/ikiwiki/wikilist.d/* > $wikilisttempfile" );
 system( "/bin/mv $wikilisttempfile /etc/ikiwiki/wikilist" ); # This is marginally racy, but the consequences are probably ignorable.
 
 open ( CGITRC, ">/etc/cgitrc.d/$reponame" ); # Maybe someday cgit will support cgitrc.d.
-print( CGITRC
+print( CGITRC # In the meantime, we fake it.
 "repo.url=$reponame
 repo.path=/srv/git/$reponame.git
 repo.desc=$description
 repo.owner=$email
 
-" ); # In the meantime, we fake it.
+" ); # cgit already escapes HTML, so we don't need to do it on $description
 close( CGITRC );
 $cgitrctempfile = `/bin/mktemp`;
 chomp( $cgitrctempfile );