Code

git-notify: Make the state file group writable
authorHolger Weiss <holger@zedat.fu-berlin.de>
Mon, 26 Oct 2009 01:41:51 +0000 (02:41 +0100)
committerHolger Weiss <holger@zedat.fu-berlin.de>
Mon, 26 Oct 2009 01:41:51 +0000 (02:41 +0100)
For shared repositories, the state file used by git-notify should
usually be group writable, so we now set the umask to 0002 by default.
This can be adjusted by setting the "notify.umask" configuration key or
by using the "-U" option on the command line.

tools/git-notify

index 68456fb579cab37ab7b6fcd26d9c6070aac5ffe2..97bd2adae74cb0421875302e4ac17176f8f67285 100755 (executable)
@@ -78,6 +78,9 @@ my @exclude_list = split /\s+/, git_config( "notify.exclude" ) || "";
 # the state file we use (can be changed with the -t option)
 my $state_file = git_config( "notify.statefile" ) || "/var/tmp/git-notify.state";
 
+# umask for creating the state file (can be set with -U option)
+my $mode_mask = git_config( "notify.umask" ) || 002;
+
 # Extra options to git rev-list
 my @revlist_options;
 
@@ -91,6 +94,7 @@ sub usage()
     print "   -s bytes  Set the maximum diff size in bytes (-1 for no limit)\n";
     print "   -t file   Set the file to use for reading and saving state\n";
     print "   -u url    Set the URL to the gitweb browser\n";
+    print "   -U mask   Set the umask for creatung the state file\n";
     print "   -i branch If at least one -i is given, report only for specified branches\n";
     print "   -x branch Exclude changes to the specified branch from reports\n";
     print "   -X        Exclude merge commits\n";
@@ -270,6 +274,7 @@ sub parse_options()
         elsif ($arg eq '-s') { $max_diff_size = shift @ARGV; }
         elsif ($arg eq '-t') { $state_file = shift @ARGV; }
         elsif ($arg eq '-u') { $gitweb_url = shift @ARGV; }
+        elsif ($arg eq '-U') { $mode_mask = shift @ARGV; }
         elsif ($arg eq '-i') { push @include_list, shift @ARGV; }
         elsif ($arg eq '-x') { push @exclude_list, shift @ARGV; }
         elsif ($arg eq '-X') { push @revlist_options, "--no-merges"; }
@@ -562,6 +567,8 @@ sub send_all_notices($$$)
 
 parse_options();
 
+umask( $mode_mask );
+
 # append repository path to URL
 $gitweb_url .= "/$repos_name.git" if $gitweb_url;