On Sat, Sep 21, 2013 at 6:56 PM, brian m. carlson
<sandals@xxxxxxxxxxxxxxxxxxxx> wrote:
> On Sat, Sep 21, 2013 at 05:52:05PM -0500, Felipe Contreras wrote:
>> On Sat, Sep 21, 2013 at 4:29 PM, brian m. carlson
>> <sandals@xxxxxxxxxxxxxxxxxxxx> wrote:
>> Now, if anybody has ideas into how the bindings could be more object
>> oriented, I'm all ears, but unfortunately what I foresee is that
>> nobody will consider this proposal seriously.
>
> My concern is that the Ruby code will end up not being idiomatic, and
> people will view it as bizarre and unmaintainable.
>
> for_each_ref could end up being something like REPOSITORY.refs.each,
> which would be more idiomatic. repository.refs would probably be an
> Enumerator in that case. If the decision is made to incorporate Ruby
> code, I'm happy to submit some patches to help provide a sane interface,
> even though I'm not that familiar with Ruby.
I think my proposed bindings are quite idiomatic.
git ruby - master origin/master <<EOF
commits = ARGV.map { |id| Git::Commit.get(get_sha1(id)) }
puts get_merge_bases(commits, 0).map { |commit| sha1_to_hex(commit.sha1) }
EOF
--
Felipe Contreras
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html