-
-
Notifications
You must be signed in to change notification settings - Fork 18.1k
Closed
Labels
9.needs: documentationThis needs to be documented well.This needs to be documented well.
Description
Problem
Our Qt section says:
nixpkgs/doc/languages-frameworks/qt.section.md
Lines 25 to 30 in 405d3ba
| It is important to import Qt modules directly, that is: `qtbase`, `qtdeclarative`, etc. *Do not* import Qt package sets such as `qt5` because the Qt versions of dependencies may not be coherent, causing build and runtime failures. | |
| Additionally all Qt packages must include `wrapQtAppsHook` in `nativeBuildInputs`, or you must explicitly set `dontWrapQtApps`. | |
| `pkgs.callPackage` does not provide injections for `qtbase` or the like. | |
| Instead you want to either use `pkgs.libsForQt5.callPackage`, or `pkgs.qt6Packages.callPackage`, depending on the Qt version you want to use. |
But this is not compatible with the new by-name directory structure.
Proposal
There are already plenty of packages in pkgs/by-name invoking qt5 or qt6 directly. And if the structure of both sets remain the same, maybe we can use callPackage ./../.. { qt6 = qt5; } or something?
Checklist
- checked latest Nixpkgs manual (source) and latest NixOS manual (source)
- checked open documentation issues for possible duplicates
- checked open documentation pull requests for possible solutions
Add a 👍 reaction to issues you find important.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
9.needs: documentationThis needs to be documented well.This needs to be documented well.