curl is needed for the download_script. There's probably many other prereqs (git/bash) that are required, but they're sort of obvious to system builders. So I could see removing it for that reason.
On Tue, Feb 7, 2017 at 6:51 PM, Jignesh Patel <jmp.quicks...@gmail.com> wrote: > Marc, > > Should we get rid of curl from the build instructions? We don’t need it > with the current instruction flow. > > Cheers, > Jignesh > > On 2/7/17, 6:43 PM, "Marc Spehlmann" <spehl.apa...@gmail.com> wrote: > > For example, here is the edited doc: > https://github.com/cramja/incubator-quickstep/blob/ > refactor-building/BUILDING.md#prerequisites > > On Tue, Feb 7, 2017 at 6:28 PM, Marc Spehlmann <spehl.apa...@gmail.com > > > wrote: > > > Any objections if I remove the 'Getting cmake' section? It seems > long, and > > like information that a database programmer would probably already > > know/google and quickly find out. > > > > Instead I can link to cmake's webpage in the prereq section > > > > On Tue, Feb 7, 2017 at 6:25 PM, Marc Spehlmann < > spehl.apa...@gmail.com> > > wrote: > > > >> Re-reading BUILDING.md, it seems like this doc might be a victim of > being > >> added to iteratively over time without consideration to the overall > >> structure. I am purposing to par down some of the language and > shift the > >> sections around in a flow that a novice quickstep user would > probably want: > >> > >> 0 a header with TOC > >> 1 basic building instructions > >> 2 common flags we use for quick troubleshooting > >> 3 List of advanced cmake flags > >> 4 appendix of other, uncommon features > >> - building on windows > >> - vagrant > >> > >> I'll make a PR with these changes, but does anyone have an initial > >> objection to the change in structure? > >> > > > > > > > >
