git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [GSoC] Convert “git stash” to builtin proposal

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Mar 23, 2018, 17:06 UTC
Message-ID
<nycvar.QRO.7.76.6.1803231754390.77@ZVAVAG-6OXH6DA.rhebcr.pbec.zvpebfbsg.pbz>
In-Reply-To
<1521760546.11809.20.camel@gmail.com>
Hi Paul-Sebastian,
On Fri, 23 Mar 2018, Paul-Sebastian Ungureanu wrote:
Show 26 quoted lines
> On Tue, 2018-03-20 at 23:08 +0100, Christian Couder wrote:
> > Hi,
> > 
> > On Tue, Mar 20, 2018 at 9:09 PM, Paul-Sebastian Ungureanu
> > <ungureanupaulsebastian@gmail.com> wrote:
> > > 
> > > * Convert function: this step is basically makes up the goal of
> > > this
> > > project.
> > 
> > Could you explain a bit more how you would convert a function? Or
> > could you explain for example how you would convert "git stash list"
> > below?
> 
> In order to convert a command, all the functions which are used by the
> command must be converted first. The conversion will start with the
> bottom-level functions, which do not have any dependencies.
> 
> For example, to convert "git stash list", the parser will call
> “list_stash”, which will call “have_stash”. The conversion of these
> functions will be made in reverse order they were mentioned (have_stash
> first and then list_stash).
> 
> It is very important to know the Git source well in order to avoid
> reimplementing functionality. In this case “have_stash()” is somehow
> already implemented as “get_oid(ref_stash, &obj)”. 
Very good
> [... proposal ...]

This is a pretty good proposal. The initial draft at converting `stash list` is a good start (it will need to be converted to avoid spawning an extra process, but that is something we can do incrementally, together).

Ciao, Johannes

Previous: Paul-Sebastian UngureanuNext: Paul-Sebastian Ungureanu
Message 4 of 11 in “[GSoC] Convert “git stash” to builtin proposal”
  1. Paul-Sebastian UngureanuMar 20, 2018
  2. Christian CouderMar 20, 2018
  3. Paul-Sebastian UngureanuMar 22, 2018
  4. Johannes SchindelinMar 23, 2018
  5. Paul-Sebastian UngureanuMar 25, 2018
  6. Christian CouderMar 23, 2018
  7. Paul-Sebastian UngureanuMar 24, 2018
  8. Christian CouderMar 25, 2018
  9. Paul-Sebastian UngureanuMar 25, 2018
  10. Johannes SchindelinMar 26, 2018
  11. Christian CouderMar 27, 2018

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.