fix: unbreak the build for platforms w/o docker support #1294
No reviewers
Labels
No labels
FreeBSD
Kind/Breaking
Kind/Bug
Kind/Chore
Kind/DependencyUpdate
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/Security
Kind/Testing
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Reviewed
Confirmed
Reviewed
Duplicate
Reviewed
Invalid
Reviewed
Won't Fix
Status
Abandoned
Status
Blocked
Status
Need More Info
Windows
linux-powerpc64le
linux-riscv64
linux-s390x
run-end-to-end-tests
run-forgejo-tests
No milestone
No project
No assignees
4 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
forgejo/runner!1294
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "alexrp/runner:netbsd-fix"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Such as NetBSD.
There's a
forgejo-runner.x86_64-netbsdin the PR.Oops. Would you mind if I changed the
.gitignoreentry toforgejo-runner*? We cross-compile the runner for many targets, which is why we use binary names like that.71e0fc340d26320bbce9@alexrp wrote in #1294 (comment):
Makes sense to me.
Asking, because I currently don't have the capacity to investigate that myself right now: Would cross compiling Forgejo Runner on Linux help to prevent such issues?
26320bbce970249af0e5@aahlenst wrote in #1294 (comment):
Probably; that's what I do for Zig. All of these are built from
x86_64-linux: https://codeberg.org/ziglang/runner/releases/tag/v12.1.0-zsfSpecifically:
@alexrp Thanks for the fix!
@mfenniak Would you be on board with cross-compiling during CI, without actually publishing binaries?
I'd be interested in trying it -- without it being a commitment to support those platforms.
This kind of compilation error is easily avoidable during a PR if it's noticed, and we could at least hit that level of support. If other more complex situations come up where the value versus effort isn't as clear, we could discuss the situation in an informed way rather than blunder into breaking something.
sounds also good to me. should be documented too, so it's clear we only check for compile errors, but not for runtime breaking issues