urls: prefer /archive/refs/tags urls over /archive

[chenrui333]

unblocks Homebrew/brew#16126

[Bo98]

+ many more - I didn't check them all, only A-D + LuaJIT.

As I said in Homebrew/brew#16126 (comment), this is wrong for commit URLs and the regex needs updating to exclude them (e.g. any that's exclusively [0-9a-f])

[chenrui333]

make sense

[chenrui333]

excluded all the commit sha tgz urls from the changeset (and updated the audit regex in the brew side)

[chenrui333]

• ran brew fetch for the changeset, no checksum mismatch found
• reverted some changes for repo_removed formulae (like neovide and nethacked)

[SMillerDev]

This really shouldn't all be in one commit.

[chenrui333]

This really shouldn't all be in one commit.

yeah, I was thinking about to do something similar to #139592

it is mostly style changes though, I can go either way.

[Bo98]

yeah, I was thinking about to do something similar to #139592

Plan was to rewrite history after that PR and it was useful at that point in time to have a single point in Git history that denoted the change, but history rewriting never happened in the end.

I wouldn't use that PR as precedence - it was a bit of a special PR that required temporarily changing repository settings to merge and doesn't match how we typically do brew style --fix PRs.

[chenrui333]

ok, let me spread the commits then :)

[chenrui333]

spreading the commits might too large for merge still, I will break down the PR alphabetically

[MikeMcQuaid]

spreading the commits might too large for merge still

why would it be?

[chenrui333]

spreading the commits might too large for merge still

why would it be?

it would be just 1800 commits and hard to review?

[chenrui333]

ok, I will provide two options :) (see how folks prefer)