* Get fbcondecor to work.

* Barf on ignored options.

svn path=/nixpkgs/branches/kernel-config/; revision=18914

pkgs/os-specific/linux/kernel/builder.sh

@@ -1,6 +1,10 @@
 source $stdenv/setup
 
 
+export SHELL=bash
+export ARCH=$arch
+
+
 configurePhase() {
     if test -n "$preConfigure"; then 
         eval "$preConfigure"; 
@@ -10,12 +14,8 @@ configurePhase() {
     export INSTALL_MOD_PATH=$out
 
 
-    # Get rid of any "localversion" files installed by patches.
-    if test -z "$allowLocalVersion"; then
-        rm -f localversion*
-    fi
-
     # Set our own localversion, if specified.
+    rm -f localversion*
     if test -n "$localVersion"; then
         echo "$localVersion" > localversion-nix
     fi
@@ -28,7 +28,7 @@ configurePhase() {
     # Create the config file.
     echo "generating kernel configuration..."
     echo "$kernelConfig" > kernel-config
-    ARCH=$arch KERNEL_CONFIG=kernel-config SHELL=bash NIX_INDENT_MAKE= perl -w $generateConfig
+    KERNEL_CONFIG=kernel-config perl -w $generateConfig
 }
 
 

pkgs/os-specific/linux/kernel/generate-config.pl

@@ -123,7 +123,7 @@ while (<CONFIG>) {
 close CONFIG;
 
 foreach my $name (sort (keys %answers)) {
-    print STDERR "unused option: $name\n"
+    die "unused option: $name\n"
         unless defined $config{$name};
     die "option not set correctly: $name\n"
         if $config{$name} && $config{$name} ne $answers{$name};

pkgs/os-specific/linux/kernel/generic.nix

@@ -56,9 +56,7 @@ stdenv.mkDerivation {
 
   generateConfig = ./generate-config.pl;
 
-  inherit preConfigure;
+  inherit preConfigure src module_init_tools localVersion;
-
-  inherit src config;
 
   patches = map (p: p.patch) kernelPatches;
 
@@ -77,13 +75,6 @@ stdenv.mkDerivation {
     if stdenv.system == "x86_64-linux" then "x86_64" else
     abort "Platform ${stdenv.system} is not supported.";
 
-  makeFlags = if userModeLinux then "ARCH=um SHELL=bash" else "";
-
-  inherit module_init_tools;
-
-  allowLocalVersion = false; # don't allow patches to set a suffix
-  inherit localVersion; # but do allow the user to set one.
-
   meta = {
     description =
       (if userModeLinux then

pkgs/os-specific/linux/kernel/linux-2.6.29.nix

@@ -33,7 +33,9 @@ import ./generic.nix (
         XEN y
 
         # We need 64 GB (PAE) support for Xen guest support.
+        ${stdenv.lib.optionalString (stdenv.system == "i686-linux") ''
           HIGHMEM64G y
+        ''}
 
         # Enable the kernel's built-in memory tester.
         MEMTEST y
@@ -91,16 +93,6 @@ import ./generic.nix (
 
         # Some settings to make sure that fbcondecor works - in particular,
         # disable tileblitting and the drivers that need it.
-        FB y
-        FB_TILEBLITTING n
-        FB_MATROX n
-        FB_S3 n
-        FB_VT8623 n
-        FB_ARK n
-        FB_CFB_FILLRECT y
-        FB_CFB_COPYAREA y
-        FB_CFB_IMAGEBLIT y
-        FB_VESA y
 
         # Enable various FB devices.
         FB_EFI y

pkgs/top-level/all-packages.nix

@@ -5477,7 +5477,23 @@ let
           url = http://dev.gentoo.org/~spock/projects/fbcondecor/archive/fbcondecor-0.9.6-2.6.29.2.patch;
           sha256 = "1yppvji13sgnql62h4wmskzl9l198pp1pbixpbymji7mr4a0ylx1";
         };
-        extraConfig = "CONFIG_FB_CON_DECOR=y";
+        extraConfig =
+          ''
+            FB_CON_DECOR y
+
+            # fbcondecor is picky about some other settings.
+            FB y
+            FB_TILEBLITTING n
+            FB_MATROX n
+            FB_S3 n
+            FB_VT8623 n
+            FB_ARK n
+            FB_CFB_FILLRECT y
+            FB_CFB_COPYAREA y
+            FB_CFB_IMAGEBLIT y
+            FB_VESA y
+            FRAMEBUFFER_CONSOLE y
+          '';
         features = { fbConDecor = true; };
       }
       { name = "sec_perm-2.6.24";