-
Notifications
You must be signed in to change notification settings - Fork 81
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
Warning for missing source map files #366
Comments
Many thanks for pointing it out. It does not make sense to include the sourcemap when the source is not included. In the next release, the source maps will be removed from the npm package. I think that should resolve the issue. A note to myself, it needs to be done for rx-stomp and ng2-stompjs as well. |
Please try the newly released 6.1.1, it should resolve the issue. |
I have tried it and now I get a similar but different message: ` WARNING Compiled with 15 warnings 08:28:36 warning in ./node_modules/@stomp/stompjs/esm6/augment-websocket.js Module Warning (from ./node_modules/source-map-loader/index.js): @ ./node_modules/@stomp/stompjs/esm6/stomp-handler.js 6:0-55 207:12-28 warning in ./node_modules/@stomp/stompjs/esm6/byte.js Module Warning (from ./node_modules/source-map-loader/index.js): @ ./node_modules/@stomp/stompjs/esm6/stomp-handler.js 1:0-30 165:41-45 warning in ./node_modules/@stomp/stompjs/esm6/client.js Module Warning (from ./node_modules/source-map-loader/index.js): @ ./node_modules/@stomp/stompjs/esm6/index.js 1:0-25 1:0-25 warning in ./node_modules/@stomp/stompjs/esm6/compatibility/compat-client.js Module Warning (from ./node_modules/source-map-loader/index.js): @ ./node_modules/@stomp/stompjs/esm6/index.js 10:0-46 10:0-46 warning in ./node_modules/@stomp/stompjs/esm6/frame-impl.js Module Warning (from ./node_modules/source-map-loader/index.js): @ ./node_modules/@stomp/stompjs/esm6/index.js 2:0-29 2:0-29 warning in ./node_modules/@stomp/stompjs/esm6/compatibility/heartbeat-info.js Module Warning (from ./node_modules/source-map-loader/index.js): @ ./node_modules/@stomp/stompjs/esm6/compatibility/compat-client.js 2:0-49 27:34-47 warning in ./node_modules/@stomp/stompjs/esm6/index.js Module Warning (from ./node_modules/source-map-loader/index.js): @ ./src/main/webapp/app/shared/stomp/components/StompSessionProvider.tsx 4:0-40 19:28-34 warning in ./node_modules/@stomp/stompjs/esm6/parser.js Module Warning (from ./node_modules/source-map-loader/index.js): @ ./node_modules/@stomp/stompjs/esm6/index.js 3:0-25 3:0-25 warning in ./node_modules/@stomp/stompjs/esm6/stomp-config.js Module Warning (from ./node_modules/source-map-loader/index.js): @ ./node_modules/@stomp/stompjs/esm6/index.js 4:0-31 4:0-31 warning in ./node_modules/@stomp/stompjs/esm6/stomp-handler.js Module Warning (from ./node_modules/source-map-loader/index.js): @ ./node_modules/@stomp/stompjs/esm6/client.js 10:0-47 212:37-49 warning in ./node_modules/@stomp/stompjs/esm6/stomp-headers.js Module Warning (from ./node_modules/source-map-loader/index.js): @ ./node_modules/@stomp/stompjs/esm6/index.js 5:0-32 5:0-32 warning in ./node_modules/@stomp/stompjs/esm6/stomp-subscription.js Module Warning (from ./node_modules/source-map-loader/index.js): @ ./node_modules/@stomp/stompjs/esm6/index.js 6:0-37 6:0-37 warning in ./node_modules/@stomp/stompjs/esm6/compatibility/stomp.js Module Warning (from ./node_modules/source-map-loader/index.js): @ ./node_modules/@stomp/stompjs/esm6/index.js 11:0-38 11:0-38 warning in ./node_modules/@stomp/stompjs/esm6/types.js Module Warning (from ./node_modules/source-map-loader/index.js): @ ./node_modules/@stomp/stompjs/esm6/client.js 11:0-61 95:21-36 150:30-45 171:27-42 179:26-41 246:39-54 250:42-57 306:31-46 307:38-53 310:30-45 316:46-62 324:34-49 warning in ./node_modules/@stomp/stompjs/esm6/versions.js Module Warning (from ./node_modules/source-map-loader/index.js): @ ./node_modules/@stomp/stompjs/esm6/index.js 8:0-27 8:0-27 ⚠ 「wdm」: 1609 modules WARNING in ./node_modules/@stomp/stompjs/esm6/augment-websocket.js WARNING in ./node_modules/@stomp/stompjs/esm6/byte.js WARNING in ./node_modules/@stomp/stompjs/esm6/client.js WARNING in ./node_modules/@stomp/stompjs/esm6/compatibility/compat-client.js WARNING in ./node_modules/@stomp/stompjs/esm6/frame-impl.js WARNING in ./node_modules/@stomp/stompjs/esm6/compatibility/heartbeat-info.js WARNING in ./node_modules/@stomp/stompjs/esm6/index.js WARNING in ./node_modules/@stomp/stompjs/esm6/parser.js WARNING in ./node_modules/@stomp/stompjs/esm6/stomp-config.js WARNING in ./node_modules/@stomp/stompjs/esm6/stomp-handler.js WARNING in ./node_modules/@stomp/stompjs/esm6/stomp-headers.js WARNING in ./node_modules/@stomp/stompjs/esm6/stomp-subscription.js WARNING in ./node_modules/@stomp/stompjs/esm6/compatibility/stomp.js WARNING in ./node_modules/@stomp/stompjs/esm6/types.js WARNING in ./node_modules/@stomp/stompjs/esm6/versions.js Now it's complaining that the SourceMap file is not found. |
Thanks for pointing it out, I am reopening this issue. |
I have pushed v6.1.2 - this version includes both source-maps and sources. Please check if it resolves the issue. |
In v7, the following changes will happen:
|
Hello,
I'm getting warning when I run
npm start
like thisWhen I look into the source map files I see references to folder
../src/client.ts
as mentioned in the warning but they don't seem to be part of the npm distribution. In mynode_modules
I have only the foldersbundles
andesm6
, thesrc
folder which I see part of the github source seems not to be included when I install stompjs usingnpm install
.Is there a way to get also the source files included or to remove them from the source map files (i.e.
client.js.map
)?Thanks!
The text was updated successfully, but these errors were encountered: