Re: gitweb feature request: tarball for each commit

2005-09-08 Thread Sven Verdoolaege
On Thu, Sep 08, 2005 at 04:30:22PM +1200, Martin Langhoff wrote:
 Actually... should get it done. I'll see if I can sneak it in sometime 
 soon... 

This has been done at least twice already.
See e.g., http://www.liacs.nl/~sverdool/gitweb.cgi?p=gitweb.git;a=summary
Check the archives for the other implementation.

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


Re: gitweb feature request: tarball for each commit

2005-09-08 Thread Martin Langhoff
  Yes, this is nice for smaller projects. But I don't think, that we want
  to do such a thing on the kernel.org servers. 

 I think this is a very useful feature for for some, but not all,
 repositories. Default it to off and have a magic file (like git-daemon),
 or a config variable turn it on.

+1! It'd be murder for large and/or popular projects, but it's a
really conventient thing to have as an optional feature. Archzoom has
it, and defaults to off ;)

I've checked out Sven's tree as well -- his implementation is pretty
cool actually, much more sophisticated that I'd planned, and using
POST to avoid stupid bots requesting the tarballs.

In this situation (look at my repo via gitweb, it has some cool
patches over otherhacker's branch) it is hard to discriminate what
patches differentiate Sven's repo from Kay's. I'm thinking of, when
looking at a branch, having a link that shows the equivalent of 
`cg-log -r thisbranch:origin` and another one for `cg-log -r
origin:thisbranch`.

cheers,



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


Re: gitweb - feature request

2005-08-12 Thread Kay Sievers
On Wed, Aug 10, 2005 at 06:53:40AM +0100, Ian Campbell wrote:
 On Tue, 2005-08-09 at 21:58 +0200, Kay Sievers wrote:
 
  I was hoping people that want stuff like this would use a RSS reader. :)
 
 I used to subscribe to the kernel RSS feed (using blam) but I found I
 was only getting the most recent 20 commits, which wasn't much good when
 a big batch went in because I would miss some. Has this been fixed or
 was it something I was doing wrong?

It's 30 now and up to 150 if they are not older than 48 hours. We can
tweak the numbers if you like to have it different...

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


Re: gitweb - feature request

2005-08-12 Thread Mitchell Blank Jr
Kay Sievers wrote:
 It's 30 now and up to 150 if they are not older than 48 hours.
 We can change the numbers, if you hava a better idea...

Is it really hard to just make it purely time-based (git-rev-list --max-age)?
Think of if Linus is merging with a lot of people and then pushes the results
to the master repository -- suddenly there's, say, 400 new commits since the
last time my aggregator checked 2 hours ago.

 For now it just lists all changed files to the log message, similar to the
 commit view. Is that ok, or do we really need the diffstat,

It looks great, thanks!  diffstat would be slightly nicer but not a big
deal.

 It may be
 a bit expensive to generate it for all the commits with every RSS request...

Well if the RSS feed's popularity takes off you'll probably want to *not*
generate it every time and instead serve it from a static file.  This can
be as simple as a Makefile like:

commits.rss: $(GITDIR)/refs/heads/master
/path/to/myperlscript.pl $(GITDIR)  commits.rss.NEW  chmod 
444 commits.rss.NEW  mv commits.rss.NEW commits.rss

and then call make -C /my/rss/dir -s -f /path/to/mymakefile.mk from cron
every minute.

Serving the rss from a static file has the big advantage that a well-behaved
aggregator will only request it if the modification date changed which saves
everyone bandwidth.

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


Re: gitweb - feature request

2005-08-10 Thread Mitchell Blank Jr
Ian Campbell wrote:
 I used to subscribe to the kernel RSS feed (using blam) but I found I
 was only getting the most recent 20 commits, which wasn't much good when
 a big batch went in because I would miss some.

Yes, I have that problem too.  It appears to be just the way that gitweb
works - look at the git_rss function in the source:
  ftp://ftp.kernel.org/pub/software/scm/gitweb/gitweb.cgi

Kay -- is there any chance of fixing this?  I love reading the kernel
commits via RSS but this makes it a lot less usable than it could be.
Really it should return all commits within, say, the last 36 hours so as
long as your aggregator polls reasonably often you won't miss anyhing.

The other thing on my wishlish is diffstat -- sometimes the commit messages
can be a little ambiguous and just adding what files were changed would
help alot.  For commits that touch a large number of files maybe it could
just show the files that changed the most like:

   net/bar.c  |  412 -
   drivers/char/foo.c |  354 +-
   arch/baz/boot.S|   99 
   [16 other files changed]

Other than that though I really love gitweb and the RSS support is a
great touch.  It really makes tracking kernel commits painless.

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


Re: gitweb - feature request

2005-08-09 Thread Kay Sievers
On Tue, Aug 09, 2005 at 09:31:04PM +0200, Sam Ravnborg wrote:
 Hi Kay.
 
 When browsing http://www.kernel.org/git I often find myself looking for
 the most recently changed tree.
 For this it is very good that you have the last change in italic and
 bolded if newer than a few hours (I think).

Hmm, if last change is minutes it's bold, if it's hours old it's green.
What do you miss?

 A nice additional feature would be the possibility to sort the output
 according to last change, owner and description.
 Using a click-able table heading would be the most intuitive way.
 
 I have not looked into the source for gitweb, so I really do not know
 how difficult this would be.

I was hoping people that want stuff like this would use a RSS reader. :)

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


Re: gitweb - feature request

2005-08-09 Thread Johannes Schindelin
Hi,

On Tue, 9 Aug 2005, Sam Ravnborg wrote:

 On Tue, Aug 09, 2005 at 09:58:18PM +0200, Kay Sievers wrote:
  
  I was hoping people that want stuff like this would use a RSS reader. :)
 Some day I will try to use such a beast...

You have Firefox, don't you? Next time you surf to gitweb, right click on 
the funny yellow symbol in the lower right corner of your Firefox. It 
should say something like Subscribe to Do it.

Then, in you bookmark menu, you should see the entry, with a small arrow 
to the right. Click on the entry...

Ciao,
Dscho

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


Re: gitweb - feature request

2005-08-09 Thread Sam Ravnborg
 You have Firefox, don't you? Next time you surf to gitweb, right click on 
 the funny yellow symbol in the lower right corner of your Firefox. It 
 should say something like Subscribe to Do it.

Unfortunately not on my firefox. 1.0.6 on gentoo.
Puzzled...

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


Re: gitweb - feature request

2005-08-09 Thread John Benes
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Sam Ravnborg wrote:
 You have Firefox, don't you? Next time you surf to gitweb, right click on 
 the funny yellow symbol in the lower right corner of your Firefox. It 
 should say something like Subscribe to Do it.
 
 Unfortunately not on my firefox. 1.0.6 on gentoo.
 Puzzled...

I don't see it either with Firefox/1.0+ ID:2005080905.

For it to work, this code needs to be added to the HEAD (slightly modified)

link rel=alternate type=application/rss+xml title=GIT Trees
href=http://www.kernel.org/git/?a=opml; /

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.1 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iQIVAwUBQvkXWGF0oWcU9kCNAQLNiA//aeblC39Vlc6z6e3GjG84+9V/vPcXjrE9
MdCllzb9D5aITJSWTxK7Ow5Y1nat0g9Cnf1AgrG6wPT6Uvm+7j61EqH5rcjYx/mU
kNe+251cHHBnGohdXyV4n20KCoEnXKZOQX7p7R3dJWifR4eyUzacSQiA51WmNPIl
vkV+JpTD6lPAIcZxJMXOJWUblaDA0reMPoCQ2nz7gqZOsQP7qWvzi0uyIAxjhaPd
AQ5UzEE9l9Wt5J7Oyx/ldz9i5e54EZxFgkaDWlLpDwVpUChmlVBx5V1ltTRh3Ymp
7lCPfy7JFN7eIQpAKp2/rldxvWK2gobSErTJR6nfs3ohSxe6WPLEpVS5Bfxrjo+8
a0LUOi/9E2+Kj1ja1Wf165GkQlI7ZfQgrmFRm8pVWS9WwkjlDvZbHjArv4lg6X6w
02yGuiyT2P0MIhia7SJF7xFM+lUi8mDdT/psnrZQ1++UhckuGPwWTa0EJ8nVWfk2
KkN3G3R1RVdSSgQSlh71ngNzwyFmxvMjHAZHB5ETtiO+/Of6TMkW+o3S8oJrbJSh
9M7WjDlKSUl7R3NcBE2iqx9NUyz7ldwPme+iG/w3aTAsvNaMOgjzsVloEosVeDIh
I/GOOujl+ym2IiJsAX9YRyaKmcBTZGEjfnUNWhf14hMDeM5fB5N8P6f+6JsMOB/N
s8U1L/LUiCc=
=ai+1
-END PGP SIGNATURE-
-
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: gitweb - feature request

2005-08-09 Thread Kay Sievers
On Tue, Aug 09, 2005 at 03:51:37PM -0500, John Benes wrote:
 Sam Ravnborg wrote:
  You have Firefox, don't you? Next time you surf to gitweb, right click on 
  the funny yellow symbol in the lower right corner of your Firefox. It 
  should say something like Subscribe to Do it.
  
  Unfortunately not on my firefox. 1.0.6 on gentoo.
  Puzzled...
 
 I don't see it either with Firefox/1.0+ ID:2005080905.
 
 For it to work, this code needs to be added to the HEAD (slightly modified)
 
 link rel=alternate type=application/rss+xml title=GIT Trees
 href=http://www.kernel.org/git/?a=opml; /

Are you sure that firefox handles opml without a plugin?

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


Re: gitweb - feature request

2005-08-09 Thread Linus Torvalds


On Tue, 9 Aug 2005, Johannes Schindelin wrote:
 
 You have Firefox, don't you? Next time you surf to gitweb, right click on 
 the funny yellow symbol in the lower right corner of your Firefox. It 
 should say something like Subscribe to Do it.

Left-click. And you need to be inside the project you want to rss (it 
would be nice if you could be at the projects page and it would give you 
a list of things to subscribe to, but that may not be possible).

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