Nothing is converted when the file path contains a space
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 84/100
Piste de recherche
Lis JsModules.py en te concentrant sur JsModulesFileCommand et JsModulesDirCommand, où shell_cmd est construit à partir du chemin du fichier actif. Compare le tableau d’arguments cmd documenté par Sublime avec shell_cmd, puis vérifie que les deux commandes gèrent le chemin signalé contenant des espaces. Le travail est terminé lorsque le convertisseur trouve et convertit le fichier dans un chemin contenant des espaces et que les chemins ordinaires fonctionnent toujours.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Claude reports: on Windows the commands convert nothing when the file path contains a space, and the exec panel says "Found 0 files".
Tested with JsModules 0.0.2 on Sublime Text 4215 (Windows), cjs-to-es6 installed globally with npm. The same two-line file a.js (const x = require('y'); module.exports = x;) in two folders:
...\jsmok\a.js: "JsModules: Convert file to ES module" prints "Found 1 files ... 1 ok"....\jsm space dir\a.js: the console showsRunning cjs-to-es6 C:\...\jsm space dir\a.js, and the exec panel shows "Found 0 files." followed by "Error: You have to provide at least one file/directory to transform."
JsModules.py builds the command with "cjs-to-es6 {0}".format(active_file) and passes it as shell_cmd, so the path is split at the spaces. Both commands do this (JsModulesFileCommand and JsModulesDirCommand).
Running the same command through the exec command with the path as its own list item works for the folder with spaces:
exec {"cmd": ["cjs-to-es6", "C:\\...\\jsm space dir\\a.js"], "shell": true}
That printed "Found 1 files ... 1 ok" for the same file. Sublime's build system docs describe cmd as an array of arguments and shell_cmd as a string passed to the shell (https://www.sublimetext.com/docs/build_systems.html). Changing both commands to the cmd form (or quoting the path) should fix it.
- Langage dominant
- Python
- Étoiles
- 0
- Forks
- 0
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de luwes/SublimeJsModules
-
Difficulté 1/5 1-3 heures Accessibilité débutants 78/100
-
enhancement
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
Toutes les issues de luwes/SublimeJsModules
Issues similaires
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 72/100
MIT-LCP/wfdb-python#590 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 77/100
pollen-robotics/reachy_mini#1517 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 69/100
NRLMMD-GEOIPS/pluginify#41 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 67/100
scverse/spatialdata#1293 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
UKGovernmentBEIS/inspect_evals#2788 ·
Les mainteneurs répondent en général sous 1 jour