-
-
Notifications
You must be signed in to change notification settings - Fork 307
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
monolith https://win98icons.alexmeub.com/ -o test.html #357
Comments
What an interesting find! So, it looks like this file https://win98icons.alexmeub.com/win-icons.min.css refers this file https://win98icons.alexmeub.com/css_sprites.png more than 1000 times, means Monolith will try to embed the same file over 1000 times, and it makes the process run out of RAM (or just upsets it so much that it gives up and quits). Even if it did manage to save it as one file, using a browser to open it later would likely crash it, it'd be like a 100MB file at least. I'd say, let's try that page again when I add MHTML support, then it'll save it just once and reference multiple times, exactly like on the web. |
Found a similar issue! (many refs to a image by css) Very cool project! |
Yes, very nice project indeed! Now you mentioned MHTML, I cannot get around the conspiracy in my head that the name Monolith is somehow inspired by it :-) Oh and I see you just added
that makes dependency number 3 😄 |
It kind of just happened, with MHTML sounding similar. The name for the project (Monolith) was fitting, since it describes the result (one file, with everything in it), and also happens to contain all the letters of HTML in it. But with MHTML, it even ends up fitting "Monolithic HTML" well, pure luck.
|
Running monolith with in the following way:
RUST_BACKTRACE=full monolith https://win98icons.alexmeub.com/ -o test.html
gives me the following backtrace:
Let me know if you need more information!
The text was updated successfully, but these errors were encountered: