Add clone information to BUILD.md

[bdemers]

Adds git clone step to BUILD.md, to ensure that the submodule step is not missed

[bdemers]

*NOTE: There are a few locations that do point out the sub modules, but not all. For example coming into the project from the "build on a Mac" path there is no mention of cloning the repo.
(This isn't necessarily bad, but missing submodule step may be a common issue for folks that don't typically deal with them)

[auturgy]

Thanks Brian: useful feedback. Would it be better to find a way to define the entry point into the wiki a bit better for new devs, rather than increase duplication (for maintainability)? Also, as you’ve likely observed build.md needs a bit of a general refresh :)

…

On 9 Aug 2020, at 8:09 am, Brian Demers ***@***.***> wrote:  *NOTE: There are a few locations that do point out the sub modules, but not all. For example coming into the project from the "build on a Mac" path there is no mention of cloning the repo. (This isn't necessarily bad, but missing submodule step may be a common issue for folks that don't typically deal with them) — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe.

[peterbarker]

Looks like a reasonable change.

Commit message needs to be fixed to something like BUILD_MD: add instruction to clone repository as first step to meet our guidelines.

Let me know if you'd like me to change that commit message for you; I don't think the github interface makes that bit easy.

[bdemers]

@peterbarker done!

[Hwurzburg]

@peterbarker why do we recommend repeatedly elsewhere to use git submodule update --init --recursive after a normal clone...instead of --recursive or --recurse-submodules during the clone?

[peterbarker]

On Sun, 9 Aug 2020, Henry Wurzburg wrote: @peterbarker why do we recommend repeatedly elsewhere to use git submodule update --init --recursive after a normal clone...instead of --recursive or --recurse-submodules during the clone?

The only reason I can think of is that the separate operation is useful after you've got a clone, when you're trying to bring everything up-to-date. Perhaps a long-term sort of a hint.

[peterbarker]

Merged, thanks!