On Mon, Oct 05, 2015 at 01:49:09PM -0700, Jonathan Nieder wrote: > Remi Pommarel wrote: > > > Signed-off-by: Remi Pommarel <repk@xxxxxxxxxxxx> > > For what it's worth, with or without the tweaks below, > > Reviewed-by: Jonathan Nieder <jrnieder@xxxxxxxxx> Thanks. > > [...] > > --- a/Makefile > > +++ b/Makefile > > @@ -39,6 +39,10 @@ all:: > > # Define CURLDIR=/foo/bar if your curl header and library files are in > > # /foo/bar/include and /foo/bar/lib directories. > > # > > +# Define CURL_CONFIG to curl's configuration program that gives informations > > +# about the library (ie curl's version, cflags, ...) . If not set it defaults > > +# to 'curl-config'. > > Extra spaces, some wording tweaks: > > s/gives informations about the library/prints information about the library/ > s/(ie curl's version, cflags, .../) /(e.g., its version number)/ > s/If not set it defaults to/The default is/ > Do I need to resend a v4 patch with these modifications ? I am not sure about the correct workflow here. -- Rémi -- 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