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

fix: Updated build and build:linux to specify -t for 22.2.0 #57

Merged
merged 1 commit into from
May 31, 2024

Conversation

bizob2828
Copy link
Contributor

@bizob2828 bizob2828 commented May 31, 2024

Follow up from #54

@bmacnaughton
Copy link
Contributor

thanks bob; i think the consistency is important.

@bmacnaughton bmacnaughton merged commit e41ee1b into Contrast-Security-Inc:main May 31, 2024
12 checks passed
@bmacnaughton
Copy link
Contributor

i will get a release out by monday.

@bizob2828
Copy link
Contributor Author

@bmacnaughton I see the 4.2.0 tag failed, was that something I did? let me know how i can help

@bmacnaughton
Copy link
Contributor

bmacnaughton commented Jun 4, 2024

if you have an m1 mac, you can try to debug it. i don't. when one of the guys here that has an m1 mac has a chance to get to it, we'll hopefully make some progress.

it looks like a problem with the version of the c++ compiler. i'm guessing it changed when github actions starting using m1 images for macos-latest.

@bizob2828
Copy link
Contributor Author

I have a mac m1. I'll put it on my list to look into over the next few weeks

@tough-griff
Copy link
Contributor

I have a mac m1. I'll put it on my list to look into over the next few weeks

Finally got a chance to dig in and fix the mac build--this is finally deployed!

@bizob2828
Copy link
Contributor Author

I have a mac m1. I'll put it on my list to look into over the next few weeks

Finally got a chance to dig in and fix the mac build--this is finally deployed!

Thanks Griff 🤜🏻

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.

3 participants