@all, have gone ahead with `apache-brooklyn-cli`, I'll update when its
merged and accessible.

On Mon, 23 May 2016 at 12:16 Thomas Bouron <[email protected]>
wrote:

> Hi John.
>
> As stated in Gitter, I +1 for `apache-brooklyn-cli` as the name of the brew
> package.
> Looking forward to use it!
>
> Best.
>
> On Mon, 23 May 2016 at 10:55 John McCabe <[email protected]> wrote:
>
> > Thanks Richard, will aim to proceed with ApacheBrooklynCli, so to flow on
> > OSX would be:
> >
> >   # brew install apache-brooklyn-cli
> >   # br login http://localhost:8081
> >   # etc..
> >
> > Will hold off committing the change for the moment to give others a
> chance
> > to comment.
> >
> > On Mon, 23 May 2016, 10:45 Richard Downer, <[email protected]> wrote:
> >
> > > John,
> > >
> > > On 23 May 2016 at 10:34, John McCabe <[email protected]> wrote:
> > >
> > > > Naming of the package within Brew *only*, it does not impact the
> > > resulting
> > > > binary naming at all:
> > > >
> > > > # brew install apache-brooklyn-cli
> > > > # br login http://localhost:8081
> > >
> > >
> > > Excellent - my concerns were misplaced :-)
> > >
> > >
> > > > *Maintainers:*
> > > > Brew don't appear to have support for a maintainer tag, but adding a
> > > > maintainer in a comment is feasible.
> > > >
> > > > What maintainer address would you recommend?
> > > >
> > >
> > > No need to worry. If it's not common practice in the Brew world to
> name a
> > > maintainer (like it is in the Debian world, for example), then just
> > comply
> > > with their existing practices.
> > >
> > > I would prefer that at [1] you name the class `ApacheBrooklyn` instead
> of
> > > `Brooklyn` (but the naming of private variables inside the class is
> fine
> > as
> > > it is).
> > >
> > > Thanks for responding to my concerns! I have no other objections to
> this,
> > > so go ahead.
> > >
> > > Cheers
> > > Richard.
> > >
> > >
> > > [1]
> > >
> > >
> >
> https://github.com/Homebrew/homebrew-core/pull/1326/files#diff-476062d25b84e180693641a30b797a5aR5
> > >
> >
> --
>
> Thomas Bouron • Software Engineer @ Cloudsoft Corporation •
> http://www.cloudsoftcorp.com/
> Github: https://github.com/tbouron
> Twitter: https://twitter.com/eltibouron
>

Reply via email to