-
Notifications
You must be signed in to change notification settings - Fork 266
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
ci: use consistent binary dest path naming #4772
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
Terraform Cloud Plan Output
|
Waiting to see what the CI output looks like |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Something went off here https://github.com/firezone/firezone/actions/runs/8822073323/job/24219388108#step:11:23
Performance Test ResultsTCP
UDP
|
CI is green now 🤞 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like it might work. That's a lot more places to put versions in, though.
cp ../target/release/firezone "$BINARY_DEST_PATH"-amd64 | ||
cp ../target/release/firezone-gui-client.dwp "$BINARY_DEST_PATH"-amd64.dwp | ||
cp ../target/release/bundle/deb/*_amd64.deb "$BINARY_DEST_PATH"_amd64.deb |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the deb having an underscore where the other ones had a hyphen was just a convention, and Debian won't actually care if we break that, so this should be okay.
No description provided.