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

Discrakt should find credentials.ini relative to executable path and instead of the current working directory #47

Closed
1 task done
sitiom opened this issue Jan 21, 2023 · 2 comments · Fixed by #78
Closed
1 task done

Comments

@sitiom
Copy link
Contributor

sitiom commented Jan 21, 2023

🔍 Is there already an issue for your problem?

  • I have checked older issues, open and closed

📝 Description

I have added discrakt.exe in my PATH but running it gave me this error:

> discrakt
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: "couldn't open credentials.ini: The system cannot find the file specified. (os error 2)"', src\utils.rs:13:36
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

The credentials.ini file is inside the discrakt folder, and indeed changing my directory to that folder works. Discrakt should try to find credentials.ini relative to the executable path and instead on the current working directory.

🪜 Steps To Reproduce

  1. Configure credentials.ini
  2. Add discrakt.exe to the PATH
  3. Running discrakt will fail anywhere except in the executable directory

ℹ System Info

Windows 11 22H2
Discrakt v2.2.4

Note: Compiled it myself because of #45

📸 Screenshots

No response

@sitiom sitiom changed the title Discrakt should find credentials.ini relative to executable path and not on the current working directory Discrakt should find credentials.ini relative to executable path and instead of the current working directory Jan 21, 2023
@sitiom
Copy link
Contributor Author

sitiom commented Mar 6, 2024

@afonsojramos credentials.ini is missing on the latest release. Is this expected? The Scoop package is failing to update because of it.

@afonsojramos
Copy link
Owner

It seems to have been removed in the work of #52. Re-adding!

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

Successfully merging a pull request may close this issue.

2 participants