Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: NixOS/nixpkgs
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: cc7ed9671666
Choose a base ref
...
head repository: NixOS/nixpkgs
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 6a1978f7b489
Choose a head ref
  • 3 commits
  • 13 files changed
  • 3 contributors

Commits on Sep 25, 2017

  1. linuxPackages: properly propagate features and extraConfig

    `kernel.override { features = ... }` didn't work before, now it works as
    expected.
    
    (cherry picked from commit 8525b78)
    oxij authored and joachifm committed Sep 25, 2017

    Unverified

    This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
    Copy the full SHA
    7c80f69 View commit details
  2. tor-browser-bundle-bin: optionally disable multiprocess support

    Multiprocess tabs always crash, as first reported by the issue mentioned
    below.  It is now consistently reproducible both on NixOS and non-NixOS
    for me, so I've decided to add a toggle to conveniently disable
    multiprocess support as a work-around.
    
    Closes #27759 but does
    not really fix the underlying problem ...
    
    (cherry picked from commit 69e3817)
    joachifm committed Sep 25, 2017

    Unverified

    This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
    Copy the full SHA
    66384f9 View commit details
  3. nixos/radicale: fix default version (#29743)

    (cherry picked from commit a8c97ad)
    infinisil authored and joachifm committed Sep 25, 2017

    Unverified

    This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
    Copy the full SHA
    6a1978f View commit details
2 changes: 1 addition & 1 deletion nixos/modules/services/networking/radicale.nix
Original file line number Diff line number Diff line change
@@ -9,7 +9,7 @@ let
confFile = pkgs.writeText "radicale.conf" cfg.config;

# This enables us to default to version 2 while still not breaking configurations of people with version 1
defaultPackage = if versionAtLeast "17.09" config.system.stateVersion then {
defaultPackage = if versionAtLeast config.system.stateVersion "17.09" then {
pkg = pkgs.radicale2;
text = "pkgs.radicale2";
} else {
1 change: 1 addition & 0 deletions nixos/tests/radicale.nix
Original file line number Diff line number Diff line change
@@ -43,6 +43,7 @@ in
});
})
];
system.stateVersion = "17.03";
};
radicale1_export = lib.recursiveUpdate radicale1 {
services.radicale.extraArgs = [
Original file line number Diff line number Diff line change
@@ -44,6 +44,10 @@
, hicolor_icon_theme
, shared_mime_info

# Whether to disable multiprocess support to work around crashing tabs
# TODO: fix the underlying problem instead of this terrible work-around
, disableContentSandbox ? true

# Extra preferences
, extraPrefs ? ""
}:
@@ -210,6 +214,10 @@ stdenv.mkDerivation rec {
lockPref("extensions.torlauncher.control_port_use_ipc", true);
lockPref("extensions.torlauncher.socks_port_use_ipc", true);
// Optionally disable multiprocess support. We always set this to ensure that
// toggling the pref takes effect.
lockPref("browser.tabs.remote.autostart.2", ${if disableContentSandbox then "false" else "true"});
${optionalString (extraPrefs != "") ''
${extraPrefs}
''}
32 changes: 21 additions & 11 deletions pkgs/os-specific/linux/kernel/generic.nix
Original file line number Diff line number Diff line change
@@ -36,6 +36,22 @@ let

lib = stdenv.lib;

# Combine the `features' attribute sets of all the kernel patches.
kernelFeatures = lib.fold (x: y: (x.features or {}) // y) ({
iwlwifi = true;
efiBootStub = true;
needsCifsUtils = true;
netfilterRPFilter = true;
} // features) kernelPatches;

configWithPlatform = kernelPlatform: import ./common-config.nix {
inherit stdenv version kernelPlatform extraConfig;
features = kernelFeatures; # Ensure we know of all extra patches, etc.
};

config = configWithPlatform stdenv.platform;
configCross = configWithPlatform hostPlatform.platform;

kernelConfigFun = baseConfig:
let
configFromPatches =
@@ -115,23 +131,17 @@ let
};

passthru = {
# Combine the `features' attribute sets of all the kernel patches.
features = lib.fold (x: y: (x.features or {}) // y) features kernelPatches;
features = kernelFeatures;

meta = kernel.meta // extraMeta;

passthru = kernel.passthru // (removeAttrs passthru [ "passthru" "meta" ]);
};

configWithPlatform = kernelPlatform: import ./common-config.nix
{ inherit stdenv version kernelPlatform extraConfig;
features = passthru.features; # Ensure we know of all extra patches, etc.
};

config = configWithPlatform stdenv.platform;
configCross = configWithPlatform hostPlatform.platform;

nativeDrv = lib.addPassthru kernel.nativeDrv passthru;

crossDrv = lib.addPassthru kernel.crossDrv passthru;
in if kernel ? crossDrv then nativeDrv // { inherit nativeDrv crossDrv; } else lib.addPassthru kernel passthru

in if kernel ? crossDrv
then nativeDrv // { inherit nativeDrv crossDrv; }
else lib.addPassthru kernel passthru
7 changes: 0 additions & 7 deletions pkgs/os-specific/linux/kernel/linux-4.12.nix
Original file line number Diff line number Diff line change
@@ -8,11 +8,4 @@ import ./generic.nix (args // rec {
url = "mirror://kernel/linux/kernel/v4.x/linux-${version}.tar.xz";
sha256 = "09zxmknh6awhqmj8dyq95bdlwcasryy35hkjxjlzixdgn52kzaw6";
};

kernelPatches = args.kernelPatches;

features.iwlwifi = true;
features.efiBootStub = true;
features.needsCifsUtils = true;
features.netfilterRPFilter = true;
} // (args.argsOverride or {}))
7 changes: 0 additions & 7 deletions pkgs/os-specific/linux/kernel/linux-4.13.nix
Original file line number Diff line number Diff line change
@@ -8,11 +8,4 @@ import ./generic.nix (args // rec {
url = "mirror://kernel/linux/kernel/v4.x/linux-${version}.tar.xz";
sha256 = "1ayai3i0jakxzszpijhknjiwwi055wa74bpmnr0n7dh2l5s2rlh3";
};

kernelPatches = args.kernelPatches;

features.iwlwifi = true;
features.efiBootStub = true;
features.needsCifsUtils = true;
features.netfilterRPFilter = true;
} // (args.argsOverride or {}))
7 changes: 0 additions & 7 deletions pkgs/os-specific/linux/kernel/linux-4.9.nix
Original file line number Diff line number Diff line change
@@ -8,11 +8,4 @@ import ./generic.nix (args // rec {
url = "mirror://kernel/linux/kernel/v4.x/linux-${version}.tar.xz";
sha256 = "12mag09scyzi5a352y39y4b6rjh89qqca53hhmjc396q09hsdyl3";
};

kernelPatches = args.kernelPatches;

features.iwlwifi = true;
features.efiBootStub = true;
features.needsCifsUtils = true;
features.netfilterRPFilter = true;
} // (args.argsOverride or {}))
7 changes: 0 additions & 7 deletions pkgs/os-specific/linux/kernel/linux-hardened-copperhead.nix
Original file line number Diff line number Diff line change
@@ -27,11 +27,4 @@ import ./generic.nix (args // {
repo = "linux-hardened";
rev = "${version}.${revision}";
};

kernelPatches = args.kernelPatches;

features.iwlwifi = true;
features.efiBootStub = true;
features.needsCifsUtils = true;
features.netfilterRPFilter = true;
} // (args.argsOverride or {}))
8 changes: 1 addition & 7 deletions pkgs/os-specific/linux/kernel/linux-mptcp.nix
Original file line number Diff line number Diff line change
@@ -15,8 +15,6 @@ import ./generic.nix (args // rec {
sha256 = "0vqjnkzcbbvyq24w3cryfmw7hhws1xqkkxqcv71szkbqqs6mcr14";
};

kernelPatches = args.kernelPatches;

extraConfig = ''
IPV6 y
MPTCP y
@@ -41,10 +39,6 @@ import ./generic.nix (args // rec {
TCP_CONG_OLIA m
TCP_CONG_WVEGAS m
TCP_CONG_BALIA m
'';
features.iwlwifi = true;
features.efiBootStub = true;
features.needsCifsUtils = true;
features.netfilterRPFilter = true;
'' + (args.extraConfig or "");
} // (args.argsOverride or {}))
6 changes: 3 additions & 3 deletions pkgs/os-specific/linux/kernel/linux-rpi.nix
Original file line number Diff line number Diff line change
@@ -15,9 +15,9 @@ stdenv.lib.overrideDerivation (import ./generic.nix (args // rec {
sha256 = "1ly0x7a43zvig0fv7lc6rpq49pcdb7i9sdb78p4gi5485zap40kb";
};

features.iwlwifi = true;
features.needsCifsUtils = true;
features.netfilterRPFilter = true;
features = {
efiBootStub = false;
} // (args.features or {});

extraMeta.hydraPlatforms = [];
})) (oldAttrs: {
5 changes: 0 additions & 5 deletions pkgs/os-specific/linux/kernel/linux-samus-4.12.nix
Original file line number Diff line number Diff line change
@@ -14,10 +14,5 @@ import ./generic.nix (args // rec {
sha256 = "1dr74i79p8r13522w2ppi8gnjd9bhngc9d2hsn91ji6f5a8fbxx9";
}; in "${upstream}/build/linux";

features.iwlwifi = true;
features.efiBootStub = true;
features.needsCifsUtils = true;
features.netfilterRPFilter = true;

extraMeta.hydraPlatforms = [];
} // (args.argsOverride or {}))
5 changes: 0 additions & 5 deletions pkgs/os-specific/linux/kernel/linux-testing-bcachefs.nix
Original file line number Diff line number Diff line change
@@ -16,11 +16,6 @@ import ./generic.nix (args // rec {
BCACHEFS_FS m
'';

features.iwlwifi = true;
features.efiBootStub = true;
features.needsCifsUtils = true;
features.netfilterRPFilter = true;

# Should the testing kernels ever be built on Hydra?
extraMeta.hydraPlatforms = [];

5 changes: 0 additions & 5 deletions pkgs/os-specific/linux/kernel/linux-testing.nix
Original file line number Diff line number Diff line change
@@ -10,11 +10,6 @@ import ./generic.nix (args // rec {
sha256 = "1xkfxsvdhd0xcfjcr78222sa58i02z2ca9fv804jbyp7w9g628rm";
};

features.iwlwifi = true;
features.efiBootStub = true;
features.needsCifsUtils = true;
features.netfilterRPFilter = true;

# Should the testing kernels ever be built on Hydra?
extraMeta.hydraPlatforms = [];