~sircmpwn/dispatch.sr.ht#24: 
Handle .git postfix on source URLs when doing source replacement

See downstream issue for details: https://github.com/ziglang/zig/issues/1960

Status
REPORTED
Submitter
~andrewrk
Assigned to
No-one
Submitted
2 years ago
Updated
2 years ago
Labels
No labels applied.

~sircmpwn 2 years ago

Try updating your .build.yml to check out zig without the .git at the end of the URL.

~andrewrk 2 years ago

Thanks - that fixed it. I'll leave this issue open with the feature request that if this mistake is made, the CI fails with an error message telling how to fix the yml

~sircmpwn 2 years ago

Where should that error message go?

~sircmpwn REPORTED BY_DESIGN 2 years ago

Also, it's not always in error. Sometimes people submit build manifests that don't actually need to clone the repo they're a member of.

~andrewrk 2 years ago

The issue is that this is a poor user experience:

  • fill in a valid looking git URL into the sources field
  • enable the "pull request" dispatch
  • things appear to be working, but find out several weeks later that the "pull request -> build" dispatch hook is actually building master branch and not pull requests

Must I have a solution in mind for this to be an open todo item?

~sircmpwn 2 years ago

I suppose that the GItHub-specific integration can deal with the GitHub-specific .git postfixes

~sircmpwn BY_DESIGN REPORTED 2 years ago

Register here or Log in to comment, or comment via email.