On Fri, Aug 04, 2017 at 02:53:11PM -0700, Brandon Williams wrote:
> On 08/03, Stefan Beller wrote:
> > On Thu, Aug 3, 2017 at 11:19 AM, Brandon Williams <bmwill@xxxxxxxxxx> wrote:
> > > The function 'submodule_from_name()' is being used incorrectly here as a
> > > submodule path is being used instead of a submodule name. Since the
> > > correct function to use with a path to a submodule is already being used
> > > ('submodule_from_path()') let's remove the call to
> > > 'submodule_from_name()'.
> > >
> > > Signed-off-by: Brandon Williams <bmwill@xxxxxxxxxx>
> >
> > In case a reroll is needed, you could incorperate Jens feedback
> > stating that 851e18c385 should have done it.
>
> K I'll add that into the commit message.
Well, thats not 100% correct... IMO, it should have been a follow up patch
which I never got to implement. See my other reply to the v1 of this
patch I just sent out.
As stated there I will have a look into where it makes sense to pass a
commit id and behave more correctly.
Cheers Heiko
