Skip to content
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

Map Nix attribute path to srcname #1317

Merged
merged 1 commit into from
Mar 29, 2023
Merged

Map Nix attribute path to srcname #1317

merged 1 commit into from
Mar 29, 2023

Conversation

rhendric
Copy link
Contributor

As hypothesized in repology/repology-webapp#218.

@AMDmi3
Copy link
Member

AMDmi3 commented Mar 28, 2023

Let's add binname to NIX_PNAME mapping as discussed in the very same issue, right away.

@rhendric
Copy link
Contributor Author

I don't think Nix has a binname, as you defined it, or if it does it's also the NIX_ATTRIBUTE_PATH. Attribute paths are used to install packages. Neither name is really used to refer to individual installed packages; store paths generally serve that purpose. Attribute paths can suffice in some circumstances but not every installed package corresponds to an attribute path. What a pname does is describe an installed package in terms that might be familiar to other users of that software on non-Nix platforms, but from the perspective of Nix a pname is not really an identifier, just some descriptive text.

But the most important thing to me is getting the attribute path exposed somehow so that Nix ecosystem tools can consume it, so if you want binname mapped to NIX_PNAME and you're okay with knowing that it's not really what you say it is then I don't really mind.

@AMDmi3
Copy link
Member

AMDmi3 commented Mar 29, 2023

Okay, makes sense.

@AMDmi3 AMDmi3 merged commit 5ffe785 into repology:master Mar 29, 2023
AMDmi3 added a commit to repology/repology-webapp that referenced this pull request Mar 29, 2023
This is possible after repology/repology-updater#1317, related issue #218
@rhendric rhendric deleted the patch-1 branch March 29, 2023 17:55
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.

2 participants