Vinod, are there patches out for these? I would like to help close this off.
On Tue, Feb 16, 2016 at 6:30 PM, Vinod Kone <vinodk...@apache.org> wrote:
> Thanks for the feedback. I'll send out a patch.
>
> On Tue, Feb 16, 2016 at 9:12 AM, haosdent <haosd...@gmail.com> wrote:
>
> > >running `find /usr/local/bin -name "*-*" | wc -l` returned 142 while
> `find
> > /usr/local/bin -name "*_*" | wc -l` was only 17. So I feel using hyphen
> for
> > executables is more standard.
> >
> > +1 Same for me.
> >
> > On Wed, Feb 17, 2016 at 1:10 AM, Alexander Rojas <
> alexan...@mesosphere.io>
> > wrote:
> >
> > > +1 for consistency, +1 for executables.
> > >
> > > I do enough finger yoga while using emacs!
> > >
> > > running `find /usr/local/bin -name "*-*" | wc -l` returned 142 while
> > `find
> > > /usr/local/bin -name "*_*" | wc -l` was only 17. So I feel using hyphen
> > for
> > > executables is more standard.
> > >
> > > > On 11 Feb 2016, at 14:58, Michael Park <mp...@apache.org> wrote:
> > > >
> > > > +1 for consistency, +1 for hyphens for executables.
> > > >
> > > > On 11 February 2016 at 14:25, Kevin Klues <klue...@gmail.com> wrote:
> > > >
> > > >> I typically think of files having dashes as binaries or scripts that
> > > >> are runnable, whereas files with underscores are meant as source or
> > > >> otherwise supplementary to the binary produced (e.g. a supplementary
> > > >> python library that the main python program imports). I'm not sure
> > > >> where I inherited this convention from, but it's always been the way
> > > >> I've done things.
> > > >>
> > > >> As far as our code base goes, we seem to use this convention as well
> > > >> with our mesos-master.sh. mesos-slave.sh, etc. binaries.
> > > >>
> > > >> On Thu, Feb 11, 2016 at 2:17 PM, Vinod Kone <vinodk...@apache.org>
> > > wrote:
> > > >>> Why hyphens? Most of the files in our repo use underscores. I would
> > > like
> > > >> us
> > > >>> to be consistent on how we name files in the repo.
> > > >>>
> > > >>> On Thu, Feb 11, 2016 at 1:40 PM, Kevin Klues <klue...@gmail.com>
> > > wrote:
> > > >>>
> > > >>>> I prefer hyphens as well....
> > > >>>>
> > > >>>> On Thu, Feb 11, 2016 at 1:28 PM, Jojy Varghese <
> j...@mesosphere.io>
> > > >> wrote:
> > > >>>>> hyphen++. Is google friendly apparently. Also less keys to press
> > :)
> > > >>>>>
> > > >>>>> -Jojy
> > > >>>>>
> > > >>>>>
> > > >>>>>
> > > >>>>>> On Feb 11, 2016, at 12:43 PM, Greg Mann <g...@mesosphere.io>
> > wrote:
> > > >>>>>>
> > > >>>>>> +1
> > > >>>>>>
> > > >>>>>> On Thu, Feb 11, 2016 at 11:41 AM, Vinod Kone <
> > vinodk...@apache.org>
> > > >>>> wrote:
> > > >>>>>>
> > > >>>>>>> Some the scripts in the "support" directory have dashes ("-")
> in
> > > >> their
> > > >>>>>>> names (e.g., apply-review.sh, apply-reviews.py), whereas some
> > have
> > > >>>>>>> underscores ("_") (e.g., docker_build.sh, mesos_split.py).
> > > >>>>>>>
> > > >>>>>>> This is really confusing and we should stick with one style. I
> > > >> propose
> > > >>>> to
> > > >>>>>>> change all them to use underscores. I will make sure the CI
> jobs
> > > are
> > > >>>>>>> updated accordingly.
> > > >>>>>>>
> > > >>>>>>> Any objections?
> > > >>>>>>>
> > > >>>>>>> Thanks,
> > > >>>>>>> Vinod
> > > >>>>>>>
> > > >>>>>
> > > >>>>
> > > >>>>
> > > >>>>
> > > >>>> --
> > > >>>> ~Kevin
> > > >>>>
> > > >>
> > > >>
> > > >>
> > > >> --
> > > >> ~Kevin
> > > >>
> > >
> > >
> >
> >
> > --
> > Best Regards,
> > Haosdent Huang
> >
>
