ku1ik / git-dude

Git commit notifier
http://ku1ik.com/
GNU General Public License v3.0
948 stars 68 forks source link

Initial git-svn support #11

Closed ku1ik closed 12 years ago

ku1ik commented 13 years ago

Hey @ktoso, can you try this one with your git-svn repos?

ktoso commented 13 years ago

Seems like there is some issue with it:

fatal: ambiguous argument '712d4a5f04781583ccbdc90583b5fa7efb184a0b~..c3f6a93f07718c48f16f6bc52ee2b9dfaf44579e': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions
ku1ik commented 13 years ago

What public svn repo have you tried? Was initial fetch done by you or by git-dude?

ktoso commented 13 years ago

Hm hm... The initiall fetch was done by git-dude if I remember well.

I'll do some more testing tomorrow and let you know when exactly this happens.

ku1ik commented 13 years ago

I've added small improvement. Anyway I noticed that if initial fetch is done by dude then it throws the error. If you do it yourself first, then subsequent fetches by dude work fine.

ktoso commented 13 years ago

Confirmed, subsequent fetches work perfect.

ktoso commented 13 years ago

Any plans on merging this @sickill? :)

ku1ik commented 12 years ago

Still interested in merging this? If you don't use git-svn anymore then let's leave it here (for 2 reasons: it complicates the code and it's conflicted already with current master)

ku1ik commented 12 years ago

@ktoso ^^^

ktoso commented 12 years ago

Oh, I forgot about this. So...yeah, I've stopped using it with git-svn, as I'm not forced to use svn anymore :-)

We can leave it unmerged, no prob. If someone interested will dropby, they can just +1 or create new feature requests. So - leave it open or just close the issue, anything that suits you best :-)

ku1ik commented 12 years ago

Allrighty, closing. There's always option to re-open.