{"thread":{"id":"11970","subject":"git.owner, hope everything is fine","startedAt":"2008-02-08T16:38:02Z","lastAt":"2008-02-13T01:05:47Z","messageCount":5,"participants":["Bruno Ribas","Bruno Cesar Ribas","Junio C Hamano"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"67965","messageId":"1202488684-13266-1-git-send-email-ribas@c3sl.ufpr.br","threadId":"11970","inReplyTo":null,"subject":"git.owner, hope everything is fine","fromName":"Bruno Ribas","fromEmail":"ribas@c3sl.ufpr.br","sentAt":"2008-02-08T16:38:02Z","receivedAt":"2008-02-08T16:38:02Z","isPatch":false,"sender":{"key":"ribas@c3sl.ufpr.br","avatar":null},"body":"\nMade final adjustments and hope it is OK now.\n"},{"id":"67966","messageId":"1202488684-13266-2-git-send-email-ribas@c3sl.ufpr.br","threadId":"11970","inReplyTo":"1202488684-13266-1-git-send-email-ribas@c3sl.ufpr.br","subject":"[PATCH] gitweb: Use the config file to set repository owner's name.","fromName":"Bruno Ribas","fromEmail":"ribas@c3sl.ufpr.br","sentAt":"2008-02-08T16:38:03Z","receivedAt":"2008-02-08T16:38:03Z","isPatch":true,"sender":{"key":"ribas@c3sl.ufpr.br","avatar":null},"body":"Now gitweb checks if gitweb.owner exists before trying to get filesystem's\nowner.\n\nAllow to use configuration variable gitweb.owner set the repository owner,\nit checks the gitweb.owner, if not set it uses filesystem directory's owner.\n\nUseful when we don't want to maintain project list file, and all\nrepository directories have to have the same owner (for example when the\nsame SSH account is shared for all projects, using ssh_acl to control\naccess instead).\n\nSigned-off-by: Bruno Ribas <ribas@c3sl.ufpr.br>\n---\n gitweb/gitweb.perl |    6 +++++-\n 1 files changed, 5 insertions(+), 1 deletions(-)\n\ndiff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl\nindex 8ef2735..c8fe22a 100755\n--- a/gitweb/gitweb.perl\n+++ b/gitweb/gitweb.perl\n@@ -1759,6 +1759,7 @@ sub git_get_project_owner {\n \tmy $owner;\n \n \treturn undef unless $project;\n+\t$git_dir = \"$projectroot/$project\";\n \n \tif (!defined $gitweb_project_owner) {\n \t\tgit_get_project_list_from_file();\n@@ -1767,8 +1768,11 @@ sub git_get_project_owner {\n \tif (exists $gitweb_project_owner->{$project}) {\n \t\t$owner = $gitweb_project_owner->{$project};\n \t}\n+\tif (!defined $owner){\n+\t\t$owner = git_get_project_config('owner');\n+\t}\n \tif (!defined $owner) {\n-\t\t$owner = get_file_owner(\"$projectroot/$project\");\n+\t\t$owner = get_file_owner(\"$git_dir\");\n \t}\n \n \treturn $owner;\n-- \n1.5.4.24.gce08d\n"},{"id":"67967","messageId":"1202488684-13266-3-git-send-email-ribas@c3sl.ufpr.br","threadId":"11970","inReplyTo":"1202488684-13266-2-git-send-email-ribas@c3sl.ufpr.br","subject":"[PATCH] gitweb: Update gitweb/README to include the new per-repository gitweb.owner","fromName":"Bruno Ribas","fromEmail":"ribas@c3sl.ufpr.br","sentAt":"2008-02-08T16:38:04Z","receivedAt":"2008-02-08T16:38:04Z","isPatch":true,"sender":{"key":"ribas@c3sl.ufpr.br","avatar":null},"body":"\nSigned-off-by: Bruno Ribas <ribas@c3sl.ufpr.br>\n---\n gitweb/README |    4 ++++\n 1 files changed, 4 insertions(+), 0 deletions(-)\n\ndiff --git a/gitweb/README b/gitweb/README\nindex 4c8bedf..2163071 100644\n--- a/gitweb/README\n+++ b/gitweb/README\n@@ -233,6 +233,10 @@ You can use the following files in repository:\n    Displayed in the project summary page. You can use multiple-valued\n    gitweb.url repository configuration variable for that, but the file\n    takes precendence.\n+ * gitweb.owner\n+   You can use the gitweb.owner repository configuration variable to set\n+   repository's owner. It is displayed in the project list and summary\n+   page. If it's not set, filesystem directory's owner is used.\n  * various gitweb.* config variables (in config)\n    Read description of %feature hash for detailed list, and some\n    descriptions.\n-- \n1.5.4.24.gce08d\n"},{"id":"68580","messageId":"20080213005202.GA5965@c3sl.ufpr.br","threadId":"11970","inReplyTo":"1202488684-13266-2-git-send-email-ribas@c3sl.ufpr.br","subject":"Re: [PATCH] gitweb: Use the config file to set repository owner's name.","fromName":"Bruno Cesar Ribas","fromEmail":"ribas@c3sl.ufpr.br","sentAt":"2008-02-13T00:52:02Z","receivedAt":"2008-02-13T00:52:02Z","isPatch":true,"sender":{"key":"ribas@c3sl.ufpr.br","avatar":null},"body":"Was it forgotten ? 8^)\n\nOn Fri, Feb 08, 2008 at 02:38:03PM -0200, Bruno Ribas wrote:\n> Now gitweb checks if gitweb.owner exists before trying to get filesystem's\n> owner.\n> \n> Allow to use configuration variable gitweb.owner set the repository owner,\n> it checks the gitweb.owner, if not set it uses filesystem directory's owner.\n> \n> Useful when we don't want to maintain project list file, and all\n> repository directories have to have the same owner (for example when the\n> same SSH account is shared for all projects, using ssh_acl to control\n> access instead).\n> \n> Signed-off-by: Bruno Ribas <ribas@c3sl.ufpr.br>\n> ---\n>  gitweb/gitweb.perl |    6 +++++-\n>  1 files changed, 5 insertions(+), 1 deletions(-)\n> \n> diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl\n> index 8ef2735..c8fe22a 100755\n> --- a/gitweb/gitweb.perl\n> +++ b/gitweb/gitweb.perl\n> @@ -1759,6 +1759,7 @@ sub git_get_project_owner {\n>  \tmy $owner;\n>  \n>  \treturn undef unless $project;\n> +\t$git_dir = \"$projectroot/$project\";\n>  \n>  \tif (!defined $gitweb_project_owner) {\n>  \t\tgit_get_project_list_from_file();\n> @@ -1767,8 +1768,11 @@ sub git_get_project_owner {\n>  \tif (exists $gitweb_project_owner->{$project}) {\n>  \t\t$owner = $gitweb_project_owner->{$project};\n>  \t}\n> +\tif (!defined $owner){\n> +\t\t$owner = git_get_project_config('owner');\n> +\t}\n>  \tif (!defined $owner) {\n> -\t\t$owner = get_file_owner(\"$projectroot/$project\");\n> +\t\t$owner = get_file_owner(\"$git_dir\");\n>  \t}\n>  \n>  \treturn $owner;\n> -- \n> 1.5.4.24.gce08d\n\n-- \nBruno Ribas - ribas@c3sl.ufpr.br\nhttp://web.inf.ufpr.br/ribas\nC3SL: http://www.c3sl.ufpr.br \n"},{"id":"68584","messageId":"7vlk5pzmvo.fsf@gitster.siamese.dyndns.org","threadId":"11970","inReplyTo":"20080213005202.GA5965@c3sl.ufpr.br","subject":"Re: [PATCH] gitweb: Use the config file to set repository owner's name.","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2008-02-13T01:05:47Z","receivedAt":"2008-02-13T01:05:47Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Bruno Cesar Ribas <ribas@c3sl.ufpr.br> writes:\n\n> Was it forgotten ? 8^)\n\nNo, I have it on one topic, and I personally do not see an issue\nwith it, but I haven't merged it to 'pu' nor 'next' yet.  I\nthought Jakub wanted to give Pasky the first refusal right, so\nI've been waiting for a few days.\n"}]}