Re: git instaweb - share all project files

2013-04-14 Thread Jakub Narębski
W dniu 10.04.2013 19:03, Jakub Narębski pisze:
 W dniu 07.04.2013 05:02, Trenton D. Adams pisze:
 
 On that first page that shows up, it shows the .git folder.  It would
 be kind of nice if it shared out both the git repo and the actual
 current project files.  I frequently have stuff I'd like to see in a
 web browser, and even requires one (i.e. Navigating to
 file:///home/blah/blah doesn't work; ajax requests for example)
 
 There are a few possible solutions, from simplest to most complicated:
[...]

 2. In the web server configuration generated by git-instaweb, perhaps
as an option, add serving of worktree (with mod_autoindex aka.
'Options +Indexes' for Apache2, and equivalent solutions for *all*
other supported web servers: lighttpd, mongoose, plack, webrick).
 
This may require some fiddling with URI rewriting, or change of
gitweb URI, to be able to have both worktree index and gitweb
script (gitweb is now under '/'), so it should probably be protected
by an option to git-instaweb.

The trouble (besides writing the same code for 5 web servers) is that
you have to make web server detect whether you want to access workdir or
gitweb from URL alone...

...and still you wouldn't have nice UI integration.

 3. Add proper support to gitweb: add 'worktree' action (similar to
'tree' action / view).  Probably needs to be made somewhat
configurable (and of course enabled in git-instaweb).

I wonder if this should be solved via new actions, for example
'worktree' and 'file' (which could also be a solution to problem in 2.)
via URI rewriting), or via new symbolic hash name, for example
'WORKTREE' (like 'HEAD').

Hmmm... while at it, perhaps we can add browsing of INDEX version of files?

-- 
Jakub Narębski
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: git instaweb - share all project files

2013-04-13 Thread Jakub Narębski
Please do not top-post.

On 11.04.2013, Trenton D. Adams wrote:

 #1 would actually work.  Though long term it would be cool to view it
 with all the most recent commit information, kind of like github does.
  You know, showing updated 4 days ago.
 
 On Wed, Apr 10, 2013 at 11:03 AM, Jakub Narębski jna...@gmail.com wrote:
 W dniu 07.04.2013 05:02, Trenton D. Adams pisze:

 On that first page that shows up, it shows the .git folder.  It would
 be kind of nice if it shared out both the git repo and the actual
 current project files.  I frequently have stuff I'd like to see in a
 web browser, and even requires one (i.e. Navigating to
 file:///home/blah/blah doesn't work; ajax requests for example)

 There are a few possible solutions, from simplest to most complicated:

 1. Configure gitweb ran by git-instaweb to have 'worktree' link in
the action bar pointing to 'file:///path/to/repo' (or rather
'file:///path/to/workdir') via 'actions' feature, adding e.g.

  $feature{'actions'}{'default'} =
 [('worktree', 'file:///path/to/repo', 'summary')];

to gitweb_config.perl / gitweb.conf used by git-instaweb's gitweb.

This of course works only for local use, so either git-instaweb
or gitweb (in config) should check that we use it locally
(e.g. if hostname is 'localhost' or equivalent).
[...]

Unfortunately it turns out such simple solution doesn't really work.

First, for some reason Firefox 20.0 refuses to follow file:/// link,
even though it shows correctly worktree if copy'n'pasted as URL in new
window / new tab.

Second, it would work only for the top git repository.  There can be
repositories being submodules, or in untracked subdirectories (like
e.g. 't/trash directory.t*/.git' in git/.git), and said 'worktree'
link would always lead to top git repository workdir.

But anyway, here you have it:
-- 8 --
Subject: [PATCH] git-instaweb: Show project files... kind of (WIP)

On first page that git-instaweb shows, it shows the .git folder.  It
would be kind of nice if it shared out both the git repo and the
actual current project files (i.e. current worktree). [...]

[...]

This commit implements solution 1.), with git-instaweb enabling it
only for local use, which means when it is run with `--local` option.

Signed-off-by: Jakub Narebski jna...@gmail.com
---
 git-instaweb.sh | 14 ++
 1 file changed, 14 insertions(+)

diff --git a/git-instaweb.sh b/git-instaweb.sh
index 01a1b05..43ae255 100755
--- a/git-instaweb.sh
+++ b/git-instaweb.sh
@@ -22,6 +22,13 @@ restartrestart the web server
 . git-sh-setup
 
 fqgitdir=$GIT_DIR
+if test x$(git rev-parse --is-bare-repository) = xfalse
+then
+   fqworktree=$(git rev-parse --show-toplevel)
+else
+   fqworktree=
+fi
+
 local=$(git config --bool --get instaweb.local)
 httpd=$(git config --get instaweb.httpd)
 root=$(git config --get instaweb.gitwebdir)
@@ -588,6 +595,13 @@ our \$projects_list = \$projectroot;
 
 \$feature{'remote_heads'}{'default'} = [1];
 EOF
+   test x$fqworktree != x 
+   test x$local = xtrue 
+   cat $fqgitdir/gitweb/gitweb_config.perl EOF
+\$feature{'actions'}{'default'} = [
+   ('worktree', 'file://$fqworktree', 'summary')
+];
+EOF
 }
 
 configure_httpd() {
-- 
1.8.2.1.367.geafc030.dirty

--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: git instaweb - share all project files

2013-04-13 Thread Eric Wong
Jakub Narębski jna...@gmail.com wrote:
 Unfortunately it turns out such simple solution doesn't really work.
 
 First, for some reason Firefox 20.0 refuses to follow file:/// link,
 even though it shows correctly worktree if copy'n'pasted as URL in new
 window / new tab.

I can confirm this behavior with chromium on Debian testing
(25.0.1364.160 (Developer Build 186726) Debian 7.0)

I can guess this is likely for security reasons.
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: git instaweb - share all project files

2013-04-10 Thread Jakub Narębski
W dniu 07.04.2013 05:02, Trenton D. Adams pisze:

 On that first page that shows up, it shows the .git folder.  It would
 be kind of nice if it shared out both the git repo and the actual
 current project files.  I frequently have stuff I'd like to see in a
 web browser, and even requires one (i.e. Navigating to
 file:///home/blah/blah doesn't work; ajax requests for example)

There are a few possible solutions, from simplest to most complicated:

1. Configure gitweb ran by git-instaweb to have 'worktree' link in
   the action bar pointing to 'file:///path/to/repo' (or rather
   'file:///path/to/workdir') via 'actions' feature, adding e.g.

 $feature{'actions'}{'default'} =
[('worktree', 'file:///path/to/repo', 'summary')];

   to gitweb_config.perl / gitweb.conf used by git-instaweb's gitweb.

   This of course works only for local use, so either git-instaweb
   or gitweb (in config) should check that we use it locally
   (e.g. if hostname is 'localhost' or equivalent).

2. In the web server configuration generated by git-instaweb, perhaps
   as an option, add serving of worktree (with mod_autoindex aka.
   'Options +Indexes' for Apache2, and equivalent solutions for *all*
   other supported web servers: lighttpd, mongoose, plack, webrick).

   This may require some fiddling with URI rewriting, or change of
   gitweb URI, to be able to have both worktree index and gitweb
   script (gitweb is now under '/'), so it should probably be protected
   by an option to git-instaweb.

3. Add proper support to gitweb: add 'worktree' action (similar to
   'tree' action / view).  Probably needs to be made somewhat
   configurable (and of course enabled in git-instaweb).


Unfortunately solution 1.) which is simplest is not enough for your
situation...

I can add 3.) to my gitweb TODO, but I don't know when I would be able
to get to implementing it.
-- 
Jakub Narębski

--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: git instaweb - share all project files

2013-04-10 Thread Trenton D. Adams
#1 would actually work.  Though long term it would be cool to view it
with all the most recent commit information, kind of like github does.
 You know, showing updated 4 days ago.

On Wed, Apr 10, 2013 at 11:03 AM, Jakub Narębski jna...@gmail.com wrote:
 W dniu 07.04.2013 05:02, Trenton D. Adams pisze:

 On that first page that shows up, it shows the .git folder.  It would
 be kind of nice if it shared out both the git repo and the actual
 current project files.  I frequently have stuff I'd like to see in a
 web browser, and even requires one (i.e. Navigating to
 file:///home/blah/blah doesn't work; ajax requests for example)

 There are a few possible solutions, from simplest to most complicated:

 1. Configure gitweb ran by git-instaweb to have 'worktree' link in
the action bar pointing to 'file:///path/to/repo' (or rather
'file:///path/to/workdir') via 'actions' feature, adding e.g.

  $feature{'actions'}{'default'} =
 [('worktree', 'file:///path/to/repo', 'summary')];

to gitweb_config.perl / gitweb.conf used by git-instaweb's gitweb.

This of course works only for local use, so either git-instaweb
or gitweb (in config) should check that we use it locally
(e.g. if hostname is 'localhost' or equivalent).

 2. In the web server configuration generated by git-instaweb, perhaps
as an option, add serving of worktree (with mod_autoindex aka.
'Options +Indexes' for Apache2, and equivalent solutions for *all*
other supported web servers: lighttpd, mongoose, plack, webrick).

This may require some fiddling with URI rewriting, or change of
gitweb URI, to be able to have both worktree index and gitweb
script (gitweb is now under '/'), so it should probably be protected
by an option to git-instaweb.

 3. Add proper support to gitweb: add 'worktree' action (similar to
'tree' action / view).  Probably needs to be made somewhat
configurable (and of course enabled in git-instaweb).


 Unfortunately solution 1.) which is simplest is not enough for your
 situation...

 I can add 3.) to my gitweb TODO, but I don't know when I would be able
 to get to implementing it.
 --
 Jakub Narębski

--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


git instaweb - share all project files

2013-04-06 Thread Trenton D. Adams
Hello,

On that first page that shows up, it shows the .git folder.  It would
be kind of nice if it shared out both the git repo and the actual
current project files.  I frequently have stuff I'd like to see in a
web browser, and even requires one (i.e. Navigating to
file:///home/blah/blah doesn't work; ajax requests for example)

Thanks.
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html