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

multi-entry plugin no such file error #488

Closed
aminya opened this issue Jul 7, 2020 · 14 comments
Closed

multi-entry plugin no such file error #488

aminya opened this issue Jul 7, 2020 · 14 comments

Comments

@aminya
Copy link

aminya commented Jul 7, 2020

  • Rollup Plugin Name: < multi-entry-->
  • Rollup Plugin Version: 3.0.1
  • Rollup Version: 2.20.0
  • Operating System (or Browser): Win 10
  • Node Version: 14.5.0

How Do We Reproduce?

In REPL.it https://repl.it/@aminya/x-terminal#rollup.config.js
Repository: https://github.com/aminya/x-terminal/tree/multientry

I could not reproduce it completely since the node version is 12. You need have 14.5.0, but here is the configuration:
https://repl.it/join/hgnuaflr-aminya

Expected Behavior

src/x-a.js, src/x-b.js → dist...
[!] Error: ENOENT: no such file or directory, open 'C:\project\x-a\dist\_rollup:plugin-multi-entry:entry-point.js.map'
Error: ENOENT: no such file or directory, open 'C:\prject\x-a\dist\_rollup:plugin-multi-entry:entry-point.js.map'

while dist folder exists

Actual Behavior

Bundling.

@shellscape
Copy link
Collaborator

Unfortunately we'll need a complete reproduction to triage this for you. I'd suggest making a repo with instructions for which node version to use in the README

@aminya
Copy link
Author

aminya commented Jul 8, 2020

@shellscape Here is the branch for you to try:
In REPL.it https://repl.it/@aminya/x-terminal#rollup.config.js
Repository: https://github.com/aminya/x-terminal/tree/multientry

If you want the configuration without multi entry, try rollup branch

@shellscape
Copy link
Collaborator

That's appreciated, but it doesn't fit the definition of minimal as mentioned in the issue template.

  1. Provide a minimal repository link (Read https://git.io/fNzHA for instructions).
    These may take more time to triage than the other options.

@aminya
Copy link
Author

aminya commented Jul 8, 2020

Could you check the link again? I removed the not-needed folders so you can just see what is needed.
https://repl.it/@aminya/x-terminal#rollup.config.js

@cyclops24
Copy link

cyclops24 commented Jul 23, 2020

@aminya @shellscape I'm also using plugin-multi-entry and it's working without any issue but It's creating a dummy file in this path: build/_virtual/_rollup:plugin-multi-entry:entry-point and because this file name contains : in its name, my users on windows can't use my library because : character is not allowed in windows OS. We don't have any issues with Linux and Mac.
I think the issue is related to this line:

const entry = '\0rollup:plugin-multi-entry:entry-point';

Any plan to fix this? I think choosing : character as a separator in the file name is a bad decision as it's not supported on other OS.

@frank-dspeed
Copy link
Contributor

frank-dspeed commented Jul 23, 2020

@shellscape watch the comment above me there we have a example of a bug where we can not even use the repl to reproduce that as the : char is simply not supported on windows the repl does not give us a option for that we should talk about how to handle that.

@Bluefinger
Copy link
Contributor

Bluefinger commented Jul 23, 2020

I can confirm I'm having this same issue with plugin-multi-entry, again on a Windows OS (Win 10). On my private project, I included multi-entry for the intent of forcing certain files to be provided on the bundle entry point, but with my build/watch process, I end up with this error:

{
  code: 'ERROR',
  error: [Error: ENOENT: no such file or directory, open 'E:\Code\JSHell\repayne\static\js\_rollup:plugin-multi-entry:entry-point.js.map'] {
    errno: -4058,
    code: 'ENOENT',
    syscall: 'open',
    path: 'E:\\Code\\JSHell\\repayne\\static\\js\\_rollup:plugin-multi-entry:entry-point.js.map'
  }
}
  • Rollup version: 2.22.2
  • Rollup Multi-Entry version: 3.0.1
  • Operating System: Windows 10
  • Node version: 14.4.0

EDIT: Rollup is able to generate the bundles to be dynamically imported, but the entry file is missing from the generated output. Perhaps there should be a way to override the entry point filename?

@cyclops24
Copy link

Related issue: #95

@aminya
Copy link
Author

aminya commented Jul 23, 2020

@aminya @shellscape I'm also using plugin-multi-entry and it's working without any issue but It's creating a dummy file in this path: build/_virtual/_rollup:plugin-multi-entry:entry-point and because this file name contains : in its name, my users on windows can't use my library because : character is not allowed in windows OS. We don't have any issues with Linux and Mac.
I think the issue is related to this line:

const entry = '\0rollup:plugin-multi-entry:entry-point';

Any plan to fix this? I think choosing : character as a separator in the file name is a bad decision as it's not supported on other OS.

Thank you! I am very happy that you found the reason! Now, I am sure they can fix it.

@frank-dspeed
You don't have to use Repl for this. There are many number of VM machines available that can help.

Now that the reason is found, we, the windows users, may be able to fix it

@frank-dspeed
Copy link
Contributor

@aminya the repl thing is a general thing here it is a internal none related discussion between me and @shellscape :)

@aminya
Copy link
Author

aminya commented Jul 24, 2020

@frank-dspeed I made a pull request. Check it out. You can use U+A789 instead of the Colon in Windows. But the name of the final bundle is now _rollup꞉plugin-multi-entry꞉entry-point.js. Is this the desired behavior?

@shellscape
Copy link
Collaborator

Fixed by #520

@cyclops24
Copy link

cyclops24 commented Aug 14, 2020

The multi-entry plugin still creates a file with : character like this: _virtual\_virtual:multi-entry.js so just file name changed but still we have redundant : character there.
I described more in the related PR: #520

@shellscape So please reopen this issue until an actual fix. 🙏

@shellscape
Copy link
Collaborator

@cyclops24 the right way to track this is with a new issue.

@rollup rollup deleted a comment from aminya Aug 14, 2020
@rollup rollup locked as resolved and limited conversation to collaborators Aug 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants