On Tue, Aug 10, 2010 at 15:54, James <purpleidea@xxxxxxxxx> wrote:
> Sorry about that,
> I guess I had only read the README.
> Hope this is better:
Not really, no. You should send the patches you produce with
git-format-patch with git-send-email, and try sending to yourself
first and apply it with git-am (this is all mentioned in
SubmittingPatches).
This is what your new patch looks like after being applied with
git-am:
commit 7be6207e8923cd7c4c48243f5257a0fdba6bfa0a
Author: James <purpleidea@xxxxxxxxx>
Date: Tue Aug 10 11:54:43 2010 -0400
Use a default for a bad env config file variable
Sorry about that,
I guess I had only read the README.
Hope this is better:
From d29adf8c788b8a747bfd38dd7e10f684de9aa8e9 Mon Sep 17 00:00:00 2001
From: James Shubin <purpleidea@xxxxxxxxx>
Date: Tue, 10 Aug 2010 10:30:22 -0400
Subject: [PATCH] Use a default for a bad env config file variable.
Signed-off-by: James Shubin <purpleidea@xxxxxxxxx>
diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index 4efeebc..43294e1 100755
--- a/gitweb/gitweb.perl
+++ b/gitweb/gitweb.perl
@@ -605,6 +605,10 @@ sub evaluate_gitweb_config {
} elsif (-e $GITWEB_CONFIG_SYSTEM) {
do $GITWEB_CONFIG_SYSTEM;
die $@ if $@;
+ # if config file from env is missing, then try the default anyways
+ } elsif (-e "++GITWEB_CONFIG_SYSTEM++") {
+ do "++GITWEB_CONFIG_SYSTEM++";
+ die $@ if $@;
}
}
I.e. your message has become part of the patch. To include commentary
on resend add it after -- and before the diffstat (also in
SubmittingPatches).
Thanks.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html