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

Add Carbon Mod to Rust #115

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Add Carbon Mod to Rust #115

wants to merge 9 commits into from

Conversation

eumario
Copy link

@eumario eumario commented Jun 3, 2023

This PR follows the same methodology for installing Oxide, for the Carbon modding system for Rust. Same options are implemented for Carbon, as well as safety checks to ensure that one or the other mod systems are enabled, not both at the same time.

Initial Add: Carbon Support for Rust Server.
Ensure that server is safeguarded from having both Oxide and Carbon installed, Also added Update on Boot option for Carbon.
Removed Environment.sh check, as we just need to see if Carbon.dll is in it's location.
Added check to ensure that we aren't trying to enable both Oxide and Carbon at the same time.
For Production Only, Added Carbon Branch for Preview, and Edge.  Only compatible with Production Rust, will not work on Staging, AUX01 or AUX02 releases of Rust.
Fixed Typos in RUST_CARBON_BUILD
Re-worked Logic behind selecting Rust Branch, and Carbon Branch.
Fixed Mis-named Environment Variable in script, now should properly see it.
Updated documentation to reflect the addition of Carbon to the Docker image.
Copy link

@mthsmb mthsmb left a comment

Choose a reason for hiding this comment

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

Yes baby, LGTM

@ervins0x
Copy link

ervins0x commented Sep 6, 2023

When this is planned to get merged?

@eberdt
Copy link

eberdt commented Nov 27, 2023

@Dids @eumario
Is there a chance to merge this request? I think the feature would be a super plus and lgtm

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