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

treewide: change the occurrences of "name" to "pname-version" #104662

Closed
wants to merge 0 commits into from
Closed

treewide: change the occurrences of "name" to "pname-version" #104662

wants to merge 0 commits into from

Conversation

AndersonTorres
Copy link
Member

Motivation for this change
Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

pkgs/applications/audio/dirt/default.nix Outdated Show resolved Hide resolved
pkgs/applications/audio/gbsplay/default.nix Outdated Show resolved Hide resolved
pkgs/applications/audio/fluidsynth/default.nix Outdated Show resolved Hide resolved
pkgs/applications/audio/bitwig-studio/bitwig-studio3.nix Outdated Show resolved Hide resolved
pkgs/applications/audio/jackmeter/default.nix Outdated Show resolved Hide resolved
pkgs/shells/dash/default.nix Outdated Show resolved Hide resolved
pkgs/shells/es/default.nix Outdated Show resolved Hide resolved
pkgs/shells/zsh/zsh-deer/default.nix Outdated Show resolved Hide resolved
pkgs/tools/X11/arandr/default.nix Outdated Show resolved Hide resolved
pkgs/tools/X11/imwheel/default.nix Outdated Show resolved Hide resolved
@RaghavSood
Copy link
Member

For some requests, I had to leave comments instead of suggestions as GitHub seems to limit how many lines around a change you can make a suggestion on

@AndersonTorres
Copy link
Member Author

@RaghavSood many thanks! For now I am just changing the most obvious things.
But I will fix this too.

@RaghavSood
Copy link
Member

Certainly - most of my suggestions are simply to fix up stuff missed in the initial commit - the only one that is an actual issue is pkgs/applications/audio/nootka/default.nix -> Looks like one reference to ${name} wasn't changed to ${pname}, so that would break the build

The rest are just adding on to the legwork you already did in updating most scenarios :)

@AndersonTorres
Copy link
Member Author

For now just point the lines.
There is no serious need of using the "suggesting changes" feature of github.

@xaverdh
Copy link
Contributor

xaverdh commented Nov 23, 2020

While there is certainly consensus to use pname and version instead of name, I am not so sure about changing the repo part in fetchFromGitLab / fetchFromGitHub expressions to use pname.
I am not sure if that's the correct semantics.. the project name might change while still having the source in the same git repository and vice versa. To my mind, these are actually separate pieces of information, that often just happen to agree.

Either way thanks for working on this!

@RaghavSood
Copy link
Member

Conversely, if the project has changed their repo name, there's usually a good chance that they are changing the project name, in which case pname should also be updated, and perhaps an alias added for the old one. They would generally continue to agree in that case

I'm not really sure on how often a project + repo renames vs. just renaming the repo, I can't imagine it being terribly common in either scenario - we might see one or two a month across the entire tree

@AndersonTorres
Copy link
Member Author

There are some rare but noteworthy exceptions.
Lomiri still uses the old unity8 name in their repos: https://github.com/ubports/unity8

For now my concerns are

  • changing name to pname-version;
  • parameterize references to name and version (example: url = http://example.org/beautiful-software-1.5.gz; should be url=http://example.org/${pname}-${version}.gz)

@AndersonTorres
Copy link
Member Author

Why does it rebuild the world?

@ofborg ofborg bot added the 6.topic: TeX Issues regarding texlive and TeX in general label Nov 25, 2020
@AndersonTorres AndersonTorres marked this pull request as draft December 12, 2020 12:47
@AndersonTorres
Copy link
Member Author

/rebase-staging

@github-actions
Copy link
Contributor

github-actions bot commented Jan 3, 2021

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants