dthree/vorpal

[for discussion] move default localStorage location under $HOME

Open

#122 geöffnet am 1. März 2016

Auf GitHub ansehen
 (11 Kommentare) (3 Reaktionen) (0 zugewiesene Personen)JavaScript (5.609 Stars) (313 Forks)batch import
bughelp wanted

Beschreibung

i ran into a problem when trying out your cash project. With my nodejs/npm setup, global installs require sudo: sudo npm i -g cash

however, because of that, cash doesn't have permissions to write to its own localstorage directory:

$ cash
fs.js:794
  return binding.mkdir(pathModule._makeLong(path),
                 ^

Error: EACCES: permission denied, mkdir '/Users/tenaciousmv/npm/lib/node_modules/cash/node_modules/vorpal/dist/.local_storage'

obviously i don't want to run cash with sudo :)

isn't it more typical to put this kind of storage in ~/.vorpal or ~/.cash or somewhere else under $HOME?

Contributor Guide

Tech Stack
javascriptnodejs
Domain
cli
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Node.jsvorpal frameworkbasic file system knowledge
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70
Research-Richtung
The issue suggests moving the default localStorage directory from inside the npm global install path to under the user's home directory (e.g., ~/.vorpal). To fix, locate the code in vorpal that sets the localStorage path (likely in the local storage.js or similar file in the dist folder). Change the default to use os.homedir() or process.env.HOME, and ensure the directory is created with appropriate permissions. Test the change by installing cash globally with sudo and running cash to verify no permission errors.