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.

Sign up for GitHub

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

Jump to bottom

webpack-dev-server 3.8 broken on windows 10 #4424

Closed
docteurzoidberg opened this issue Aug 12, 2019 · 9 comments
Closed

webpack-dev-server 3.8 broken on windows 10 #4424

docteurzoidberg opened this issue Aug 12, 2019 · 9 comments
Labels
needs reproduction This issue is missing a minimal runnable reproduction, provided by the author upstream

Comments

@docteurzoidberg
Copy link

Version

3.10.0

Reproduction link

[npm run serve](npm run serve)

Environment info

Environment Info:

  System:
    OS: Windows 10
    CPU: (4) x64 Intel(R) Core(TM) i5-7300U CPU @ 2.60GHz
  Binaries:
    Node: 10.16.1 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.17.3 - C:\Users\DrZoid\AppData\Roaming\npm\yarn.CMD
    npm: 6.10.3 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: 44.18362.1.0
  npmPackages:
    @vue/babel-helper-vue-jsx-merge-props:  1.0.0
    @vue/babel-plugin-transform-vue-jsx:  1.0.0
    @vue/babel-preset-app:  3.10.0
    @vue/babel-preset-jsx:  1.1.0
    @vue/babel-sugar-functional-vue:  1.0.0
    @vue/babel-sugar-inject-h:  1.0.0
    @vue/babel-sugar-v-model:  1.0.0
    @vue/babel-sugar-v-on:  1.1.0
    @vue/cli-overlay:  3.10.0
    @vue/cli-plugin-babel: ^3.10.0 => 3.10.0
    @vue/cli-plugin-eslint: ^3.10.0 => 3.10.0
    @vue/cli-service: ^3.10.0 => 3.10.0
    @vue/cli-shared-utils:  3.10.0
    @vue/component-compiler-utils:  2.6.0 (3.0.0)
    @vue/preload-webpack-plugin:  1.1.1
    @vue/web-component-wrapper:  1.2.0
    eslint-plugin-vue: ^5.0.0 => 5.2.3 (4.7.1)
    vue: ^2.6.10 => 2.6.10
    vue-eslint-parser:  5.0.0 (2.0.3)
    vue-hot-reload-api:  2.3.3
    vue-loader:  15.7.1
    vue-style-loader:  4.1.2
    vue-template-compiler: ^2.6.10 => 2.6.10
    vue-template-es2015-compiler:  1.9.1
  npmGlobalPackages:
    @vue/cli: Not Found

Steps to reproduce

"npm run serve" on newly create project with "vue create" (default options)

What is expected?

'npm run serve' should work

What is actually happening?

'npm run serve' give this error:

INFO Starting development server...
10% building 2/2 modules 0 active ERROR Error: Watching remote files is not supported.
Error: Watching remote files is not supported.
at Server.setupWatchStaticFeature

"it worked last time !"
@docteurzoidberg docteurzoidberg mentioned this issue Aug 12, 2019
Closed
2 tasks
@haoqunjiang
Copy link
Member

Can't reproduce. Please provide a reproduction repo.

And, what file system are you on? What's your project path (run pwd in the terminal and paste the output here)?

@haoqunjiang haoqunjiang added needs reproduction This issue is missing a minimal runnable reproduction, provided by the author upstream labels Aug 12, 2019
@docteurzoidberg
Copy link
Author

docteurzoidberg commented Aug 12, 2019 •
edited
Loading

pwd = c:\dev\some-repo (ntfs filesystem, nothing special)

Doesn't need a repo, i just created the app from vue-cli with nothing in it (vue cli did the npm install)

@docteurzoidberg
Copy link
Author

docteurzoidberg commented Aug 12, 2019 •
edited
Loading

oh no, don't tell me it's because of my "dev" folder :D

@docteurzoidberg
Copy link
Author

docteurzoidberg commented Aug 12, 2019 •
edited
Loading

It's the pwd thing ! i just did the project in another folder and it works flawlessly :o

@sodatea

@haoqunjiang
Copy link
Member

False-positive in https://github.com/sindresorhus/is-absolute-url (webpack-dev-server changed from testing /https?:/ to using this library in the 3.8 release)

Please report to that repository.

I believe it's because of the lower case c. But how come the lower case volume label? I thought it would always be UPPER CASE.

@elchinzadeh
Copy link

I removed space from the parent folder name and it worked without problem

@NateScarlet
Copy link

I thought it would always be UPPER CASE

Windows uses case insensitive path. c:\ is equivalent to C:\.

@docteurzoidberg
Copy link
Author

False-positive in https://github.com/sindresorhus/is-absolute-url (webpack-dev-server changed from testing /https?:/ to using this library in the 3.8 release)

Please report to that repository.

I believe it's because of the lower case c. But how come the lower case volume label? I thought it would always be UPPER CASE.

maybe vue-cli should downgrade the version of webpack-dev-server 3.8 to 3.7.2 until it's fixed in is-absolute-url ?

@viruscamp
Copy link

viruscamp commented Aug 16, 2019 •
edited
Loading

I found that VS Code debug launcher start node.exe with working folder which has lower case volume label.

I used ProcessHacker to inspect the process property of node.exe, and found when the Current Directory was 'e:/abc/proj', the issue happened, when is was 'E:/abc/proj', it was ok.
I ran 'npm run serve' from a standalone cmd, it was ok. Current Directory was 'E:/abc/proj'
I ran 'npm run serve' from an embedded cmd of VS Code, it was ok. Current Directory was 'E:/abc/proj'
I met the issue only I started debugging from VS Code debug launch. Current Directory was 'e:/abc/proj'

haoqunjiang added a commit to haoqunjiang/vue-cli that referenced this issue Jan 20, 2020
@haoqunjiang
feat: convert project name to lowercase and warn on creation
e504f43 
closes vuejs#2547
closes vuejs#5032

I'm still very hesitant on adding this feature, though.

First, this change allows project creation in a folder with uppercase
letters in its name. It is strongly discouraged and may cause many
weird issues all over the ecosystem.

For example, vuejs#5022, vuejs#4424, vuejs#3665, vuejs#4174#issuecomment-569709494 are all
caused by case issues. Adding support for uppercase project names will
only worsen this situation.

Secondly, it adds a lot of maintenance burden to us. As noted in the
comments, these prompts are hard to test right now (because
`createTestProject` runs in another process so it's hard to intercept
the prompts). Even if such test utilities are added in the future, it's
still very tedious to take care of all the case issues in the test
suite.

What's worse is that we can affect the project folders created by
@vue/cli by converting the project name to lower case. But for
`vue create .`, we cannot change the current folder's name. So, we'll
have another edge case to test.
@haoqunjiang haoqunjiang mentioned this issue Jan 20, 2020
Closed
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
needs reproduction This issue is missing a minimal runnable reproduction, provided by the author upstream
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@docteurzoidberg @haoqunjiang @viruscamp @NateScarlet @elchinzadeh