Skip to content
This repository has been archived by the owner on Nov 6, 2023. It is now read-only.

Fix HTTP file by including Microsoft Windows support. #2

Merged
merged 4 commits into from Feb 17, 2023
Merged

Fix HTTP file by including Microsoft Windows support. #2

merged 4 commits into from Feb 17, 2023

Conversation

ghost
Copy link

@ghost ghost commented Feb 16, 2023

No description provided.

@ghost
Copy link
Author

ghost commented Feb 16, 2023

Before the changes:

error: environment variable `HOME` not defined
  --> src\pkg\core\http.rs:61:25
   |
61 |         format!("{}{}", env!("HOME"), cookie_path.trim_start_matches('~'))
   |                         ^^^^^^^^^^^^
   |
   = note: this error originates in the macro `env` (in Nightly builds, run with -Z macro-backtrace for more info)

error: could not compile `binggpt` due to previous error
warning: build failed, waiting for other jobs to finish...

After the changes:

C:\Users\shawn\GitHub\BingGPT>cargo build
   Compiling binggpt v0.1.2 (C:\Users\shawn\GitHub\BingGPT)
    Finished dev [unoptimized + debuginfo] target(s) in 3.63s

@ghost ghost marked this pull request as draft February 16, 2023 17:44
@ghost ghost marked this pull request as ready for review February 16, 2023 17:57
@jlvihv
Copy link
Owner

jlvihv commented Feb 17, 2023

thank you very much

@jlvihv jlvihv merged commit 9db5764 into jlvihv:main Feb 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant