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 <[email protected]>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 <[email protected]> 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 <[email protected]> wrote: > >>> > >>> On 2 Oct 2011, at 18:55, Paul Davis wrote: > >>> > >>>> On Sun, Oct 2, 2011 at 12:18 PM, Noah Slater <[email protected]> > wrote: > >>>>> > >>>>> On 2 Oct 2011, at 12:14, [email protected] 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? > > > > >
