It looks great to me.
Thanks,
--
Jharrod LaFon
OpenEye Scientific Software
On Aug 19, 2013, at 2:54 PM, Junio C Hamano wrote:
> Jharrod LaFon writes:
>
>> I will keep trying this until it's perfect, and I thank you for
>> the help. When I resubmit this, would you like me to include your
>>
Jharrod LaFon writes:
> I will keep trying this until it's perfect, and I thank you for
> the help. When I resubmit this, would you like me to include your
> sign-off line as well?
If the one I attached at the end of the message you are responding
to looks fine to you, I'd just apply it without
On Mon, Aug 19, 2013 at 11:56:17AM -0700, Junio C Hamano wrote:
> Jharrod LaFon writes:
>
> > Updated the patch and the patch submission.
> >
>
> Getting a lot warmer ;-).
Thanks, I agree with all of the stuff you said. The end result that you
included looks good to me.
-Peff
--
To unsubscrib
Jharrod LaFon writes:
> Updated the patch and the patch submission.
>
Getting a lot warmer ;-).
> -- >8 --
>
> Git segmentation faults if submodule path is empty.
If this is meant to replace the MUA's Subject: line, then please add
"Subject: " prefix, like the example at the end of this messa
Updated the patch and the patch submission.
-- >8 --
Git segmentation faults if submodule path is empty.
Git fails due to a segmentation fault if a submodule path is empty.
Here is an example .gitmodules that will cause a segmentation fault:
[submodule "foo-module"]
path
On Fri, Aug 16, 2013 at 10:59:35AM -0700, Jharrod LaFon wrote:
> Here is an updated patch with a test.
Bits like this that should not be part of the commit message should
either go after the "---" lines near the diffstat, or should come before
a scissors line, like:
Here is my new patch.
--
Here is an updated patch with a test.
Git fails due to a segmentation fault if a submodule path is empty.
Here is an example .gitmodules that will cause a segmentation fault:
[submodule "foo-module"]
path
url = http://host/repo.git
$ git status
Segmentation fault (core dumped)
This occurs be
OK, I'll incorporate Jeff's changes, add a test and resubmit the patch.
Thanks,
--
Jharrod LaFon
OpenEye Scientific Software
On Aug 16, 2013, at 7:14 AM, Jeff King wrote:
> On Fri, Aug 16, 2013 at 09:09:58AM -0400, Jeff King wrote:
>
- if (parse_config_key(var, "submodule", &name, &nam
On Fri, Aug 16, 2013 at 09:09:58AM -0400, Jeff King wrote:
> > > - if (parse_config_key(var, "submodule", &name, &namelen, &key) < 0 ||
> > > !name)
> > > + if (parse_config_key(var, "submodule", &name, &namelen, &key) < 0 ||
> > > !name || !value)
> > > return 0;
>
> I think this is
On Fri, Aug 16, 2013 at 08:48:43AM +0200, Jens Lehmann wrote:
> > This patch addresses the issue by returning from the function if
> > 'value' is null before the call to xstrdup is made.
>
> Hmm, I'm not sure silently ignoring the misconfiguration is the best
> way to go. A submodule config havin
Jharrod LaFon writes:
> Git fails due to a segmentation fault if a submodule path is empty.
> Here is an example .gitmodules that will cause a segmentation fault:
> [submodule "foo-module"]
> path
> url = http://host/repo.git
> $ git status
> Segmentation fault (core dumped)
Can you turn
Am 16.08.2013 03:51, schrieb Jharrod LaFon:
> Git fails due to a segmentation fault if a submodule path is empty.
> Here is an example .gitmodules that will cause a segmentation fault:
> [submodule "foo-module"]
> path
> url = http://host/repo.git
> $ git status
> Segmentation fault (core d
Git fails due to a segmentation fault if a submodule path is empty.
Here is an example .gitmodules that will cause a segmentation fault:
[submodule "foo-module"]
path
url = http://host/repo.git
$ git status
Segmentation fault (core dumped)
This occurs because in the function parse_submodul
13 matches
Mail list logo