couchdb-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Randall Leeds <randall.le...@gmail.com>
Subject Re: [2/3] git commit: improve argument parsing in couchjs
Date Sun, 02 Oct 2011 21:13:57 GMT
Thanks you two. Pushed a cleanup to remove the short options and
s/install/enable/.
Updated the wiki documentation.

On Sun, Oct 2, 2011 at 11:23, Paul Davis <paul.joseph.davis@gmail.com>wrote:

> Right, I was saying we should change the help text so that it says
> enable so as to convey the intention of this option.
>
> By default the bindings are disabled so that they are guaranteed to
> not be available to scripts running map/reduce stuff. They only get
> enabled when running the Futon tests from the command line.
>
> On Sun, Oct 2, 2011 at 1:19 PM, Noah Slater <nslater@apache.org> wrote:
> > The help output says "install" so that's what I'm going on. My point
> being, either the cURL bindings are enabled at install time or they're not.
> I don't see why this command would let you change that, or "install" them.
> >
> > On 2 Oct 2011, at 19:14, Paul Davis wrote:
> >
> >> s/install/enable/ ?
> >>
> >> On Sun, Oct 2, 2011 at 1:02 PM, Noah Slater <nslater@apache.org> wrote:
> >>>
> >>> On 2 Oct 2011, at 18:55, Paul Davis wrote:
> >>>
> >>>> On Sun, Oct 2, 2011 at 12:18 PM, Noah Slater <nslater@apache.org>
> wrote:
> >>>>>
> >>>>> On 2 Oct 2011, at 12:14, randall@apache.org wrote:
> >>>>>
> >>>>>> +    "  -H, --http          install %s cURL bindings (only
> avaiable\n"
> >>>>>> +    "                      if package was built with cURL
> available)\n"
> >>>>>
> >>>>> What does this do?
> >>>>>
> >>>>>
> >>>>
> >>>> Enables the cURL bindings so couchjs can run the javascript tests from
> >>>> the command line.
> >>>
> >>> Why is this in the command line options to the script? Either they are
> enabled at install time, or they are not. A script like this should not be
> talking about "installing" anything. Right?
> >
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message