laipeng/reflex
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Resolve npm path and fnm path on Windows (#2015)

Browse Source
This commit is contained in:
Masen Furer 2023-10-23 08:59:48 -07:00 committed by GitHub
parent d00425276d
commit 91bbf91c52
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
reflex/utils/prerequisites.py
View File

@ -96,7 +96,7 @@ def get_install_package_manager() -> str | None:
"""
# On Windows, we use npm instead of bun.
if constants.IS_WINDOWS:
return path_ops.get_npm_path()
return get_package_manager()
# On other platforms, we use bun.
return get_config().bun_path
@ -109,7 +109,10 @@ def get_package_manager() -> str | None:
Returns:
The path to the package manager.
"""
return path_ops.get_npm_path()
npm_path = path_ops.get_npm_path()
if npm_path is not None:
npm_path = str(Path(npm_path).resolve())
return npm_path
def get_app(reload: bool = False) -> ModuleType:
@ -424,11 +427,13 @@ def install_node():
if constants.IS_WINDOWS:
# Install node
fnm_exe = Path(constants.Fnm.EXE).resolve()
fnm_dir = Path(constants.Fnm.DIR).resolve()
process = processes.new_process(
[
"powershell",
"-Command",
f'& "{constants.Fnm.EXE}" install {constants.Node.VERSION} --fnm-dir "{constants.Fnm.DIR}"',
f'& "{fnm_exe}" install {constants.Node.VERSION} --fnm-dir "{fnm_dir}"',
],
)
else: # All other platforms (Linux, MacOS).