juspay / omnix

🚧 A Nix wrapper to improve developer experience
https://omnix.page
GNU Affero General Public License v3.0
70 stars 5 forks source link

`om show` should handle all systems #193

Open srid opened 1 month ago

srid commented 1 month ago
image

The flake in question is from a private repo, but here's a copy:

{
  inputs = {
    nixpkgs.url = "github:nixos/nixpkgs/nixpkgs-unstable";
    flake-parts.url = "github:hercules-ci/flake-parts";
  };
  outputs = inputs:
    inputs.flake-parts.lib.mkFlake { inherit inputs; } {
      systems = [ "x86_64-linux" ]; # iconv doesn't work on macOS
      perSystem = { pkgs, lib, ... }: {
        formatter = pkgs.nixpkgs-fmt;
        apps.default.program = lib.getExe (pkgs.writeShellApplication {
          // snip fields
        });
        devShells.default = pkgs.mkShell {
          packages = [ pkgs.httrack pkgs.wget ];
        };
      };
    };
}

Notice the systems = [ "x86_64-linux" ] - which means running om show won't show anything. Perhaps it should still show other systems. Maybe spit out an error saying no output for current system, and that the user can run om show . --all-systems to display for all systems? Need to figure out UX.

srid commented 3 weeks ago

Consider how this would work if IFD is used.