-
-
Notifications
You must be signed in to change notification settings - Fork 15.5k
Comparing changes
Open a pull request
base repository: NixOS/nixpkgs
base: 3407b8c0fb25
head repository: NixOS/nixpkgs
compare: 27b9b7b3af72
- 12 commits
- 11 files changed
- 2 contributors
Commits on Apr 24, 2020
-
php.buildEnv: Make the exported php package overridable
This implements the override pattern for builds done with buildEnv, so that we can, for example, write php.override { fpmSupport = false; } and get a PHP package with the default extensions enabled, but PHP compiled without fpm support.
Configuration menu - View commit details
-
Copy full SHA for a463261 - Browse repository at this point
Copy the full SHA a463261View commit details -
php: Make all arguments to a PHP build overridable
Make all arguments to a PHP build overridable; i.e, both configuration flags, such as valgrindSupport, and packages, such as valgrind: php.override { valgrindSupport = false; valgrind = valgrind-light; } This applies to packages built by generic and buildEnv/withExtensions; i.e, it works with both phpXX and phpXXBase packages. The following changes were also made to facilitate this: - generic and generic' are merged into one function - generic now takes all required arguments for a complete build and is meant to be called by callPackage - The main function called from all-packages.nix no longer takes all required arguments for a complete build - all arguments passed to it are however forwarded to the individual builds, thus default arguments can still be overridden from all-packages.nix
Configuration menu - View commit details
-
Copy full SHA for dde5f5f - Browse repository at this point
Copy the full SHA dde5f5fView commit details -
php.buildEnv: Respect override
Make buildEnv take earlier overridden values into account by forwarding all arguments (a merge of generic's arguments, all previous arguments and the current arguments) to the next invocation of buildEnv.
Configuration menu - View commit details
-
Copy full SHA for abedfad - Browse repository at this point
Copy the full SHA abedfadView commit details
Commits on Apr 26, 2020
-
php.buildEnv: Provide a list of currently enabled extensions
Rework withExtensions / buildEnv to handle currently enabled extensions better and make them compatible with override. They now accept a function with the named arguments enabled and all, where enabled is a list of currently enabled extensions and all is the set of all extensions. This gives us several nice properties: - You always get the right version of the list of currently enabled extensions - Invocations chain - It works well with overridden PHP packages - you always get the correct versions of extensions As a contrived example of what's possible, you can add ImageMagick, then override the version and disable fpm, then disable cgi, and lastly remove the zip extension like this: { pkgs ? (import <nixpkgs>) {} }: with pkgs; let phpWithImagick = php74.withExtensions ({ all, enabled }: enabled ++ [ all.imagick ]); phpWithImagickWithoutFpm743 = phpWithImagick.override { version = "7.4.3"; sha256 = "wVF7pJV4+y3MZMc6Ptx21PxQfEp6xjmYFYTMfTtMbRQ="; fpmSupport = false; }; phpWithImagickWithoutFpmZip743 = phpWithImagickWithoutFpm743.withExtensions ( { enabled, all }: lib.filter (e: e != all.zip) enabled); phpWithImagickWithoutFpmZipCgi743 = phpWithImagickWithoutFpmZip743.override { cgiSupport = false; }; in phpWithImagickWithoutFpmZipCgi743
Configuration menu - View commit details
-
Copy full SHA for 2ba7926 - Browse repository at this point
Copy the full SHA 2ba7926View commit details -
php: Get rid of all config.php parameters
Since all options controlled by the config.php parameters can now be overridden directly, there's no reason to keep them around.
Configuration menu - View commit details
-
Copy full SHA for 72636bc - Browse repository at this point
Copy the full SHA 72636bcView commit details -
php.buildEnv: Automatically include extension dependencies
Some extensions depend on other extensions. Previously, these had to be added manually to the list of included extensions, or we got a cryptic error message pointing to strings-with-deps.nix, which wasn't very helpful. This makes sure all required extensions are included in the set from which textClosureList chooses its snippets.
Configuration menu - View commit details
-
Copy full SHA for d610407 - Browse repository at this point
Copy the full SHA d610407View commit details
Commits on Apr 29, 2020
-
Configuration menu - View commit details
-
Copy full SHA for c3d5d92 - Browse repository at this point
Copy the full SHA c3d5d92View commit details -
php.buildEnv: Provide the unwrapped php package in php.unwrapped
This is useful if you need to access the dev output of the unwrapped derivation.
Configuration menu - View commit details
-
Copy full SHA for ef99096 - Browse repository at this point
Copy the full SHA ef99096View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3bfd4e8 - Browse repository at this point
Copy the full SHA 3bfd4e8View commit details -
php: Get rid of the phpXXbase attributes, update docs
Since the introduction of php.unwrapped there's no real need for the phpXXbase attributes, so let's remove them to lessen potential confusion and clutter. Also update the docs to make it clear how to get hold of an unwrapped PHP if needed.
Configuration menu - View commit details
-
Copy full SHA for 5cad1b4 - Browse repository at this point
Copy the full SHA 5cad1b4View commit details -
php: Unify the usage of the php package in php-packages.nix
Instead of using two different php packages in php-packages.nix, one wrapper and one unwrapped, simply use the wrapper and use its "unwrapped" attribute when necessary. Also, get rid of the packages and extensions attributes from the base package, since they're no longer needed.
Configuration menu - View commit details
-
Copy full SHA for 2535cdf - Browse repository at this point
Copy the full SHA 2535cdfView commit details -
Merge pull request #85026 from talyz/php_buildenv_override
php.buildEnv: Make the exported php package overridable, improve handling of currently enabled extensions, etc
Configuration menu - View commit details
-
Copy full SHA for 27b9b7b - Browse repository at this point
Copy the full SHA 27b9b7bView commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff 3407b8c0fb25...27b9b7b3af72