Skip to content

Make repoPathRegex more permissive.#4438

Closed
mstange wants to merge 1 commit into
firefox-devtools:mainfrom
mstange:permissive-regex-for-git-rev
Closed

Make repoPathRegex more permissive.#4438
mstange wants to merge 1 commit into
firefox-devtools:mainfrom
mstange:permissive-regex-for-git-rev

Conversation

@mstange

@mstange mstange commented Jan 20, 2023

Copy link
Copy Markdown
Contributor

It will now accept "revisions" which are git tag names. For example: git:github.com/torvalds/linux:arch/x86/mm/fault.c:v5.15
In this example, the tag name / "revision" is "v5.15". The CORS-URL for this file would then be
https://raw.githubusercontent.com/torvalds/linux/v5.15/arch/x86/mm/fault.c

I'm not planning to use this capability, but maybe it'll be useful for somebody at some point.

I was initially planning to use it for kernel sources in samply, but then I noticed that Ubuntu has its own patches on top of the original kernel source, so I'll need to get the source from somewhere else after all.

It will now accept "revisions" which are git tag names. For example:
git:github.com/torvalds/linux:arch/x86/mm/fault.c:v5.15
In this example, the tag name / "revision" is "v5.15".
The CORS-URL for this file would then be
https://raw.githubusercontent.com/torvalds/linux/v5.15/arch/x86/mm/fault.c
@mstange mstange requested a review from canova January 20, 2023 21:40
@julienw

julienw commented Jan 20, 2023

Copy link
Copy Markdown
Contributor

image

It's so painful since we moved to the GHE-based environment :-(

@mstange mstange closed this Jan 20, 2023
@mstange

mstange commented Jan 20, 2023

Copy link
Copy Markdown
Contributor Author

Ugh, I logged out and back in and re-authed and then closed this PR and reopened it as #4439 but it still doesn't work.

@julienw

julienw commented Jan 23, 2023

Copy link
Copy Markdown
Contributor

It looks like the other one worked finally...

@mstange

mstange commented Jan 23, 2023

Copy link
Copy Markdown
Contributor Author

Did it?

@julienw

julienw commented Jan 23, 2023

Copy link
Copy Markdown
Contributor

Yeah, I see an error clearly, but the individual tasks were run. Not sure how to make sense out of that!

@mstange

mstange commented Jan 23, 2023

Copy link
Copy Markdown
Contributor Author

I've force pushed to rebase, and now the error message is gone.

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.

2 participants