From: Junio C Hamano <junkio@cox.net>
To: Linus Torvalds <torvalds@osdl.org>
Cc: Daniel Barkalow <barkalow@iabervon.org>,
git@vger.kernel.org, Petr Baudis <pasky@ucw.cz>
Subject: Re: Complete http-pull; where should it go?
Date: Sun, 01 May 2005 14:25:40 -0700 [thread overview]
Message-ID: <7voebuskob.fsf@assigned-by-dhcp.cox.net> (raw)
In-Reply-To: <Pine.LNX.4.58.0505011337070.2296@ppc970.osdl.org> (Linus Torvalds's message of "Sun, 1 May 2005 13:46:37 -0700 (PDT)")
>>>>> "LT" == Linus Torvalds <torvalds@osdl.org> writes:
LT> On Sun, 1 May 2005, Daniel Barkalow wrote:
>>
>> Right; what I want to make programs able to do is take explicit
>> references, instead of only taking the objects they reference. So you
>> say heads/master or heads/linus instead of
>> "198b0fb635ed8a007bac0c16eab112c5e2c7995c".
LT> That's fine.
LT> This is really just an issue of havign a function that does "get_sha1()",
LT> and then making the things that take command line arguments use that
LT> one instead of "get_sha1_hex()".
LT> Then you can have rules like:
LT> - if it's a hex number, take it
LT> - if it's a filename, look it up
LT> - if ".git/refs + str is a filename, look it up.
LT> The _only_ thing I want to be careful about is that all the _internal_
LT> stuff still have to use the strict "get_sha1_hex()" function, ie we should
LT> never _ever_ accept a tree object where the "sha1" ends up being anything
LT> but the hex thing. So this "generalized get_sha1()" would have to be used
LT> only on real user input (ie argv[] array and the like).
I agree with Linus. Dan, you are welcome to lift the code from
the find-in-vicinity() function in jit-util.c in my JIT toolset,
found at http://members.cox.net/junkio/.
prev parent reply other threads:[~2005-05-01 21:32 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-05-01 19:29 Complete http-pull; where should it go? Daniel Barkalow
2005-05-01 19:44 ` Linus Torvalds
2005-05-01 20:30 ` Daniel Barkalow
2005-05-01 20:46 ` Linus Torvalds
2005-05-01 21:25 ` Daniel Barkalow
2005-05-01 23:23 ` Linus Torvalds
2005-05-01 23:46 ` Daniel Barkalow
2005-05-01 23:48 ` Linus Torvalds
2005-05-02 0:09 ` Junio C Hamano
2005-05-02 1:50 ` Linus Torvalds
2005-05-02 2:03 ` Junio C Hamano
2005-05-02 2:43 ` Linus Torvalds
2005-05-02 2:31 ` Daniel Barkalow
2005-05-01 21:25 ` Junio C Hamano [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=7voebuskob.fsf@assigned-by-dhcp.cox.net \
--to=junkio@cox.net \
--cc=barkalow@iabervon.org \
--cc=git@vger.kernel.org \
--cc=pasky@ucw.cz \
--cc=torvalds@osdl.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox