Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support XDG basedir specification #175

Closed
eheintzmann opened this issue Dec 12, 2012 · 4 comments · Fixed by #1570
Closed

Support XDG basedir specification #175

eheintzmann opened this issue Dec 12, 2012 · 4 comments · Fixed by #1570
Assignees

Comments

@eheintzmann
Copy link

Please support FreeDesktop.org XDG basedir specification for node-gyp

http://ploum.net/post/207-modify-your-application-to-use-xdg-folders
https://live.gnome.org/GnomeGoals/XDGConfigFolders
http://standards.freedesktop.org/basedir-spec/latest/

@TooTallNate
Copy link
Contributor

Seems reasonable. Someone should write a node-xdg module to kick things off...

@JayBeavers
Copy link
Contributor

Closing due to age and no takers.

@Siilwyn
Copy link
Contributor

Siilwyn commented Sep 22, 2018

@refack did you make progress on this?

I've done this for several modules using env-paths if this change can be made as a major version bump it will be straightforward to do and I'm willing to take this issue and #1124.

The relevant code is placed this right?

node-gyp/bin/node-gyp.js

Lines 19 to 23 in b2e5cf0

var homeDir = osenv.home()
if (prog.devDir) {
prog.devDir = prog.devDir.replace(/^~/, homeDir)
} else if (homeDir) {
prog.devDir = path.resolve(homeDir, '.node-gyp')

@Siilwyn
Copy link
Contributor

Siilwyn commented Oct 9, 2018

@rvagg thoughts? ☝️

rvagg pushed a commit that referenced this issue Apr 24, 2019
PR-URL: #1570
Fixes: #175
Fixes: #1124
Reviewed-By: Refael Ackermann <[email protected]>
gabylb pushed a commit to ibmruntimes/node-gyp that referenced this issue Oct 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants