Skip to content

Commit

Permalink
container.nix -> docker-container.nix
Browse files Browse the repository at this point in the history
  • Loading branch information
edolstra committed Apr 19, 2015
1 parent 57b0576 commit 1cb5583
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -45,10 +45,6 @@ in {
${config.nix.package}/bin/nix-env -p /nix/var/nix/profiles/system --set /run/current-system
'';

# Disable some features that are not useful in a container.
sound.enable = mkDefault false;
services.udisks2.enable = mkDefault false;

# Install new init script
system.activationScripts.installInitScript = ''
ln -fs $systemConfig/init /init
Expand Down
2 changes: 1 addition & 1 deletion nixos/modules/virtualisation/docker-image.nix
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

{
imports = [
../profiles/container.nix
../profiles/docker-container.nix # FIXME, shouldn't include something from profiles/
];

boot.postBootCommands =
Expand Down
2 changes: 1 addition & 1 deletion nixos/modules/virtualisation/lxc-container.nix
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ with lib;

{
imports = [
../profiles/container.nix
../profiles/docker-container.nix # FIXME, shouldn't include something from profiles/

This comment has been minimized.

Copy link
@rickynils

rickynils Sep 7, 2016

Member

@edolstra Why should things from profiles/ not be included here? And, more generally, what should be placed in profiles/ and what should be placed in virtualisation/? In profiles/ we have for example qemu-guest.nix and docker-container.nix, and here we have container-config.nix etc.

];

# Allow the user to login as root without password.
Expand Down

0 comments on commit 1cb5583

Please sign in to comment.