Omnix fails to parse `nix --version`

[pizzapim]

Omnix started failing to parse my server's Nix version:

✍️ Overwriting server "build-cache"
👟 Reading [om.ci](http://om.ci/) config from flake
❄️ nix eval --json '.#om' --quiet --quiet️
👟 Gathering NixInfo
❄️ nix --version️
Error: Unable to gather nix info
Caused by:
0: Nix command error: Failed to decode command stdout (from_str error): Failed to parse string: Parse error (int): `nix --version` cannot be parsed
1: Failed to decode command stdout (from_str error): Failed to parse string: Parse error (int): `nix --version` cannot be parsed
2: Failed to parse string: Parse error (int): `nix --version` cannot be parsed

My Nix version is:

# nix --version
nix (Nix) 2.31.2+1

Perhaps it's failing on the +1? This server is running Nixpkgs unstable

[srid]

How do we reproduce this?

[MikaelElkiaer]

I am also seeing this running unstable.

nix-env --install --file '<nixpkgs>' --attr nix cacert -I nixpkgs=channel:nixpkgs-unstable
nix --accept-flake-config profile install github:juspay/omnix
om show .

[HarrisonCentner]

This happens to me when I bump nixpkgs-unstable.

Ignoring the build metadata in the nix version should be sufficient. The updated regex in crates/nix_rs/src/version.rs can be r"(?:nix \(Nix\) )?(\d+)\.(\d+)\.(\d+)(\+(\d+))?$".

I'm happy to submit a PR but don't have permissions create a new branch.

[AndreiCravtov]

Also getting this on a fresh nixos install. just followed the nixos-unified-template guide and now I am getting this error

[AndreiCravtov]

This happens to me when I bump nixpkgs-unstable.

Ignoring the build metadata in the nix version should be sufficient. The updated regex in crates/nix_rs/src/version.rs can be r"(?:nix \(Nix\) )?(\d+)\.(\d+)\.(\d+)(\+(\d+))?$".

I'm happy to submit a PR but don't have permissions create a new branch.

I submitted a PR #489 doing just that, hopefully someone can review it relatively quickly.

[shivaraj-bh]

Note: the +1 suffix is a nixpkgs thing (see https://github.com/NixOS/nixpkgs/blob/1306659b587dc277866c7b69eb97e5f07864d8c4/pkgs/tools/package-management/nix/modular/packaging/components.nix#L100-L104), which denotes the number of patches applied on the official release. Upstream still follows major.minor.patch versioning scheme.

[shivaraj-bh]

NixOS/nixpkgs#472294

[imide]

Perhaps re-open this issue? I am encountering the same issue.
om show . output:

Error: Nix command error: Failed to decode command stdout (from_str error): Failed to parse string: Parse error (int): `nix --version` cannot be parsed

Caused by:
    0: Failed to decode command stdout (from_str error): Failed to parse string: Parse error (int): `nix --version` cannot be parsed
    1: Failed to parse string: Parse error (int): `nix --version` cannot be parsed

nix --version output (private info replaced with REMOVED):

nix (Lix, like Nix) 2.94.0
System type: aarch64-darwin
Additional system types: x86_64-darwin
Features: gc, signed-caches
System configuration file: /etc/nix/nix.conf
User configuration files: /Users/REMOVED/.config/nix/nix.conf:/Users/REMOVED/.nix-profile/etc/xdg/nix/nix.conf:/etc/profiles/per-user/REMOVED/etc/xdg/nix/nix.conf:/run/current-system/sw/etc/xdg/nix/nix.conf:/nix/var/nix/profiles/default/etc/xdg/nix/nix.conf
Store directory: /nix/store
State directory: /nix/var/nix
Data directory: /nix/store/gxpdrn2sw1xpymxmsrxc7c706b7higws-lix-2.94.0/share

Version: omnix-cli 1.3.2
Seems like a simple fix however. Just change the regex in crates/nix_rs/src/version.rs. I cannot do regex so I am unable to contribute, sorry!

[Daholli]

I think this could work:

(?m)(?:nix \([^)]*\) )?(\d+)\.(\d+)\.(\d+)(?:\+(\d+))?$

RegexBuilder

[Daholli]

#504 should fix the lix issue