Skip to content

Strip .git extension from scoop source URLs #3383

New issue

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

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

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Mar 1, 2025

Conversation

SierraKomodo
Copy link
Contributor

  • I have read the contributing guidelines, and I agree with the Code of Conduct.
  • Have you checked that there aren't other open pull requests for the same changes?
  • Have you tested that the committed code can be executed without errors?
  • This PR is not composed of garbage changes used to farm GitHub activity to enter potential Crypto AirDrops.
    Any user suspected of farming GitHub activity with crypto purposes will get banned. Submitting broken code wastes the contributors' time, who have to spend their free time reviewing, fixing, and testing code that does not even compile breaks other features, or does not introduce any useful changes. I appreciate your understanding.

Uses regex to remove .git from the end of Scoop source URLs, so that generated manifest links in package information are valid.

The specific regex pattern ^(.*)\.git$ is used in case of urls in custom buckets having .git in the domain name. I.e., someguy.gitrepo.tld. The only part that needs to be removed here is the suffix if present.

Will be rendered redundant and should be reverted if ScoopInstaller/Scoop#6297 is addressed directly by Scoop.


Fixes #3367

Fixes marticliment#3367

Will be rendered redundant and should be reverted if
ScoopInstaller/Scoop#6297 is addressed
directly by Scoop.
@SierraKomodo SierraKomodo force-pushed the fix/scoop-source-urls branch from f4638c5 to d374431 Compare March 1, 2025 17:09
@SierraKomodo
Copy link
Contributor Author

Regex was slightly malformed yet still functioned. Funny but oops.

@marticliment
Copy link
Owner

Thanks!

I will test and merge later today.

@marticliment marticliment merged commit 2cd76b8 into marticliment:main Mar 1, 2025
2 checks passed
@SierraKomodo SierraKomodo deleted the fix/scoop-source-urls branch March 11, 2025 23:53
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.

[BUG] Manifest URLs to GitHub for scoop packages in details view is incorrect
2 participants