search

cvogt / cbt

CBT - fun, fast, intuitive, compositional, statically checked builds written in Scala
Other
488 stars 60 forks source link

clarification on project structure #568

Closed bbarker closed 6 years ago

bbarker commented 6 years ago

Does the change make sense?

cvogt commented 6 years ago

Sorry this fell through the cracks. I am personally not a fan of the package directory structure. Java requires it, but Scala doesn't and especially that leading chain e.g. org/cvogt/cbt/... just adds more folders to navigate. So I wouldn't want to advocate that. People are free to chose it if they like. So I'll close this. Hope that's ok.

bbarker commented 6 years ago

Hmm, it wasn't so much I was advocating it (quite the opposite) as I thought it was required with CBT when using the package hierarchy chain:

source in BUILDROOT/: OK source in BUILDROOT/src/ org/cvogt/cbt/: OK source in BUILDROOT/ org/cvogt/cbt/: NOT OK

On Tue, Dec 5, 2017 at 2:12 PM, Jan Christopher Vogt < notifications@github.com> wrote:

Closed #568 https://github.com/cvogt/cbt/pull/568.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/cvogt/cbt/pull/568#event-1373333389, or mute the thread https://github.com/notifications/unsubscribe-auth/AA37jpF27F5ubfmK-0YEoxompQRN3tqhks5s9ZWkgaJpZM4QiuvB .

-- Brandon Barker brandon.barker@gmail.com