services.xserver.videoDrivers allows arbitrary value

[chrissound]

Issue description

  services.xserver = {
    #videoDrivers = [ "nvidia" ];
    videoDrivers = [ "aoeueoeuo8euoarue8euoa8" ];

The above would 'work' fine - a nixos-rebuild switch reports no errors, when really it should report an error?

[Ma27]

The current type declaration in the module says types.attrs types.str, so indeed any string can be specified as video driver.

However this might be confusion, but isn't actually a problem as each string is filtered out by the attrByPath expression while generating the X11 config.

I'm not sure if I'm missing anything, but actually it should be possible to replace the else null by a throw to ensure that evaluation breaks if an invalid driver is specified.

[stale]

Thank you for your contributions.

This has been automatically marked as stale because it has had no activity for 180 days.

If this is still important to you, we ask that you leave a comment below. Your comment can be as simple as "still important to me". This lets people see that at least one person still cares about this. Someone will have to do this at most twice a year if there is no other activity.

Here are suggestions that might help resolve this more quickly:

1. Search for maintainers and people that previously touched the related code and @ mention them in a comment.
2. Ask on the NixOS Discourse.
3. Ask on the #nixos channel on irc.freenode.net.