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

Fixed Icarus Crash Issue #121 #131

Merged
merged 2 commits into from
Feb 9, 2025
Merged

Conversation

BolverBlitz
Copy link
Contributor

Description

The egg crashed on startup when importing a VC DLL, removing vcrun2019 will stop the game from loading this file and removing winetricks all together makes it start much faster

Checklist for all submissions

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you tested and reviewed your changes with confidence that everything works?
  • Did you branch your changes and PR from that branch and not from your master branch?
    • If not, why?:
  • You verify that the start command applied does not use a shell script
    • If some script is needed then it is part of a current yolk or a PR to add one
  • The egg was exported from the panel

"meta": {
"version": "PTDL_v2",
"update_url": null
},
"exported_at": "2024-06-01T00:03:56+00:00",
"exported_at": "2025-02-05T17:40:05+01:00",
"name": "Icarus-Dedicated",
"author": "[email protected]",
"uuid": "7f2f0676-3d0c-457f-a3aa-b2af40966698",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check that, i do not have a pelican panel so i just copyed this in the exported file.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You messed up. And wrote the pterodactyl exported egg over the pelican one. Please fix.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not touch the once you do not have. We will take care about that.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not touch the once you do not have. We will take care about that.

So should i revert the file back to its original state or will you fix it?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yea just revert it.

@gOOvER
Copy link
Contributor

gOOvER commented Feb 5, 2025

The Egg works fine with vcrun2019, vcrun2022

@BolverBlitz
Copy link
Contributor Author

BolverBlitz commented Feb 5, 2025

The Egg works fine with vcrun2019, vcrun2022

Not on any of my pterodactyl nodes

Edit: Just vcrun2022 works but why install it when it works without that?

028c:trace:loaddll:build_module Loaded L"C:\\windows\\system32\\IPHLPAPI.DLL" at 00006FFFFB6A0000: builtin
028c:trace:loaddll:build_module Loaded L"C:\\windows\\system32\\SETUPAPI.dll" at 00006FFFFBC90000: builtin
028c:trace:loaddll:build_module Loaded L"C:\\windows\\system32\\dwmapi.dll" at 00006FFFF90F0000: builtin
028c:trace:loaddll:build_module Loaded L"C:\\windows\\system32\\jsproxy.dll" at 00006FFFF8F20000: builtin
028c:trace:loaddll:build_module Loaded L"C:\\windows\\system32\\WINHTTP.dll" at 00006FFFF8F90000: builtin
028c:err:module:import_dll Loading library MSVCP140.dll (which is needed by L"Z:\\home\\container\\Icarus\\Binaries\\Win64\\IcarusServer-Win64-Shipping.exe") failed (error c000007b).
028c:trace:loaddll:build_module Loaded L"C:\\windows\\system32\\VCRUNTIME140.dll" at 00006FFFFBEB0000: native
028c:trace:loaddll:build_module Loaded L"C:\\windows\\system32\\VCRUNTIME140_1.dll" at 00006FFFF8F00000: native
028c:err:module:loader_init Importing dlls for L"Z:\\home\\container\\Icarus\\Binaries\\Win64\\IcarusServer-Win64-Shipping.exe" failed, status c0000135
container@pterodactyl~ Server marked as offline...
[Pterodactyl Daemon]: ---------- Detected server process in a crashed state! ----------
[Pterodactyl Daemon]: Exit code: 53
[Pterodactyl Daemon]: Out of memory: false
[Pterodactyl Daemon]: Checking server disk space usage, this could take a few seconds...
[Pterodactyl Daemon]: Updating process configuration files...
[Pterodactyl Daemon]: Ensuring file permissions are set correctly, this could take a few seconds...

@gOOvER
Copy link
Contributor

gOOvER commented Feb 6, 2025

The Egg works fine with vcrun2019, vcrun2022

Not on any of my pterodactyl nodes

Edit: Just vcrun2022 works but why install it when it works without that?

028c:trace:loaddll:build_module Loaded L"C:\\windows\\system32\\IPHLPAPI.DLL" at 00006FFFFB6A0000: builtin
028c:trace:loaddll:build_module Loaded L"C:\\windows\\system32\\SETUPAPI.dll" at 00006FFFFBC90000: builtin
028c:trace:loaddll:build_module Loaded L"C:\\windows\\system32\\dwmapi.dll" at 00006FFFF90F0000: builtin
028c:trace:loaddll:build_module Loaded L"C:\\windows\\system32\\jsproxy.dll" at 00006FFFF8F20000: builtin
028c:trace:loaddll:build_module Loaded L"C:\\windows\\system32\\WINHTTP.dll" at 00006FFFF8F90000: builtin
028c:err:module:import_dll Loading library MSVCP140.dll (which is needed by L"Z:\\home\\container\\Icarus\\Binaries\\Win64\\IcarusServer-Win64-Shipping.exe") failed (error c000007b).
028c:trace:loaddll:build_module Loaded L"C:\\windows\\system32\\VCRUNTIME140.dll" at 00006FFFFBEB0000: native
028c:trace:loaddll:build_module Loaded L"C:\\windows\\system32\\VCRUNTIME140_1.dll" at 00006FFFF8F00000: native
028c:err:module:loader_init Importing dlls for L"Z:\\home\\container\\Icarus\\Binaries\\Win64\\IcarusServer-Win64-Shipping.exe" failed, status c0000135
container@pterodactyl~ Server marked as offline...
[Pterodactyl Daemon]: ---------- Detected server process in a crashed state! ----------
[Pterodactyl Daemon]: Exit code: 53
[Pterodactyl Daemon]: Out of memory: false
[Pterodactyl Daemon]: Checking server disk space usage, this could take a few seconds...
[Pterodactyl Daemon]: Updating process configuration files...
[Pterodactyl Daemon]: Ensuring file permissions are set correctly, this could take a few seconds...

Check, if the server needs vcruntimes. How you know, that ius not needed? Only , because server starts?

@parkervcp parkervcp mentioned this pull request Feb 9, 2025
6 tasks
Copy link
Contributor

@parkervcp parkervcp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have validated this change works

@parkervcp parkervcp merged commit 318cc16 into pelican-eggs:main Feb 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants