fetchFromGitHub: force re-fetch when rev changes #11

Merged
bbjubjub2494 merged 4 commits from fetchgithub-refetch into main 2024-05-21 05:45:52 +00:00
Showing only changes of commit 882bdc43f2 - Show all commits

View file

@ -8,6 +8,7 @@ lib.makeOverridable (
Sorixelle commented 2024-05-19 08:11:11 +00:00 (Migrated from github.com)
Review

TBH, I'd also change source to something like github-repo, just to make it clearer in the name that the derivation comes from a GitHub checkout.

TBH, I'd also change `source` to something like `github-repo`, just to make it clearer in the name that the derivation comes from a GitHub checkout.
Sorixelle commented 2024-05-19 08:12:06 +00:00 (Migrated from github.com)
Review

nit:

, name ? null # Override with null to use the default value
nit: ```suggestion , name ? null # Override with null to use the default value ```
bbjubjub2494 commented 2024-05-19 08:24:50 +00:00 (Migrated from github.com)
Review

Here's a slightly different proposal: append the github base url as provenance like so: /nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv. That's a good self-description, leaves the source that people might be used to, and generalizes unambiguously to other fetchFroms. We can also put it before the owner.

Here's a slightly different proposal: append the github base url as provenance like so: `/nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv`. That's a good self-description, leaves the `source` that people might be used to, and generalizes unambiguously to other `fetchFrom`s. We can also put it before the `owner`.
Sorixelle commented 2024-05-20 05:45:55 +00:00 (Migrated from github.com)
Review

Only potential issue I see is creating some pretty long path names, but I don't see it being a massive issue. Seems fine to me.

Only potential issue I see is creating some pretty long path names, but I don't see it being a *massive* issue. Seems fine to me.
Sorixelle commented 2024-05-19 08:11:11 +00:00 (Migrated from github.com)
Review

TBH, I'd also change source to something like github-repo, just to make it clearer in the name that the derivation comes from a GitHub checkout.

TBH, I'd also change `source` to something like `github-repo`, just to make it clearer in the name that the derivation comes from a GitHub checkout.
Sorixelle commented 2024-05-19 08:12:06 +00:00 (Migrated from github.com)
Review

nit:

, name ? null # Override with null to use the default value
nit: ```suggestion , name ? null # Override with null to use the default value ```
bbjubjub2494 commented 2024-05-19 08:24:50 +00:00 (Migrated from github.com)
Review

Here's a slightly different proposal: append the github base url as provenance like so: /nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv. That's a good self-description, leaves the source that people might be used to, and generalizes unambiguously to other fetchFroms. We can also put it before the owner.

Here's a slightly different proposal: append the github base url as provenance like so: `/nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv`. That's a good self-description, leaves the `source` that people might be used to, and generalizes unambiguously to other `fetchFrom`s. We can also put it before the `owner`.
Sorixelle commented 2024-05-20 05:45:55 +00:00 (Migrated from github.com)
Review

Only potential issue I see is creating some pretty long path names, but I don't see it being a massive issue. Seems fine to me.

Only potential issue I see is creating some pretty long path names, but I don't see it being a *massive* issue. Seems fine to me.
, deepClone ? false, private ? false, forceFetchGit ? false
, sparseCheckout ? []
, githubBase ? "github.com", varPrefix ? null
, passthru ? { }
Sorixelle commented 2024-05-19 08:11:11 +00:00 (Migrated from github.com)
Review

TBH, I'd also change source to something like github-repo, just to make it clearer in the name that the derivation comes from a GitHub checkout.

TBH, I'd also change `source` to something like `github-repo`, just to make it clearer in the name that the derivation comes from a GitHub checkout.
Sorixelle commented 2024-05-19 08:12:06 +00:00 (Migrated from github.com)
Review

nit:

, name ? null # Override with null to use the default value
nit: ```suggestion , name ? null # Override with null to use the default value ```
bbjubjub2494 commented 2024-05-19 08:24:50 +00:00 (Migrated from github.com)
Review

Here's a slightly different proposal: append the github base url as provenance like so: /nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv. That's a good self-description, leaves the source that people might be used to, and generalizes unambiguously to other fetchFroms. We can also put it before the owner.

Here's a slightly different proposal: append the github base url as provenance like so: `/nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv`. That's a good self-description, leaves the `source` that people might be used to, and generalizes unambiguously to other `fetchFrom`s. We can also put it before the `owner`.
Sorixelle commented 2024-05-20 05:45:55 +00:00 (Migrated from github.com)
Review

Only potential issue I see is creating some pretty long path names, but I don't see it being a massive issue. Seems fine to me.

Only potential issue I see is creating some pretty long path names, but I don't see it being a *massive* issue. Seems fine to me.
, meta ? { }
, ... # For hash agility
}@args:
@ -21,6 +22,9 @@ let
Sorixelle commented 2024-05-19 08:11:11 +00:00 (Migrated from github.com)
Review

TBH, I'd also change source to something like github-repo, just to make it clearer in the name that the derivation comes from a GitHub checkout.

TBH, I'd also change `source` to something like `github-repo`, just to make it clearer in the name that the derivation comes from a GitHub checkout.
Sorixelle commented 2024-05-19 08:12:06 +00:00 (Migrated from github.com)
Review

nit:

, name ? null # Override with null to use the default value
nit: ```suggestion , name ? null # Override with null to use the default value ```
bbjubjub2494 commented 2024-05-19 08:24:50 +00:00 (Migrated from github.com)
Review

Here's a slightly different proposal: append the github base url as provenance like so: /nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv. That's a good self-description, leaves the source that people might be used to, and generalizes unambiguously to other fetchFroms. We can also put it before the owner.

Here's a slightly different proposal: append the github base url as provenance like so: `/nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv`. That's a good self-description, leaves the `source` that people might be used to, and generalizes unambiguously to other `fetchFrom`s. We can also put it before the `owner`.
Sorixelle commented 2024-05-20 05:45:55 +00:00 (Migrated from github.com)
Review

Only potential issue I see is creating some pretty long path names, but I don't see it being a massive issue. Seems fine to me.

Only potential issue I see is creating some pretty long path names, but I don't see it being a *massive* issue. Seems fine to me.
Sorixelle commented 2024-05-19 08:11:11 +00:00 (Migrated from github.com)
Review

TBH, I'd also change source to something like github-repo, just to make it clearer in the name that the derivation comes from a GitHub checkout.

TBH, I'd also change `source` to something like `github-repo`, just to make it clearer in the name that the derivation comes from a GitHub checkout.
Sorixelle commented 2024-05-19 08:12:06 +00:00 (Migrated from github.com)
Review

nit:

, name ? null # Override with null to use the default value
nit: ```suggestion , name ? null # Override with null to use the default value ```
bbjubjub2494 commented 2024-05-19 08:24:50 +00:00 (Migrated from github.com)
Review

Here's a slightly different proposal: append the github base url as provenance like so: /nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv. That's a good self-description, leaves the source that people might be used to, and generalizes unambiguously to other fetchFroms. We can also put it before the owner.

Here's a slightly different proposal: append the github base url as provenance like so: `/nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv`. That's a good self-description, leaves the `source` that people might be used to, and generalizes unambiguously to other `fetchFrom`s. We can also put it before the `owner`.
Sorixelle commented 2024-05-20 05:45:55 +00:00 (Migrated from github.com)
Review

Only potential issue I see is creating some pretty long path names, but I don't see it being a massive issue. Seems fine to me.

Only potential issue I see is creating some pretty long path names, but I don't see it being a *massive* issue. Seems fine to me.
else builtins.unsafeGetAttrPos "rev" args
);
baseUrl = "https://${githubBase}/${owner}/${repo}";
newPassthru = passthru // {
Sorixelle commented 2024-05-19 08:11:11 +00:00 (Migrated from github.com)
Review

TBH, I'd also change source to something like github-repo, just to make it clearer in the name that the derivation comes from a GitHub checkout.

TBH, I'd also change `source` to something like `github-repo`, just to make it clearer in the name that the derivation comes from a GitHub checkout.
Sorixelle commented 2024-05-19 08:12:06 +00:00 (Migrated from github.com)
Review

nit:

, name ? null # Override with null to use the default value
nit: ```suggestion , name ? null # Override with null to use the default value ```
bbjubjub2494 commented 2024-05-19 08:24:50 +00:00 (Migrated from github.com)
Review

Here's a slightly different proposal: append the github base url as provenance like so: /nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv. That's a good self-description, leaves the source that people might be used to, and generalizes unambiguously to other fetchFroms. We can also put it before the owner.

Here's a slightly different proposal: append the github base url as provenance like so: `/nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv`. That's a good self-description, leaves the `source` that people might be used to, and generalizes unambiguously to other `fetchFrom`s. We can also put it before the `owner`.
Sorixelle commented 2024-05-20 05:45:55 +00:00 (Migrated from github.com)
Review

Only potential issue I see is creating some pretty long path names, but I don't see it being a massive issue. Seems fine to me.

Only potential issue I see is creating some pretty long path names, but I don't see it being a *massive* issue. Seems fine to me.
inherit rev owner repo;
Sorixelle commented 2024-05-19 08:11:11 +00:00 (Migrated from github.com)
Review

TBH, I'd also change source to something like github-repo, just to make it clearer in the name that the derivation comes from a GitHub checkout.

TBH, I'd also change `source` to something like `github-repo`, just to make it clearer in the name that the derivation comes from a GitHub checkout.
Sorixelle commented 2024-05-19 08:12:06 +00:00 (Migrated from github.com)
Review

nit:

, name ? null # Override with null to use the default value
nit: ```suggestion , name ? null # Override with null to use the default value ```
bbjubjub2494 commented 2024-05-19 08:24:50 +00:00 (Migrated from github.com)
Review

Here's a slightly different proposal: append the github base url as provenance like so: /nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv. That's a good self-description, leaves the source that people might be used to, and generalizes unambiguously to other fetchFroms. We can also put it before the owner.

Here's a slightly different proposal: append the github base url as provenance like so: `/nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv`. That's a good self-description, leaves the `source` that people might be used to, and generalizes unambiguously to other `fetchFrom`s. We can also put it before the `owner`.
Sorixelle commented 2024-05-20 05:45:55 +00:00 (Migrated from github.com)
Review

Only potential issue I see is creating some pretty long path names, but I don't see it being a massive issue. Seems fine to me.

Only potential issue I see is creating some pretty long path names, but I don't see it being a *massive* issue. Seems fine to me.
};
Sorixelle commented 2024-05-19 08:11:11 +00:00 (Migrated from github.com)
Review

TBH, I'd also change source to something like github-repo, just to make it clearer in the name that the derivation comes from a GitHub checkout.

TBH, I'd also change `source` to something like `github-repo`, just to make it clearer in the name that the derivation comes from a GitHub checkout.
Sorixelle commented 2024-05-19 08:12:06 +00:00 (Migrated from github.com)
Review

nit:

, name ? null # Override with null to use the default value
nit: ```suggestion , name ? null # Override with null to use the default value ```
bbjubjub2494 commented 2024-05-19 08:24:50 +00:00 (Migrated from github.com)
Review

Here's a slightly different proposal: append the github base url as provenance like so: /nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv. That's a good self-description, leaves the source that people might be used to, and generalizes unambiguously to other fetchFroms. We can also put it before the owner.

Here's a slightly different proposal: append the github base url as provenance like so: `/nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv`. That's a good self-description, leaves the `source` that people might be used to, and generalizes unambiguously to other `fetchFrom`s. We can also put it before the `owner`.
Sorixelle commented 2024-05-20 05:45:55 +00:00 (Migrated from github.com)
Review

Only potential issue I see is creating some pretty long path names, but I don't see it being a massive issue. Seems fine to me.

Only potential issue I see is creating some pretty long path names, but I don't see it being a *massive* issue. Seems fine to me.
newMeta = meta // {
homepage = meta.homepage or baseUrl;
} // lib.optionalAttrs (position != null) {
@ -57,16 +61,19 @@ let
Sorixelle commented 2024-05-19 08:11:11 +00:00 (Migrated from github.com)
Review

TBH, I'd also change source to something like github-repo, just to make it clearer in the name that the derivation comes from a GitHub checkout.

TBH, I'd also change `source` to something like `github-repo`, just to make it clearer in the name that the derivation comes from a GitHub checkout.
Sorixelle commented 2024-05-19 08:12:06 +00:00 (Migrated from github.com)
Review

nit:

, name ? null # Override with null to use the default value
nit: ```suggestion , name ? null # Override with null to use the default value ```
bbjubjub2494 commented 2024-05-19 08:24:50 +00:00 (Migrated from github.com)
Review

Here's a slightly different proposal: append the github base url as provenance like so: /nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv. That's a good self-description, leaves the source that people might be used to, and generalizes unambiguously to other fetchFroms. We can also put it before the owner.

Here's a slightly different proposal: append the github base url as provenance like so: `/nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv`. That's a good self-description, leaves the `source` that people might be used to, and generalizes unambiguously to other `fetchFrom`s. We can also put it before the `owner`.
Sorixelle commented 2024-05-20 05:45:55 +00:00 (Migrated from github.com)
Review

Only potential issue I see is creating some pretty long path names, but I don't see it being a massive issue. Seems fine to me.

Only potential issue I see is creating some pretty long path names, but I don't see it being a *massive* issue. Seems fine to me.
Sorixelle commented 2024-05-19 08:11:11 +00:00 (Migrated from github.com)
Review

TBH, I'd also change source to something like github-repo, just to make it clearer in the name that the derivation comes from a GitHub checkout.

TBH, I'd also change `source` to something like `github-repo`, just to make it clearer in the name that the derivation comes from a GitHub checkout.
Sorixelle commented 2024-05-19 08:12:06 +00:00 (Migrated from github.com)
Review

nit:

, name ? null # Override with null to use the default value
nit: ```suggestion , name ? null # Override with null to use the default value ```
bbjubjub2494 commented 2024-05-19 08:24:50 +00:00 (Migrated from github.com)
Review

Here's a slightly different proposal: append the github base url as provenance like so: /nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv. That's a good self-description, leaves the source that people might be used to, and generalizes unambiguously to other fetchFroms. We can also put it before the owner.

Here's a slightly different proposal: append the github base url as provenance like so: `/nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv`. That's a good self-description, leaves the `source` that people might be used to, and generalizes unambiguously to other `fetchFrom`s. We can also put it before the `owner`.
Sorixelle commented 2024-05-20 05:45:55 +00:00 (Migrated from github.com)
Review

Only potential issue I see is creating some pretty long path names, but I don't see it being a massive issue. Seems fine to me.

Only potential issue I see is creating some pretty long path names, but I don't see it being a *massive* issue. Seems fine to me.
fetcherArgs = (if useFetchGit
then {
inherit rev deepClone fetchSubmodules sparseCheckout; url = gitRepoUrl;
passthru = newPassthru;
Sorixelle commented 2024-05-19 08:11:11 +00:00 (Migrated from github.com)
Review

TBH, I'd also change source to something like github-repo, just to make it clearer in the name that the derivation comes from a GitHub checkout.

TBH, I'd also change `source` to something like `github-repo`, just to make it clearer in the name that the derivation comes from a GitHub checkout.
Sorixelle commented 2024-05-19 08:12:06 +00:00 (Migrated from github.com)
Review

nit:

, name ? null # Override with null to use the default value
nit: ```suggestion , name ? null # Override with null to use the default value ```
bbjubjub2494 commented 2024-05-19 08:24:50 +00:00 (Migrated from github.com)
Review

Here's a slightly different proposal: append the github base url as provenance like so: /nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv. That's a good self-description, leaves the source that people might be used to, and generalizes unambiguously to other fetchFroms. We can also put it before the owner.

Here's a slightly different proposal: append the github base url as provenance like so: `/nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv`. That's a good self-description, leaves the `source` that people might be used to, and generalizes unambiguously to other `fetchFrom`s. We can also put it before the `owner`.
Sorixelle commented 2024-05-20 05:45:55 +00:00 (Migrated from github.com)
Review

Only potential issue I see is creating some pretty long path names, but I don't see it being a massive issue. Seems fine to me.

Only potential issue I see is creating some pretty long path names, but I don't see it being a *massive* issue. Seems fine to me.
} // lib.optionalAttrs (leaveDotGit != null) { inherit leaveDotGit; }
else {
url = "${baseUrl}/archive/${rev}.tar.gz";
passthru = {
Sorixelle commented 2024-05-19 08:11:11 +00:00 (Migrated from github.com)
Review

TBH, I'd also change source to something like github-repo, just to make it clearer in the name that the derivation comes from a GitHub checkout.

TBH, I'd also change `source` to something like `github-repo`, just to make it clearer in the name that the derivation comes from a GitHub checkout.
Sorixelle commented 2024-05-19 08:12:06 +00:00 (Migrated from github.com)
Review

nit:

, name ? null # Override with null to use the default value
nit: ```suggestion , name ? null # Override with null to use the default value ```
bbjubjub2494 commented 2024-05-19 08:24:50 +00:00 (Migrated from github.com)
Review

Here's a slightly different proposal: append the github base url as provenance like so: /nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv. That's a good self-description, leaves the source that people might be used to, and generalizes unambiguously to other fetchFroms. We can also put it before the owner.

Here's a slightly different proposal: append the github base url as provenance like so: `/nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv`. That's a good self-description, leaves the `source` that people might be used to, and generalizes unambiguously to other `fetchFrom`s. We can also put it before the `owner`.
Sorixelle commented 2024-05-20 05:45:55 +00:00 (Migrated from github.com)
Review

Only potential issue I see is creating some pretty long path names, but I don't see it being a massive issue. Seems fine to me.

Only potential issue I see is creating some pretty long path names, but I don't see it being a *massive* issue. Seems fine to me.
passthru = newPassthru // {
Sorixelle commented 2024-05-19 08:11:11 +00:00 (Migrated from github.com)
Review

TBH, I'd also change source to something like github-repo, just to make it clearer in the name that the derivation comes from a GitHub checkout.

TBH, I'd also change `source` to something like `github-repo`, just to make it clearer in the name that the derivation comes from a GitHub checkout.
Sorixelle commented 2024-05-19 08:12:06 +00:00 (Migrated from github.com)
Review

nit:

, name ? null # Override with null to use the default value
nit: ```suggestion , name ? null # Override with null to use the default value ```
bbjubjub2494 commented 2024-05-19 08:24:50 +00:00 (Migrated from github.com)
Review

Here's a slightly different proposal: append the github base url as provenance like so: /nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv. That's a good self-description, leaves the source that people might be used to, and generalizes unambiguously to other fetchFroms. We can also put it before the owner.

Here's a slightly different proposal: append the github base url as provenance like so: `/nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv`. That's a good self-description, leaves the `source` that people might be used to, and generalizes unambiguously to other `fetchFrom`s. We can also put it before the `owner`.
Sorixelle commented 2024-05-20 05:45:55 +00:00 (Migrated from github.com)
Review

Only potential issue I see is creating some pretty long path names, but I don't see it being a massive issue. Seems fine to me.

Only potential issue I see is creating some pretty long path names, but I don't see it being a *massive* issue. Seems fine to me.
inherit gitRepoUrl;
};
}
) // privateAttrs // passthruAttrs // { inherit name; };
in
fetcher fetcherArgs // { meta = newMeta; inherit rev owner repo; }
Sorixelle commented 2024-05-19 08:11:11 +00:00 (Migrated from github.com)
Review

TBH, I'd also change source to something like github-repo, just to make it clearer in the name that the derivation comes from a GitHub checkout.

TBH, I'd also change `source` to something like `github-repo`, just to make it clearer in the name that the derivation comes from a GitHub checkout.
Sorixelle commented 2024-05-19 08:12:06 +00:00 (Migrated from github.com)
Review

nit:

, name ? null # Override with null to use the default value
nit: ```suggestion , name ? null # Override with null to use the default value ```
bbjubjub2494 commented 2024-05-19 08:24:50 +00:00 (Migrated from github.com)
Review

Here's a slightly different proposal: append the github base url as provenance like so: /nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv. That's a good self-description, leaves the source that people might be used to, and generalizes unambiguously to other fetchFroms. We can also put it before the owner.

Here's a slightly different proposal: append the github base url as provenance like so: `/nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv`. That's a good self-description, leaves the `source` that people might be used to, and generalizes unambiguously to other `fetchFrom`s. We can also put it before the `owner`.
Sorixelle commented 2024-05-20 05:45:55 +00:00 (Migrated from github.com)
Review

Only potential issue I see is creating some pretty long path names, but I don't see it being a massive issue. Seems fine to me.

Only potential issue I see is creating some pretty long path names, but I don't see it being a *massive* issue. Seems fine to me.
(fetcher fetcherArgs).overrideAttrs (finalAttrs: previousAttrs: {
Sorixelle commented 2024-05-19 08:11:11 +00:00 (Migrated from github.com)
Review

TBH, I'd also change source to something like github-repo, just to make it clearer in the name that the derivation comes from a GitHub checkout.

TBH, I'd also change `source` to something like `github-repo`, just to make it clearer in the name that the derivation comes from a GitHub checkout.
Sorixelle commented 2024-05-19 08:12:06 +00:00 (Migrated from github.com)
Review

nit:

, name ? null # Override with null to use the default value
nit: ```suggestion , name ? null # Override with null to use the default value ```
bbjubjub2494 commented 2024-05-19 08:24:50 +00:00 (Migrated from github.com)
Review

Here's a slightly different proposal: append the github base url as provenance like so: /nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv. That's a good self-description, leaves the source that people might be used to, and generalizes unambiguously to other fetchFroms. We can also put it before the owner.

Here's a slightly different proposal: append the github base url as provenance like so: `/nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv`. That's a good self-description, leaves the `source` that people might be used to, and generalizes unambiguously to other `fetchFrom`s. We can also put it before the `owner`.
Sorixelle commented 2024-05-20 05:45:55 +00:00 (Migrated from github.com)
Review

Only potential issue I see is creating some pretty long path names, but I don't see it being a massive issue. Seems fine to me.

Only potential issue I see is creating some pretty long path names, but I don't see it being a *massive* issue. Seems fine to me.
meta = newMeta;
Sorixelle commented 2024-05-19 08:11:11 +00:00 (Migrated from github.com)
Review

TBH, I'd also change source to something like github-repo, just to make it clearer in the name that the derivation comes from a GitHub checkout.

TBH, I'd also change `source` to something like `github-repo`, just to make it clearer in the name that the derivation comes from a GitHub checkout.
Sorixelle commented 2024-05-19 08:12:06 +00:00 (Migrated from github.com)
Review

nit:

, name ? null # Override with null to use the default value
nit: ```suggestion , name ? null # Override with null to use the default value ```
bbjubjub2494 commented 2024-05-19 08:24:50 +00:00 (Migrated from github.com)
Review

Here's a slightly different proposal: append the github base url as provenance like so: /nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv. That's a good self-description, leaves the source that people might be used to, and generalizes unambiguously to other fetchFroms. We can also put it before the owner.

Here's a slightly different proposal: append the github base url as provenance like so: `/nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv`. That's a good self-description, leaves the `source` that people might be used to, and generalizes unambiguously to other `fetchFrom`s. We can also put it before the `owner`.
Sorixelle commented 2024-05-20 05:45:55 +00:00 (Migrated from github.com)
Review

Only potential issue I see is creating some pretty long path names, but I don't see it being a massive issue. Seems fine to me.

Only potential issue I see is creating some pretty long path names, but I don't see it being a *massive* issue. Seems fine to me.
})
Sorixelle commented 2024-05-19 08:11:11 +00:00 (Migrated from github.com)
Review

TBH, I'd also change source to something like github-repo, just to make it clearer in the name that the derivation comes from a GitHub checkout.

TBH, I'd also change `source` to something like `github-repo`, just to make it clearer in the name that the derivation comes from a GitHub checkout.
Sorixelle commented 2024-05-19 08:12:06 +00:00 (Migrated from github.com)
Review

nit:

, name ? null # Override with null to use the default value
nit: ```suggestion , name ? null # Override with null to use the default value ```
bbjubjub2494 commented 2024-05-19 08:24:50 +00:00 (Migrated from github.com)
Review

Here's a slightly different proposal: append the github base url as provenance like so: /nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv. That's a good self-description, leaves the source that people might be used to, and generalizes unambiguously to other fetchFroms. We can also put it before the owner.

Here's a slightly different proposal: append the github base url as provenance like so: `/nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv`. That's a good self-description, leaves the `source` that people might be used to, and generalizes unambiguously to other `fetchFrom`s. We can also put it before the `owner`.
Sorixelle commented 2024-05-20 05:45:55 +00:00 (Migrated from github.com)
Review

Only potential issue I see is creating some pretty long path names, but I don't see it being a massive issue. Seems fine to me.

Only potential issue I see is creating some pretty long path names, but I don't see it being a *massive* issue. Seems fine to me.
)

Sorixelle commented 2024-05-19 08:11:11 +00:00 (Migrated from github.com)
Review

TBH, I'd also change source to something like github-repo, just to make it clearer in the name that the derivation comes from a GitHub checkout.

TBH, I'd also change `source` to something like `github-repo`, just to make it clearer in the name that the derivation comes from a GitHub checkout.
Sorixelle commented 2024-05-19 08:12:06 +00:00 (Migrated from github.com)
Review

nit:

, name ? null # Override with null to use the default value
nit: ```suggestion , name ? null # Override with null to use the default value ```
bbjubjub2494 commented 2024-05-19 08:24:50 +00:00 (Migrated from github.com)
Review

Here's a slightly different proposal: append the github base url as provenance like so: /nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv. That's a good self-description, leaves the source that people might be used to, and generalizes unambiguously to other fetchFroms. We can also put it before the owner.

Here's a slightly different proposal: append the github base url as provenance like so: `/nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv`. That's a good self-description, leaves the `source` that people might be used to, and generalizes unambiguously to other `fetchFrom`s. We can also put it before the `owner`.
Sorixelle commented 2024-05-20 05:45:55 +00:00 (Migrated from github.com)
Review

Only potential issue I see is creating some pretty long path names, but I don't see it being a massive issue. Seems fine to me.

Only potential issue I see is creating some pretty long path names, but I don't see it being a *massive* issue. Seems fine to me.
Sorixelle commented 2024-05-19 08:11:11 +00:00 (Migrated from github.com)
Review

TBH, I'd also change source to something like github-repo, just to make it clearer in the name that the derivation comes from a GitHub checkout.

TBH, I'd also change `source` to something like `github-repo`, just to make it clearer in the name that the derivation comes from a GitHub checkout.
Sorixelle commented 2024-05-19 08:12:06 +00:00 (Migrated from github.com)
Review

nit:

, name ? null # Override with null to use the default value
nit: ```suggestion , name ? null # Override with null to use the default value ```
bbjubjub2494 commented 2024-05-19 08:24:50 +00:00 (Migrated from github.com)
Review

Here's a slightly different proposal: append the github base url as provenance like so: /nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv. That's a good self-description, leaves the source that people might be used to, and generalizes unambiguously to other fetchFroms. We can also put it before the owner.

Here's a slightly different proposal: append the github base url as provenance like so: `/nix/store/wzvnh3p7gv1qlqhb3iif878bfffzrvaz-source-pypa-build-refs-tags-1.1.1-github.com.drv`. That's a good self-description, leaves the `source` that people might be used to, and generalizes unambiguously to other `fetchFrom`s. We can also put it before the `owner`.
Sorixelle commented 2024-05-20 05:45:55 +00:00 (Migrated from github.com)
Review

Only potential issue I see is creating some pretty long path names, but I don't see it being a massive issue. Seems fine to me.

Only potential issue I see is creating some pretty long path names, but I don't see it being a *massive* issue. Seems fine to me.