-
Notifications
You must be signed in to change notification settings - Fork 522
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
Bazel Angular example dev serve doesn't work on windows #1604
Comments
//cc @gregmagolan as I mentioned this offline yesterday. |
Okay, so most errors excluded |
Seems like there is something wrong when resolving paths; (Notice the last item)
|
In Windows `_/ts_scripts.js` will be expanded to `C:/Program Files/Git/ts_scripts.js`. ``` [ 'C:\\users\\alag\\_bazel_alag\\rj2iva3m\\external\\build_bazel_rules_nodejs\\internal\\node\\_node_bin\\node', 'C:\\users\\alag\\_bazel_alag\\rj2iva3m\\execroot\\examples_angular\\node_modules\\html-insert-assets\\src\\main.js', '--html=src/example/index.html', '--out=bazel-out/x64_windows-fastbuild/bin/src/index.html', '--roots=.', 'bazel-out/x64_windows-fastbuild/bin/src', '--assets', 'bazel-out/x64_windows-fastbuild/bin/src/styles.css', 'external/npm/node_modules/@angular/material/prebuilt-themes/deeppurple-amber.css', 'external/npm/node_modules/zone.js/dist/zone.min.js', './_C:/Program Files/Git/ts_scripts.js' ] ``` This seems like some character escaping issue but in general escaping is hell and this is a more straightforward fix. Partially addresses: #1604
In Windows `_/ts_scripts.js` will be expanded to `C:/Program Files/Git/ts_scripts.js`. ``` [ 'C:\\users\\alag\\_bazel_alag\\rj2iva3m\\external\\build_bazel_rules_nodejs\\internal\\node\\_node_bin\\node', 'C:\\users\\alag\\_bazel_alag\\rj2iva3m\\execroot\\examples_angular\\node_modules\\html-insert-assets\\src\\main.js', '--html=src/example/index.html', '--out=bazel-out/x64_windows-fastbuild/bin/src/index.html', '--roots=.', 'bazel-out/x64_windows-fastbuild/bin/src', '--assets', 'bazel-out/x64_windows-fastbuild/bin/src/styles.css', 'external/npm/node_modules/@angular/material/prebuilt-themes/deeppurple-amber.css', 'external/npm/node_modules/zone.js/dist/zone.min.js', './_C:/Program Files/Git/ts_scripts.js' ] ``` This seems like some character escaping issue but in general escaping is hell and this is a more straightforward fix. Partially addresses: #1604
`html-insert-assets` version `0.5.0` in includes the following fix: jbedard/html-insert-assets#6 which is needed for better Windows support. Closes bazel-contrib#1604
`html-insert-assets` version `0.5.0` in includes the following fix: jbedard/html-insert-assets#6 which is needed for better Windows support. Closes bazel-contrib#1604
In Windows `_/ts_scripts.js` will be expanded to `C:/Program Files/Git/ts_scripts.js`. ``` [ 'C:\\users\\alag\\_bazel_alag\\rj2iva3m\\external\\build_bazel_rules_nodejs\\internal\\node\\_node_bin\\node', 'C:\\users\\alag\\_bazel_alag\\rj2iva3m\\execroot\\examples_angular\\node_modules\\html-insert-assets\\src\\main.js', '--html=src/example/index.html', '--out=bazel-out/x64_windows-fastbuild/bin/src/index.html', '--roots=.', 'bazel-out/x64_windows-fastbuild/bin/src', '--assets', 'bazel-out/x64_windows-fastbuild/bin/src/styles.css', 'external/npm/node_modules/@angular/material/prebuilt-themes/deeppurple-amber.css', 'external/npm/node_modules/zone.js/dist/zone.min.js', './_C:/Program Files/Git/ts_scripts.js' ] ``` This seems like some character escaping issue but in general escaping is hell and this is a more straightforward fix. Partially addresses: #1604
`html-insert-assets` version `0.5.0` in includes the following fix: jbedard/html-insert-assets#6 which is needed for better Windows support. Closes bazel-contrib#1604
`html-insert-assets` version `0.5.0` in includes the following fix: jbedard/html-insert-assets#6 which is needed for better Windows support. Closes #1604
Still seems broken on windows..? Current version is using:
Everything (ibazel) seems to "serve" fine... but navigating to the provided URL "http://DESKTOP-4ID5F8I:5432/" - in my case is "blank"... |
@ubergeoff, unfortunately I no longer have a Windows machine to check why this has regressed. Maybe someone else on the team can take a look //cc @alexeagle. |
Anyone else - able to get this Angular example to run on Windows..? |
🐞 bug report
Is this a regression?
Yes
Description
https://github.com/bazelbuild/rules_nodejs/tree/master/examples/angular doesn't work on Windows
Run
ng serve
, not thatng serve --prod
has different kind of errors due to https://bazelbuild.slack.com/archives/GMADCN4ES/p1580896704106300🔬 Minimal Reproduction
Run
ng serve
in https://github.com/bazelbuild/rules_nodejs/tree/master/examples/angular using Windows as OS.🔥 Exception or Error
In browser console
🌍 Your Environment
Operating System: Windows
Output of
bazel version
:Anything else relevant?
It seems that most of the regressions comes from
html-insert-assets
package were with0.4.0
most of the above errors have been introduced. However with0.2.0
there is still the below error;The paths for
ts_scripts
seems to be none-the-less wronghtml-insert-assets 0.2.0
html-insert-assets 0.4.0
The text was updated successfully, but these errors were encountered: