Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to run silver on Freebsd 12.2 #85

Open
ghost opened this issue Sep 30, 2021 · 1 comment
Open

Unable to run silver on Freebsd 12.2 #85

ghost opened this issue Sep 30, 2021 · 1 comment

Comments

@ghost
Copy link

ghost commented Sep 30, 2021

A freshly compiled silver via cargo will not run on FreeBSD

[chris@shell ~]$ uname -a 
FreeBSD REDACTED_HOST 12.2-RELEASE-p7 FreeBSD 12.2-RELEASE-p7 GENERIC  amd64
[chris@shell ~]$ silver init
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', /home/chris/.cargo/registry/src/github.com-1ecc6299db9ec823/silver-2.0.0/src/main.rs:43:63
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

I have run it with the backtrace on and received this:

[chris@shell ~]$ RUST_BACKTRACE=1 silver init
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', /home/chris/.cargo/registry/src/github.com-1ecc6299db9ec823/silver-2.0.0/src/main.rs:43:63
stack backtrace:
   0: rust_begin_unwind
             at /rustc/c8dfcfe046a7680554bf4eb612bad840e7631c4b/library/std/src/panicking.rs:515:5
   1: core::panicking::panic_fmt
             at /rustc/c8dfcfe046a7680554bf4eb612bad840e7631c4b/library/core/src/panicking.rs:92:14
   2: core::panicking::panic
             at /rustc/c8dfcfe046a7680554bf4eb612bad840e7631c4b/library/core/src/panicking.rs:50:5
   3: silver::main
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Since the full backtrace is verbose, I will hold off on that unless you need it. There is admittedly a good chance that this is a side effect of rust not being tier 1 on FreeBSD x86_64, but I wanted to report here before moving up the chain.

@inferiorhumanorgans
Copy link

This is likely not a FreeBSD issue as without a config file in place silver will crash on any platform. Unfortunately the config file structure cannot be serialized to TOML with serde.

toml-rs/toml-rs#145

toml-rs/toml-rs#407

inferiorhumanorgans pushed a commit to inferiorhumanorgans/silver that referenced this issue Mar 31, 2022
- The current config file layout cannot be serialized to TOML with serde,
  so switch to YAML.  This necessitates updating to an unreleased
  version of confy.  See: reujab#85, reujab#77, toml-rs/toml-rs#145,
  toml-rs/toml-rs#407.
- Print out the location of the default config file upon creation see:
  fixes reujab#73, fixes reujab#78.
- Set the default left prompt to *something* so the user isn't left
  scratching their head.
- Change the default icon set to Unicode so that this is usable out of
  the box with as few dependencies as possible.
inferiorhumanorgans pushed a commit to inferiorhumanorgans/silver that referenced this issue Mar 31, 2022
- The current config file layout cannot be serialized to TOML with serde,
  so switch to YAML.  This necessitates updating to an unreleased
  version of confy.  See: reujab#85, reujab#77, toml-rs/toml-rs#145,
  and toml-rs/toml-rs#407.
- Print out the location of the default config file upon creation see:
  fixes reujab#73, fixes reujab#78.
- Set the default left prompt to *something* so the user isn't left
  scratching their head.
- Change the default icon set to Unicode so that this is usable out of
  the box with as few dependencies as possible.
inferiorhumanorgans pushed a commit to inferiorhumanorgans/silver that referenced this issue Mar 31, 2022
- The current config file layout cannot be serialized to TOML with serde,
  so switch to YAML.  This necessitates updating to an unreleased
  version of confy.  See: reujab#85, reujab#77, toml-rs/toml-rs#145,
  and toml-rs/toml-rs#407.
- Print out the location of the default config file upon creation fixes:
  reujab#73 and reujab#78.
- Set the default left prompt to *something* so the user isn't left
  scratching their head.
- Change the default icon set to Unicode so that this is usable out of
  the box with as few dependencies as possible.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant