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

Re: [EGIT] [PATCH RFC v1 0/5] Add (static) ignore functionality to EGit

From
Shawn O. Pearce <spearce@spearce.org>
Date
Mar 30, 2009, 04:27 UTC
Message-ID
<20090330042732.GS23521@spearce.org>
In-Reply-To
<49CF50CC.10400@pelagic.nl>
"Ferry Huberts (Pelagic)" <ferry.huberts@pelagic.nl> wrote:
Show 7 quoted lines
> Robin Rosenberg wrote:
> > A quick reply (I might come up with more later): Ignore support should be mostly
> > in jgit, with only extensions into egit.
> > 
> I discussed this with shawn and proposed to first implement it in egit
> and when we have it right then move it into jgit. I think shawn agreed
> with that.
I may have agreed with it.  My memory isn't *that* good.  :-)

In general principal I agree with Robin, Git specific handling should be in JGit as much as possible so we can reuse the logic in more applications than just EGit.

But it may have been easier to get a first working prototype by doing the code in EGit, and later pulling some of it down into JGit as we identity what isn't EGit specific.

The problem with that is the dual licenses; code in EGit can't be pulled down to JGit without relicensing it under the BSD. Only the original author of the code can do that. So if you contribute ignore support to EGit under the EPL which is better placed in JGit, Robin or myself can't pull it down ourselves, we'd have to rewrite it.

But even rewriting may be difficult, as the rewrite may be too close to the original (same language, same surrounding code, likely going to produce a similar result).

-- 
Shawn.
Previous: Ferry Huberts (Pelagic)Next: Jonathan Gossage
Message 9 of 16 in “Add (static) ignore functionality to EGit”
  1. Ferry HubertsMar 26, 2009
  2. [EGIT] [PATCH RFC v1 1/5] Build up the ignore patterns cache upon workspace startup.Ferry Huberts, Mar 26, 2009
  3. [EGIT] [PATCH RFC v1 2/5] Enable the ignore handling of the pluginFerry Huberts, Mar 26, 2009
  4. [EGIT] [PATCH RFC v1 3/5] Optimise ignore evaluationFerry Huberts, Mar 26, 2009
  5. [EGIT] [PATCH RFC v1 4/5] Do not set .git as a Team ignore patternFerry Huberts, Mar 26, 2009
  6. [EGIT] [PATCH RFC v1 5/5] Use the ignore patterns cache to determine ignoresFerry Huberts, Mar 26, 2009
  7. Robin RosenbergMar 29, 2009
  8. Ferry Huberts (Pelagic)Mar 29, 2009
  9. Shawn O. PearceMar 30, 2009
  10. Jonathan GossageMar 30, 2009
  11. Robin RosenbergMar 30, 2009
  12. Shawn O. PearceApr 5, 2009
  13. Ferry Huberts (Pelagic)Apr 6, 2009
  14. Ferry Huberts (Pelagic)Apr 6, 2009
  15. Shawn O. PearceApr 6, 2009
  16. Sverre RabbelierApr 6, 2009

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.