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: fix derivation names #41162
Conversation
@GrahamcOfBorg build altcoins.mist infamousPlugins swh-lv2 manuskript neovim-remote |
@GrahamcOfBorg build wxhexeditor yi exrtools gnuclad cura-lulzbot nix-tour xrandr-invert-colors |
Success on x86_64-darwin (full log) Attempted: neovim-remote The following builds were skipped because they don't evaluate on x86_64-darwin: altcoins.mist, infamousPlugins, swh-lv2, manuskript Partial log (click to expand)
|
@GrahamcOfBorg build uzbl coyim salut_a_toi weechat gwyddion ripser |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: wxhexeditor, yi, exrtools, gnuclad, cura-lulzbot, nix-tour, xrandr-invert-colors Partial log (click to expand)
|
Failure on x86_64-darwin (full log) Attempted: weechat The following builds were skipped because they don't evaluate on x86_64-darwin: uzbl, coyim, salut_a_toi, gwyddion, ripser Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: wxhexeditor, exrtools, gnuclad, nix-tour, xrandr-invert-colors The following builds were skipped because they don't evaluate on x86_64-linux: yi, cura-lulzbot Partial log (click to expand)
|
@@ -1,11 +1,12 @@ | |||
{ stdenv, zlib, fetchFromGitHub, python3Packages }: | |||
|
|||
python3Packages.buildPythonApplication rec { | |||
name = "manuskript"; | |||
pname = "manuskript"; | |||
name = "${pname}-${version}"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Like with buildPythonPackage
you don't need to define name
here.
Success on x86_64-linux (full log) Attempted: coyim, salut_a_toi, weechat, gwyddion, ripser The following builds were skipped because they don't evaluate on x86_64-linux: uzbl Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: wxhexeditor, exrtools, gnuclad, nix-tour, xrandr-invert-colors The following builds were skipped because they don't evaluate on aarch64-linux: yi, cura-lulzbot Partial log (click to expand)
|
Failure on x86_64-linux (full log) Attempted: altcoins.mist, infamousPlugins, manuskript, neovim-remote The following builds were skipped because they don't evaluate on x86_64-linux: swh-lv2 Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: coyim, salut_a_toi, weechat, gwyddion, ripser The following builds were skipped because they don't evaluate on aarch64-linux: uzbl Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: infamousPlugins, manuskript, neovim-remote The following builds were skipped because they don't evaluate on aarch64-linux: altcoins.mist, swh-lv2 Partial log (click to expand)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I checked the changes and they seem good. Note there is a version bump inside as well.
in | ||
|
||
stdenv.mkDerivation rec { | ||
name = "simgrid-${version}"; | ||
version = "3.17"; | ||
version = "3.19.1"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
version bump
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, sorry for not mentioning that in the PR text it used to be more clear before I fixed the merge conflicts with master. I'll move this into it's own PR.
I moved the simgrid version bump to its own PR. #41178 |
Motivation for this change
all these packages were being excluded from repology because their derivation names could not be parsed.
Things done