nixos/nix
1
1
Fork 0
mirror of https://github.com/NixOS/nix.git synced 2025-12-03 23:51:00 +01:00
Code Activity

Merge pull request #13486 from fzakaria/issue-12935

Browse source 
Add helpful messages when file:// used as tarball
This commit is contained in:
tomberek 2025-07-17 02:51:31 -04:00 • committed by GitHub
commit de1d91f711
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 19 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

19
src/libfetchers/tarball.cc
View file

@ -111,6 +111,25 @@ static DownloadTarballResult downloadTarball_(
const Headers & headers, const Headers & headers,
const std::string & displayPrefix) const std::string & displayPrefix)
{ {
// Some friendly error messages for common mistakes.
// Namely lets catch when the url is a local file path, but
// it is not in fact a tarball.
if (url.rfind("file://", 0) == 0) {
// Remove "file://" prefix to get the local file path
std::string localPath = url.substr(7);
if (!std::filesystem::exists(localPath)) {
throw Error("tarball '%s' does not exist.", localPath);
}
if (std::filesystem::is_directory(localPath)) {
if (std::filesystem::exists(localPath + "/.git")) {
throw Error(
"tarball '%s' is a git repository, not a tarball. Please use `git+file` as the scheme.", localPath);
}
throw Error("tarball '%s' is a directory, not a file.", localPath);
}
}
Cache::Key cacheKey{"tarball", {{"url", url}}}; Cache::Key cacheKey{"tarball", {{"url", url}}};
auto cached = settings.getCache()->lookupExpired(cacheKey); auto cached = settings.getCache()->lookupExpired(cacheKey);