{"thread":{"id":"24123","subject":"[GSoC update] Splitting gitweb & developing write functionalities","startedAt":"2010-06-15T20:03:30Z","lastAt":"2010-06-18T20:33:37Z","messageCount":7,"participants":["Pavan Kumar Sunkara","Petr Baudis","Jakub Narebski"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"143780","messageId":"AANLkTikapZa0N7xroxhdlmm7LUvlNp-7GZy7XeG-zprC@mail.gmail.com","threadId":"24123","inReplyTo":null,"subject":"[GSoC update] Splitting gitweb & developing write functionalities","fromName":"Pavan Kumar Sunkara","fromEmail":"pavan.sss1991@gmail.com","sentAt":"2010-06-15T20:03:30Z","receivedAt":"2010-06-15T20:03:30Z","isPatch":false,"sender":{"key":"pavan.sss1991@gmail.com","avatar":"https://avatars.githubusercontent.com/u/174703?v=4"},"body":"Hi,\n\nI know it's been late to send a GSOC update but here is mine for the\nfirst few weeks of GSOC coding period.\n\nPatch list:\n\ngitweb: Use @diff_opts while using format-patch [master]\n\ngitweb: Move static files into seperate subdirectory [next]\ngitweb: Set default destination directory for installing gitweb [next]\ngit-instaweb: Put httpd logs in a \"$httpd_only\" subdirectory [next]\ngit-instaweb: Configure it to work with new gitweb structure [next]\ngit-web--browse: Add support for google chrome and chromium [next]\nMakefile: Use $(sharedir)/gitweb for target 'install-gitweb' [next]\ngit-instaweb: Add option to reuse previous config file [next]\n\ngit-instaweb: Add support for --reuse-config using gitconfig [waiting\nfor Junio] http://kerneltrap.org/mailarchive/git/2010/6/3/31711\n\ngitweb: Create Gitweb::Git module [Work in Progress]\ngitweb: Create Gitweb::Config module [Work in Progress]\ngitweb: Create Gitweb::Request module [Work in Progress]\ngitweb: Create Gitweb::Escape module [Work in Progress]\ngitweb: Create Gitweb::View module [Work in Progress]\ngitweb: Create Gitweb::Util module [Work in Progress]\ngitweb: Create Gitweb::RepoConfig module [Work in Progress]\ngitweb: Create Gitweb::Format module [Work in Progress]\ngitweb: Create Gitweb::Parse module [Work in Progress]\n"},{"id":"143781","messageId":"20100615201720.GI3109@machine.or.cz","threadId":"24123","inReplyTo":"AANLkTikapZa0N7xroxhdlmm7LUvlNp-7GZy7XeG-zprC@mail.gmail.com","subject":"Re: [GSoC update] Splitting gitweb & developing write functionalities","fromName":"Petr Baudis","fromEmail":"pasky@suse.cz","sentAt":"2010-06-15T20:17:20Z","receivedAt":"2010-06-15T20:17:20Z","isPatch":false,"sender":{"key":"pasky@ucw.cz","avatar":"https://avatars.githubusercontent.com/u/18439?v=4"},"body":"Hi!\n\nOn Wed, Jun 16, 2010 at 01:33:30AM +0530, Pavan Kumar Sunkara wrote:\n> gitweb: Use @diff_opts while using format-patch [master]\n> \n> gitweb: Move static files into seperate subdirectory [next]\n> gitweb: Set default destination directory for installing gitweb [next]\n> git-instaweb: Put httpd logs in a \"$httpd_only\" subdirectory [next]\n> git-instaweb: Configure it to work with new gitweb structure [next]\n> git-web--browse: Add support for google chrome and chromium [next]\n> Makefile: Use $(sharedir)/gitweb for target 'install-gitweb' [next]\n> git-instaweb: Add option to reuse previous config file [next]\n\nGood job!\n\n> gitweb: Create Gitweb::Git module [Work in Progress]\n> gitweb: Create Gitweb::Config module [Work in Progress]\n> gitweb: Create Gitweb::Request module [Work in Progress]\n> gitweb: Create Gitweb::Escape module [Work in Progress]\n> gitweb: Create Gitweb::View module [Work in Progress]\n\n(Is this the Gitweb::HTML module? Why have you renamed it?)\n\n> gitweb: Create Gitweb::Util module [Work in Progress]\n> gitweb: Create Gitweb::RepoConfig module [Work in Progress]\n> gitweb: Create Gitweb::Format module [Work in Progress]\n> gitweb: Create Gitweb::Parse module [Work in Progress]\n\nOk, so we seem to be behind a bit now compared to the timeline? Perhaps\nno big deal but we should be careful; maybe you will have to also start\nbrewing some write-support patches while the structure and details of\nthe split modules is still undergoing discussion.\n\nKeep up the good work!\n\n-- \n\t\t\t\tPetr \"Pasky\" Baudis\nThe true meaning of life is to plant a tree under whose shade\nyou will never sit.\n"},{"id":"143782","messageId":"AANLkTinfJOOT8NJ942bB_Sil4sGNthpk9hxvA60Y_iGB@mail.gmail.com","threadId":"24123","inReplyTo":"20100615201720.GI3109@machine.or.cz","subject":"Re: [GSoC update] Splitting gitweb & developing write functionalities","fromName":"Pavan Kumar Sunkara","fromEmail":"pavan.sss1991@gmail.com","sentAt":"2010-06-15T20:33:50Z","receivedAt":"2010-06-15T20:33:50Z","isPatch":false,"sender":{"key":"pavan.sss1991@gmail.com","avatar":"https://avatars.githubusercontent.com/u/174703?v=4"},"body":"On Wed, Jun 16, 2010 at 1:47 AM, Petr Baudis <pasky@suse.cz> wrote:\n> Hi!\n>\n> On Wed, Jun 16, 2010 at 01:33:30AM +0530, Pavan Kumar Sunkara wrote:\n>> gitweb: Use @diff_opts while using format-patch [master]\n>>\n>> gitweb: Move static files into seperate subdirectory [next]\n>> gitweb: Set default destination directory for installing gitweb [next]\n>> git-instaweb: Put httpd logs in a \"$httpd_only\" subdirectory [next]\n>> git-instaweb: Configure it to work with new gitweb structure [next]\n>> git-web--browse: Add support for google chrome and chromium [next]\n>> Makefile: Use $(sharedir)/gitweb for target 'install-gitweb' [next]\n>> git-instaweb: Add option to reuse previous config file [next]\n>\n> Good job!\n>\n>> gitweb: Create Gitweb::Git module [Work in Progress]\n>> gitweb: Create Gitweb::Config module [Work in Progress]\n>> gitweb: Create Gitweb::Request module [Work in Progress]\n>> gitweb: Create Gitweb::Escape module [Work in Progress]\n>> gitweb: Create Gitweb::View module [Work in Progress]\n>\n> (Is this the Gitweb::HTML module? Why have you renamed it?)\nYeah I thought it would be better if I have this named as View rather\nthan HTML :)\n>\n>> gitweb: Create Gitweb::Util module [Work in Progress]\n>> gitweb: Create Gitweb::RepoConfig module [Work in Progress]\n>> gitweb: Create Gitweb::Format module [Work in Progress]\n>> gitweb: Create Gitweb::Parse module [Work in Progress]\n>\n> Ok, so we seem to be behind a bit now compared to the timeline? Perhaps\n> no big deal but we should be careful; maybe you will have to also start\n> brewing some write-support patches while the structure and details of\n> the split modules is still undergoing discussion.\n>\n> Keep up the good work!\n>\n\nThanks,\nI would like to start on the writing feature soon enough.\n\nPavan.\n"},{"id":"143784","messageId":"20100615205755.GJ3109@machine.or.cz","threadId":"24123","inReplyTo":"AANLkTinfJOOT8NJ942bB_Sil4sGNthpk9hxvA60Y_iGB@mail.gmail.com","subject":"Re: [GSoC update] Splitting gitweb & developing write functionalities","fromName":"Petr Baudis","fromEmail":"pasky@suse.cz","sentAt":"2010-06-15T20:57:55Z","receivedAt":"2010-06-15T20:57:55Z","isPatch":false,"sender":{"key":"pasky@ucw.cz","avatar":"https://avatars.githubusercontent.com/u/18439?v=4"},"body":"  Hi!\n\nOn Wed, Jun 16, 2010 at 02:03:50AM +0530, Pavan Kumar Sunkara wrote:\n> >> gitweb: Create Gitweb::View module [Work in Progress]\n> >\n> > (Is this the Gitweb::HTML module? Why have you renamed it?)\n> Yeah I thought it would be better if I have this named as View rather\n> than HTML :)\n\n  I'm just a bit surprised since I have thought we have agreed that\nGitweb::View is much more misleading name for what Gitweb::HTML actually\nis, hinting at a MVC structure that simply is not there at any level.\nYou are welcome to differ in opinion but then I would expect you to\ncontinue that discussion. :-)\n\n-- \n\t\t\t\tPetr \"Pasky\" Baudis\nThe true meaning of life is to plant a tree under whose shade\nyou will never sit.\n"},{"id":"143786","messageId":"AANLkTimJqFzuWI7EMnUuH9ap69KgL_op_yndWSfyg8hX@mail.gmail.com","threadId":"24123","inReplyTo":"20100615205755.GJ3109@machine.or.cz","subject":"Re: [GSoC update] Splitting gitweb & developing write functionalities","fromName":"Pavan Kumar Sunkara","fromEmail":"pavan.sss1991@gmail.com","sentAt":"2010-06-15T21:13:57Z","receivedAt":"2010-06-15T21:13:57Z","isPatch":false,"sender":{"key":"pavan.sss1991@gmail.com","avatar":"https://avatars.githubusercontent.com/u/174703?v=4"},"body":"On Wed, Jun 16, 2010 at 2:27 AM, Petr Baudis <pasky@suse.cz> wrote:\n>  Hi!\n>\n> On Wed, Jun 16, 2010 at 02:03:50AM +0530, Pavan Kumar Sunkara wrote:\n>> >> gitweb: Create Gitweb::View module [Work in Progress]\n>> >\n>> > (Is this the Gitweb::HTML module? Why have you renamed it?)\n>> Yeah I thought it would be better if I have this named as View rather\n>> than HTML :)\n>\n>  I'm just a bit surprised since I have thought we have agreed that\n> Gitweb::View is much more misleading name for what Gitweb::HTML actually\n> is, hinting at a MVC structure that simply is not there at any level.\n> You are welcome to differ in opinion but then I would expect you to\n> continue that discussion. :-)\n>\n\nYes but after a long thought, I decided that the name to be changed to\nView because this module contains all the subroutines regarding the\ngitweb viewing functions.\n\nThanks,\nPavan.\n"},{"id":"143814","messageId":"20100616073017.GK3109@machine.or.cz","threadId":"24123","inReplyTo":"AANLkTimJqFzuWI7EMnUuH9ap69KgL_op_yndWSfyg8hX@mail.gmail.com","subject":"Re: [GSoC update] Splitting gitweb & developing write functionalities","fromName":"Petr Baudis","fromEmail":"pasky@suse.cz","sentAt":"2010-06-16T07:30:17Z","receivedAt":"2010-06-16T07:30:17Z","isPatch":false,"sender":{"key":"pasky@ucw.cz","avatar":"https://avatars.githubusercontent.com/u/18439?v=4"},"body":"On Wed, Jun 16, 2010 at 02:43:57AM +0530, Pavan Kumar Sunkara wrote:\n> Yes but after a long thought, I decided that the name to be changed to\n> View because this module contains all the subroutines regarding the\n> gitweb viewing functions.\n\nBut then you will need to continue the discussion in the respective\ntherad and present counter-arguments to our (my?) arguments, or you will\nget NAKs.\n\n(It's not like this is a totally crucial issue, but I'm now rather\nannoyed as what's the point of discussing stuff if it can be all\ndiscarded at a whim?)\n\n\t\t\t\tPetr \"Pasky\" Baudis\n"},{"id":"143919","messageId":"m3wrtwnktb.fsf@localhost.localdomain","threadId":"24123","inReplyTo":"20100616073017.GK3109@machine.or.cz","subject":"Re: [GSoC update] Splitting gitweb & developing write functionalities","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2010-06-18T20:33:37Z","receivedAt":"2010-06-18T20:33:37Z","isPatch":false,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"Petr Baudis <pasky@suse.cz> writes:\n> On Wed, Jun 16, 2010 at 02:43:57AM +0530, Pavan Kumar Sunkara wrote:\n\n> > Yes but after a long thought, I decided that the name to be changed to\n> > View because this module contains all the subroutines regarding the\n> > gitweb viewing functions.\n> \n> But then you will need to continue the discussion in the respective\n> therad and present counter-arguments to our (my?) arguments, or you will\n> get NAKs.\n> \n> (It's not like this is a totally crucial issue, but I'm now rather\n> annoyed as what's the point of discussing stuff if it can be all\n> discarded at a whim?)\n\nOn the other hand gitweb output is not limited to HTML only: it\nincludes text/plain (\"raw\" blob aka 'blob_plain' for text files,\n'patches' / 'patch', 'projects_index'), feed formats: OPML, RSS, Atom,\nand binary output (\"raw\" blob for binary files like images,\n'snapshot').\n\nBut I don't know if Gitweb::View includes subroutines related to all\nthose things.\n\n-- \nJakub Narebski\nPoland\nShadeHawk on #git\n"}]}