Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Document that ./configure installs nvm and mason #526

Closed
miccolis opened this issue Oct 30, 2014 · 4 comments · Fixed by #533
Closed

Document that ./configure installs nvm and mason #526

miccolis opened this issue Oct 30, 2014 · 4 comments · Fixed by #533

Comments

@miccolis
Copy link
Contributor

It caught me off guard that running ./configure wrote things to my home directory. As I had a old install of mason it was throwing errors that didn't make sense, which lead me to notice this ~/.mason directory.

If this is going to remain in the ./configure script should it be clear the the user that it is writing to and looking for these directories?
@springmeyer
Copy link
Contributor

Ideally we could use a local mason if no ~/.mason already exists to avoid writing to ~. As far as nvm what about trying to remove the node.js build dependency altogether?

refs mapbox/mason#13

@mikemorris
Copy link
Contributor

Refs #511 #508 #487 #417 for changes around the Node.js dependency

@kkaefer
Copy link
Member

kkaefer commented Oct 31, 2014

We could add Mason as a submodule int .mason and always use that install.

@kkaefer
Copy link
Member

kkaefer commented Oct 31, 2014

Removed nvm/node use in 080c185

kkaefer added a commit that referenced this issue Oct 31, 2014
@kkaefer
use a local version of mason instead of writing into ~/.mason
03ae142 
fixes #526
@kkaefer kkaefer mentioned this issue Oct 31, 2014
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use a local version of Mason instead of writing into ~/.mason mapbox/mapbox-gl-native
4 participants
@springmeyer @kkaefer @miccolis @mikemorris