<div dir="ltr">Brian, <div><br></div><div>From what I understand of the intentions for the component submodule (most pull requests will not touch it), I think you're correct.</div><div><br></div><div><i><span style="font-family:arial,sans-serif;font-size:12.571428298950195px">Even forcing people to do "git submodule init && git</span><br style="font-family:arial,sans-serif;font-size:12.571428298950195px">
<span style="font-family:arial,sans-serif;font-size:12.571428298950195px">submodule update" adds friction.</span></i><br></div><div><i><span style="font-family:arial,sans-serif;font-size:12.571428298950195px"><br></span></i></div>
<div style><font face="arial, sans-serif"><span style="font-size:12.571428298950195px">Tell them to do: </span></font></div><div style><font face="arial, sans-serif"><span style="font-size:12.571428298950195px"><br></span></font></div>
<div style><font face="arial, sans-serif"><span style="font-size:12.571428298950195px">git clone --recursive ipython/ipython # this does the submodule init/update bits for you</span></font></div><div style><font face="arial, sans-serif"><span style="font-size:12.571428298950195px"><br>
</span></font></div><div style><font face="arial, sans-serif"><span style="font-size:12.571428298950195px">Submodules add an unfriendly level of complexity, but it turns out that version control is a hard problem :/</span></font></div>
<div style><font face="arial, sans-serif"><span style="font-size:12.571428298950195px"><br></span></font></div><div style><font face="arial, sans-serif"><span style="font-size:12.571428298950195px">A</span></font></div><div style>
<font face="arial, sans-serif"><span style="font-size:12.571428298950195px"><br></span></font></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 2, 2013 at 7:00 PM, Brian Granger <span dir="ltr"><<a href="mailto:ellisonbg@gmail.com" target="_blank">ellisonbg@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I don't think using relative submodule urls is a good idea.  A common<br>
GitHub work flow when people start the contribute to a project is:<br>
<br>
1, Fork ipython/ipython<br>
2. Create a local clone of username/ipython<br>
3. Branch and push to username/ipython<br>
4. Create a PR<br>
<br>
This completely common workflow would require new devs to muck with<br>
the submodule url or also fork our submodules.  I don't see that as<br>
acceptable as we want to minimize the friction for new people to<br>
contribute.  Even forcing people to do "git submodule init && git<br>
submodule update" adds friction.<br>
<br>
So unless I am misunderstanding somethign, I am -1 on relative urls.<br>
<br>
Cheers,<br>
<br>
Brian<br>
<div class="im HOEnZb"><br>
On Thu, May 2, 2013 at 10:48 AM, MinRK <<a href="mailto:benjaminrk@gmail.com">benjaminrk@gmail.com</a>> wrote:<br>
</div><div class="HOEnZb"><div class="h5">> Maybe I am overestimating how many users actually want IPython to be<br>
> installable from their own fork, rather than only ever being valid after<br>
> merging into master.  I don't mean to say that maintaining separate<br>
> `components` is difficult, but it is tedious and unintuitive.<br>
><br>
> Let's give relative URL a shot for now, and revisit the issue if there are<br>
> actual problems (not counting the one I ran into when trying to write<br>
> exactly this PR).<br>
><br>
><br>
> On Thu, May 2, 2013 at 10:42 AM, W. Trevor King <<a href="mailto:wking@tremily.us">wking@tremily.us</a>> wrote:<br>
>><br>
>> On Thu, May 02, 2013 at 06:15:11PM +0100, Aron Ahmadia wrote:<br>
>> > Just wanted to point out that I found out how to use submodules from<br>
>> > your pull request to pycuda.  Thanks!<br>
>><br>
>> Glad to help :).<br>
>><br>
>> On Thu, May 02, 2013 at 10:28:51AM -0700, MinRK wrote:<br>
>> > Mirroring is a better case, since it shouldn't actually affect the<br>
>> > repo.  But it seems pretty terrible that under normal circumstances,<br>
>> > no clone of any IPython repo other than the official one or a proper<br>
>> > careful mirror will actually be functional.<br>
>><br>
>> I think “proper careful mirror” is a bit alarmist ;).  If it's just a<br>
>> single level of submodules (and it's currently just a single<br>
>> `components` submodule), all you have to do is mirror the submodules<br>
>> at the same place.  For me:<br>
>><br>
>>   $ cd /var/git<br>
>>   $ git clone git://<a href="http://github.com/ipython/ipython.git" target="_blank">github.com/ipython/ipython.git</a><br>
>>   $ git clone git://<a href="http://github.com/ipython/ipython-components.git" target="_blank">github.com/ipython/ipython-components.git</a><br>
>><br>
>> Working mirror complete ;).<br>
>><br>
>> Cheers,<br>
>> Trevor<br>
>><br>
>> --<br>
>> This email may be signed or encrypted with GnuPG (<a href="http://www.gnupg.org" target="_blank">http://www.gnupg.org</a>).<br>
>> For more information, see <a href="http://en.wikipedia.org/wiki/Pretty_Good_Privacy" target="_blank">http://en.wikipedia.org/wiki/Pretty_Good_Privacy</a><br>
>><br>
>> _______________________________________________<br>
>> IPython-dev mailing list<br>
>> <a href="mailto:IPython-dev@scipy.org">IPython-dev@scipy.org</a><br>
>> <a href="http://mail.scipy.org/mailman/listinfo/ipython-dev" target="_blank">http://mail.scipy.org/mailman/listinfo/ipython-dev</a><br>
>><br>
><br>
><br>
> _______________________________________________<br>
> IPython-dev mailing list<br>
> <a href="mailto:IPython-dev@scipy.org">IPython-dev@scipy.org</a><br>
> <a href="http://mail.scipy.org/mailman/listinfo/ipython-dev" target="_blank">http://mail.scipy.org/mailman/listinfo/ipython-dev</a><br>
><br>
<br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Brian E. Granger<br>
Cal Poly State University, San Luis Obispo<br>
<a href="mailto:bgranger@calpoly.edu">bgranger@calpoly.edu</a> and <a href="mailto:ellisonbg@gmail.com">ellisonbg@gmail.com</a><br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
IPython-dev mailing list<br>
<a href="mailto:IPython-dev@scipy.org">IPython-dev@scipy.org</a><br>
<a href="http://mail.scipy.org/mailman/listinfo/ipython-dev" target="_blank">http://mail.scipy.org/mailman/listinfo/ipython-dev</a><br>
</div></div></blockquote></div><br></div>