Skip to content

Add CoRIM binary for diagnosis - #24

Open
labbott wants to merge 2 commits into
masterfrom
corim
Open

Add CoRIM binary for diagnosis#24
labbott wants to merge 2 commits into
masterfrom
corim

Conversation

@labbott

Copy link
Copy Markdown

No description provided.

@flihp

Copy link
Copy Markdown
Contributor

Saw this come in & took the opportunity to dust off my illumos VM to build / test. build.sh creates an IPS archive as expected. Installing get me the corim binary in /usr/bin. Running this binary works as expected. I've also verified the generated version has the right components: 0.1.0 from cargo + .20 for the current commit count: LGTM

Comment threadrats-corim/build.sh Outdated
fatal "Could not find GCC in any expected location"
fi
info "using $GCC_DIR/gcc: $($GCC_DIR/gcc --version | head -1)"
info "using $GCC_DIR/g++: $($GCC_DIR/g++ --version | head -1)"

@flihpflihpDec 16, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit: gcc / g++ isn't used to compile rats-corim so these checks aren't necessary (it builds fine w/o them for me)

@labbott
labbott requested a review from jclulowJanuary 13, 2026 20:57
Sign up for freeto 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.

2 participants

@labbott@flihp