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

Windows doesn't like the log_disabled symlink #149

Closed
azriel91 opened this issue Oct 1, 2019 · 1 comment
Closed

Windows doesn't like the log_disabled symlink #149

azriel91 opened this issue Oct 1, 2019 · 1 comment

Comments

@azriel91
Copy link
Contributor

azriel91 commented Oct 1, 2019

Hiya, just discovered that on Windows, the log_disabled.rs symlink isn't happily interpreted, so it fails to build when the "logging" feature is disabled (which is the default).

Error message:

error[E0583]: file not found for module `log_disabled`
   --> D:\apps\gitlab_runner\.cargo\git\checkouts\rust-derive-builder-ac2766f218331415\67c9c95\derive_builder\src\lib.rs:576:5
    |
576 | mod log_disabled;
    |     ^^^^^^^^^^^^
    |
    = help: name the file either log_disabled.rs or log_disabled\mod.rs inside the directory "D:\apps\gitlab_runner\.cargo\git\checkouts\rust-derive-builder-ac2766f218331415\67c9c95\derive_builder\src"

I haven't thought of a solution that I like, though to "make it work ™️" one could:

  • enable the "logging" feature
  • copy paste the file in here
  • somehow convince every windows CI machine to follow the symlink

software: 1, humans: 0

@azriel91
Copy link
Contributor Author

azriel91 commented Oct 1, 2019

Figured it out. A crate with derive_builder = "0.8.0" in its dependencies works. But a crate with the following doesn't:

derive_builder = { git = "https://github.com/colin-kiegel/rust-derive-builder.git" }

Both my user and the CI user have this:

$ git --version
git version 2.23.0.windows.1

$ git config core.symlinks
true

So it looks like Cargo doesn't enable core.symlinks on windows for git dependencies. Shall follow up with an issue there.

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