On Mon, Oct 05, 2015 at 01:49:09PM -0700, Jonathan Nieder wrote:
> Remi Pommarel wrote:
> 
> > Signed-off-by: Remi Pommarel <[email protected]>
> 
> For what it's worth, with or without the tweaks below,
> 
> Reviewed-by: Jonathan Nieder <[email protected]>

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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to