This is a followup to e0bf6473ba3ecbcb1e8f25b511e9ef7e2a8c7547, fixing
mangohud.

* gnu/packages/toolkits.scm (imgui-for-mangohud): New variable.
* gnu/packages/graphics.scm (mangohud)[inputs]: Replace imgui-1.91 with
imgui-for-mangohud.

* gnu/packages/toolkits.scm (imgui-1.86): Delete variable.

* gnu/packages/toolkits.scm (motif)[source]: Indent.
 [description]: Reflow.

Change-Id: Icb323739e3022b5df4d429357379485839451092
Merges: https://codeberg.org/guix/guix/pulls/8960
Reviewed-by: Liliana Marie Prikler <liliana.prikler@gmail.com>
Signed-off-by: Nguyễn Gia Phong <cnx@loang.net>

* gnu/packages/toolkits.scm (motif): Move to here...
* gnu/packages/lesstif.scm (motif): ...from here.
  New deprecated variable.

Change-Id: Id63a6f1a94d542dc600872f3e24a48e6d158d6bb
Merges: https://codeberg.org/guix/guix/pulls/8960
Reviewed-by: Liliana Marie Prikler <liliana.prikler@gmail.com>
Signed-off-by: Nguyễn Gia Phong <cnx@loang.net>

* gnu/packages/toolkits.scm (imgui-1.91): Update to 1.91.9b.

Change-Id: If01cc8d84d1015c0950dccf4532e70714b97846f

After this commit, when both modules are imported:

- if pyproject is imported before, the warning is 'pypi-uri' is
deprecated, use '(@ (guix build-system pyproject) pypi-uri)' instead

- if python is imported before, the warning is `pypi-uri' imported from
both (guix build-system python) and (guix build-system pyproject)

This seems convenient enough to warn for deprecation in the short term,
while avoiding any breaking changes.

* guix/build-system/pyproject.scm (pypi-uri): Move the procedure from
(guix build-system python) here.

* guix/build-system/python (pypi-uri): Drop definition, import it
from (guix build-system pyproject) and deprecate it.

* gnu/packages/openldap.scm: Reorder modules to get the right warning.

* gnu/packages/pypy.scm: Likewise.

* gnu/packages/*.scm : Drop module (guix build-system python).

* tests/import/pypi.scm: Likewise.

Merges: https://codeberg.org/guix/guix/pulls/7448
Change-Id: Ib42f53bc545052eb7918a25afe9db6d5fc2cb834
Reviewed-by: Nguyễn Gia Phong <cnx@loang.net>
Signed-off-by: Sharlatan Hellseher <sharlatanus@gmail.com>

This commit was made by running this command:

  sed -e's/modify-inputs (package-\([a-z-]*\)inputs [a-zA-Z0-9-]\+)/modify-inputs \1inputs/g' -i gnu/packages/*.scm

… and then reverting individual hunks where the change would trigger unbound
variable warnings or other issues (such as ‘native-inputs’ is bound in the
body of the ‘inputs’ field, but it refers to the ‘native-inputs’ thunk defined
just above).

Change-Id: I6d94819f2809313fa1fbefc61897502ee7d66fab

This commit was made by running this command:

  sed -e's/substitute-keyword-arguments (package-arguments [a-zA-Z0-9-]\+)/substitute-keyword-arguments arguments/g' -i gnu/packages/*.scm

… and then:

  1. reverting changes from ‘gnu/packages/rust.scm’ and
     ‘gnu/packages/java.scm’ since they would incur derivation changes and/or
     breakage;
  2. reverting the change for ‘gcc-final’ in ‘gnu/packages/commencement.scm’;
  3. reverting the change for ‘onnx-optimizer’, ‘openquest’, and ‘certbot’,
     which use ‘substitute-keyword-arguments’ for arguments that are not
     inherited (and thus ‘arguments’ would be unbound);
  4. reverting the change for ‘insight-toolkit-legacy’ and ‘wine64-staging’
     which make bogus assumptions about inherited arguments.

Change-Id: I122a7cf517b6b63cae38944b5d33ade4b1f5a89c

* gnu/packages/toolkits.scm (imgui): Keep binary_to_compressed_c.cpp, since
some packages need the tool it compiles to.

Change-Id: I0a236fd57d8e8d23f25cf9a2d74ab6f09a8cbef7
Signed-off-by: Gabriel Wicki <gabriel@erlikon.ch>
Signed-off-by: Rutherther <rutherther@ditigal.xyz>

* gnu/packages/toolkits.scm (imgui-1.87): Delete variable.

Change-Id: I461c5295ac47efa0b767270115560dcd45117ebe