perlalien/alien-build-plugin-download-github

Alien::Build plugin to download from GitHub

alien-build-plugin-download-github's Issues

Add some extra unit tests

That mock the download negotiator and fetch plugins.

slurp should use slurp_raw

https://github.com/PerlAlien/Alien-Build-Plugin-Download-GitHub/blob/main/lib/Alien/Build/Plugin/Download/GitHub.pm#L244

Sometimes doesn't properly decode json

https://gist.github.com/plicease/7cf0d4e282efe80c740e16ef6d148ef1

# Failed test 'live tests'
# at t/alien_build_plugin_download_github.t line 91.
# Caught exception in subtest: Can't use string ("[
#   {
#     "url": "https://api.gi"...) as an ARRAY ref while "strict refs" in use at /tmp/loop_over_bdir-7556-ioxMTR/Alien-Build-Plugin-Download-GitHub-0.02-0/blib/lib/Alien/Build/Plugin/Download/GitHub.pm line 64.
t/alien_build_plugin_download_github.t .. 
Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/2 subtests 

Add support for the release API

Right now we only allow fetching the raw git tag, which might not include things like configure etc.

Should be able to specify no extractor

For when the asset really is the thing that you need and it doesn't need to be further extracted! I think the GitLab version of this already has this.

Sometimes returns 403 Forbidden from GH

https://gist.github.com/plicease/f5464741dc69edc9c67e29988e6cfd61

# Failed test 'live tests'
# at t/alien_build_plugin_download_github.t line 91.
# Caught exception in subtest: error fetching https://api.github.com/repos/Perl5-Alien/dontpanic/releases: 403 Forbidden at /tmp/loop_over_bdir-14705-f0UB_Z/Alien-Build-1.63-0/blib/lib/Alien/Build/Plugin/Fetch/HTTPTiny.pm line 75.
t/alien_build_plugin_download_github.t .. 
Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/2 subtests 

Plugin should delcare itself a configure requires for Aliens that use it

similar to this:

https://metacpan.org/release/Alien-Build-Plugin-Probe-GnuWin32/source/lib/Alien/Build/Plugin/Probe/GnuWin32.pm#L51

Send GitHub PAT when using AB 2.40+

Alien-Build as of 2.40 supports sending headers via HTTP, so we should now be able to send an personal authorization token as a header to avoid at least some of the usage limits that cause test failures. Doing so would be a more robust solution that #6 which passes as an argument, and may not be supported anymore anyway. I also noticed there is a bug in #6 where I used different names for the environment variable (doh!).

Question becomes where should we go looking for the token? GitHub Actions apparently stores a PAT in secrets.GITHUB_TOKEN so that seems to be at least one place. We should also check an environment variable dedicated to AB so that users can have a token for just that and isolate it from their "default" token. What about a file?

• env var ALIEN_BUILD_GITHUB_TOKEN
• env var GITHUB_TOKEN
• env var GITHUB_PAT
• documentation
• use in CI

version in file listing should use the version regex property

This doesn't work:

https://metacpan.org/source/PLICEASE/Alien-Build-Plugin-Download-GitHub-0.02/lib/Alien/Build/Plugin/Download/GitHub.pm#L67

For example with the default version regex, if the versions have a v prefix.

Need Zip Format Support

It is not correct to assume only "tar.gz" format files, because some GitHub repositories offer "zip" format files.
https://github.com/PerlAlien/Alien-Build-Plugin-Download-GitHub/blob/main/lib/Alien/Build/Plugin/Download/GitHub.pm#L135