Upgrade webpack-glsl-loader. #737
Conversation
yarn.lock
Outdated
| @@ -697,6 +697,11 @@ | |||
| enabled "2.0.x" | |||
| kuler "^2.0.0" | |||
|
|
|||
| "@davcri/webpack-glsl-loader@^1.0.2": | |||
| version "1.0.2" | |||
| resolved "http://localhost:4873/@davcri/webpack-glsl-loader/-/webpack-glsl-loader-1.0.2.tgz#657b0919bc3f9ceb6bc4cfaee2ef7db8c18bcb61" | |||
There was a problem hiding this comment.
Ah oui, c'est à cause que j'utilise verdaccio, un cache npm local. Ça change rien, yarn (et autres npm/pnpm, etc.) ne regardent pas ce champ là pour la source. Sinon je peux rouler un script pour corriger l'URL. Comme tu préfères.
There was a problem hiding this comment.
Ça a l'air que ça change tout, dans le yarn.lock c'est localhost qu'il écrit! verdaccio ne devrait-il pas y mettre l'url upstream?
| @@ -117,6 +117,6 @@ | |||
| "webpack": "^4.43.0", | |||
| "webpack-cdn-plugin": "^3.3.1", | |||
| "webpack-cli": "^3.3.12", | |||
| "webpack-glsl-loader": "^1.0.1" | |||
| "@davcri/webpack-glsl-loader": "^1.0.2" | |||
There was a problem hiding this comment.
Why @davcri loader? It has only 7 weekly downloads. Would https://www.npmjs.com/package/ts-shader-loader do the same job and fix the issue this fixes?
There was a problem hiding this comment.
J'ai regardé le code, rien de louche. C'est un fork à partir de grieve/webpack-glsl-loader#9
https://www.npmjs.com/package/ts-shader-loader pourrait être une bonne option, j'ai pas trop d'opinion, j'ai pas d'expérience avec glsl.
yarn.lock
Outdated
| @@ -9683,6 +9678,11 @@ loader-utils@^2.0.0: | |||
| emojis-list "^3.0.0" | |||
| json5 "^2.1.2" | |||
|
|
|||
| loader-utils@^3.2.0: | |||
| version "3.2.1" | |||
| resolved "http://localhost:4873/loader-utils/-/loader-utils-3.2.1.tgz#4fb104b599daafd82ef3e1a41fb9265f87e1f576" | |||
There was a problem hiding this comment.
Ouais, je pensais pas que ça causait un problème, mais j'ai vu que ça plante dans CI alors j'ai corrigé les URL dans le push suivant. Je n'avais pas remarqué parce qu'habituellement, j'utilise pnpm au lieu de npm ou yarn et pnpm ne met même pas l'URL dans son lockfile, juste le hash (qui est pareil partout).
This was a very old dependency (not sure how it's actually used) but it was pulling in https://www.npmjs.com/package/path and https://www.npmjs.com/package/fs (which are actually node core modules).
On the road to a webpack 5 upgrade.