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

Basic build instructions #12

Closed
wants to merge 1 commit into from
Closed

Conversation

itscynxx
Copy link

Tried running the instructions found from the GitHub workflow and that didn't work, asked in the Discord and got this which did work

Maybe wording could be better though?

@catornot catornot self-requested a review October 15, 2023 17:34
Copy link
Member

@catornot catornot left a comment

Choose a reason for hiding this comment

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

Still don't really know if this is needed


The Northstar RPC plugin uses [cargo](https://doc.rust-lang.org/cargo/index.html) to download dependencies and build code

Run `cargo build --release` in the root directory of the plugin. The output will be found in `target/release`.
Copy link
Member

@catornot catornot Oct 15, 2023

Choose a reason for hiding this comment

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

release is not required to build the plugin, it only disables the plugin from throwing errors and forces it to shutdown instead also ofc it optimizes it.

Copy link
Member

Choose a reason for hiding this comment

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

I though @itscynxx had issues with getting the plugin running without compiling as release? Or did I confuse something here? ^^"

Copy link
Member

Choose a reason for hiding this comment

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

cargo build should work fine. idk why someone would have issues unless they are building on linux :|

Copy link
Member

@GeckoEidechse GeckoEidechse Nov 13, 2023

Choose a reason for hiding this comment

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

So running cargo build worked just fine for me on Windows (on plugins v3).

@GeckoEidechse
Copy link
Member

Imma move this to post plugins v3

@itscynxx
Copy link
Author

Closing as isn't needed

@itscynxx itscynxx closed this Nov 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants