Skip to content

Update Valhalla build, sed use explicit current dir? #742

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Merged
merged 5 commits into from
Dec 11, 2024
Merged

Update Valhalla build, sed use explicit current dir? #742

merged 5 commits into from
Dec 11, 2024

Conversation

rbygrave
Copy link
Contributor

@rbygrave rbygrave commented Dec 11, 2024 •
edited
Loading

sed in CI works a little differently?? Ah the old Mac OS/Darwin difference with -i ... dang it.

@rbygrave rbygrave merged commit 53b56a7 into master Dec 11, 2024
14 checks passed
@rbygrave rbygrave deleted the feature/update-valhalla-build branch December 11, 2024 21:22
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@SentryMan SentryMan SentryMan approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rbygrave @SentryMan