Closed nightscape closed 2 months ago
All changes look good.
Wait for review from human collaborators.
@niheaven would you mind having a look?
You are downloading the whole repo, is there some method to download only the subdir?
@niheaven not trivially... Here's a SO thread regarding downloading a single directory from Git: https://stackoverflow.com/questions/7106012/download-a-single-folder-or-directory-from-a-github-repo
The solutions require either
They all would make the installation more complicated or less safe. Personally I would bite the bullet and accept the bigger download (~25Mb). But if you say one of the three options above is preferrable nonetheless, I'll go for it.
Hmm, another way: use https://github.com/ScoopInstaller/Binary to store these bins. Please refer https://github.com/ScoopInstaller/Main/blob/master/bucket/dark.json to make a proper manifest and upload needed zip (or 7z) to Binary repo.
@niheaven so the approach would be to
Hmm, another way to make manifest: download all files in that dir:
You could remove unneeded files above.
@niheaven I like the last approach best 🙂 It doesn't need any intermediate uploads or third-party tools, and the list of files hopefully won't change a lot. I tested it locally and it works nicely. My last commit takes your proposed changes verbatim 👍 Regarding your question about unneeded files: Depending on the use case, each file might be required, so I'd leave them all in. Good to merge from my side!
/verify
All changes look good.
Wait for review from human collaborators.
The author of the original
hadoop-winutils
is too busy to continue working on it, but there is a recommended fork that provides newer versions at cdarlint/winutils. This PR adds version 3.3 from @cdarlint's fork.Resolves https://github.com/ScoopInstaller/Extras/issues/3009