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 environment variable settings to download mirror #33

Closed
lovelope opened this issue Jan 29, 2024 · 6 comments
Closed

Support environment variable settings to download mirror #33

lovelope opened this issue Jan 29, 2024 · 6 comments
Labels
wontfix This will not be worked on

Comments

@lovelope
Copy link

Version
All
OS
macOS、Linux、Windows

Describe the bug
Like nvm, users can set the nodejs download mirror location.

NVM_NODEJS_ORG_MIRROR=https://npm.taobao.org/mirrors/node

Instead of adding additional plugins.
#24

@aooohan
Copy link
Member

aooohan commented Jan 29, 2024

I'm thinking about this too. Because vfox is extended in the form of a plugin. I'm thinking about which way is more friendly:

  1. Use environment variable settings as you mention.
  2. Use additional plugins to represent the corresponding mirrors.

Plan 1) needs to be a place to explain how to use, how to configure. Plan 2) program more intuitive, users do not need to care about how to configure, just need to install the corresponding plugin on it. But additional plugins are needed.

@yimiaoxiehou
Copy link

I like Plan 2). This tools not only for nodejs,i want't remember a lots of env args like (NODE_MIRROR, GOLANG_MIRROR, JDK_MIRROR).

@yimiaoxiehou
Copy link

yimiaoxiehou commented Jan 29, 2024

i hope can use vfox available to list mirror plugins .
it will implement this feature better.
image

vfox available can show flutter and flutter-cn now. maybe can add vfox available -u cmd to refresh list by version-fox-plugins

@yimiaoxiehou
Copy link

yimiaoxiehou commented Jan 29, 2024

i hope can use vfox available to list mirror plugins . it will implement this feature better. image

vfox available can show flutter and flutter-cn now. maybe can add vfox available -u cmd to refresh list by version-fox-plugins

em,vfox available can auto refresh by version-fox-plugins not need any arg。 😄
after my mirror plugin accept. it added in list.
image

@lovelope now, you can use vfox add nodejs/npmmirror --alias nodejs to use https://cdn.npmmirror.com (aliyun mirror) as node download source

image

@lovelope
Copy link
Author

LGTM

@aooohan
Copy link
Member

aooohan commented Jan 30, 2024

Looking at it this way, the original option 2 is more reasonable. ;)

@aooohan aooohan closed this as completed Jan 30, 2024
@aooohan aooohan added the wontfix This will not be worked on label Jan 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

3 participants