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

Re: [RFC PATCH] git-p4: add option to store files in Git LFS on import

From
Luke Diamand <luke@diamand.org>
Date
Aug 30, 2015, 16:36 UTC
Message-ID
<CAE5ih7-w5ZjO=QUE45yBvwyqdqKCADczrLt7=4W8X6BR+cy4PA@mail.gmail.com>
In-Reply-To
<9951C805-DFBF-47AC-B215-B9C657D4A852@gmail.com>
On 30 August 2015 at 11:18, Lars Schneider <larsxschneider@gmail.com> wrote:
> Thanks for your feedback!
>
> I like the “handle big files” plugin kind of idea. However, I wonder if it makes sense to put more and more stuff into git-p4.py (>3000 LOC already). What do you think about splitting git-p4 into multiple files?

I was wondering about that. I think for now, the simplicity of keeping everything in one file is worth the slight extra pain. I don't imagine that the big-file-handler code would be very large.

>
> Regarding Python 3:
> Would you drop Python 2 support or do you want to support Python 2/3 in parallel? I would prefer the former…
For quite some time we would need to support both; we can't just have
a release of git that one day breaks git-p4 for people stuck on Python
2. But it might not be that hard to support both (though converting
all those print statements could be quite tiresome).
Previous: Lars SchneiderNext: Lars Schneider
Message 6 of 9 in “git-p4: add option to store files in Git LFS on import”
  1. git-p4: add option to store files in Git LFS on importlarsxschneider@gmail.com, Aug 28, 2015
  2. git-p4: add option to store files in Git LFS on importlarsxschneider@gmail.com, Aug 28, 2015
  3. Luke DiamandAug 30, 2015
  4. Luke DiamandAug 30, 2015
  5. Lars SchneiderAug 30, 2015
  6. Luke DiamandAug 30, 2015
  7. Lars SchneiderSep 3, 2015
  8. Luke DiamandSep 3, 2015
  9. John KeepingSep 3, 2015

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.