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

environment.yml is overcomplete (and Linux specific) #71

Closed
KrisThielemans opened this issue Sep 18, 2023 · 3 comments
Closed

environment.yml is overcomplete (and Linux specific) #71

KrisThielemans opened this issue Sep 18, 2023 · 3 comments
Labels
wontfix This will not be worked on

Comments

@KrisThielemans
Copy link

On Windows from Powershell.

mamba env create  --file environment.yml

Looking for: ['bash-completion=2.11', 'ccache=4.5.1', 'clang-format=14.0.4', 'cmake=3.21.3', 'fmt=8.1.1', 'gcc_linux-64]

Could not solve for environment specs
Encountered problems while solving:
  - nothing provides requested bash-completion 2.11**
  - nothing provides requested gcc_linux-64 11.2.0**
  - nothing provides requested gdb 11.2**
  - nothing provides requested gxx_linux-64 11.2.0**
  - nothing provides requested valgrind 3.18.1**

The environment can't be solved, aborting the operation

I guess we should remove valgrind and gdb? Even bash_completion and ccache. Maybe even clang-format.

Of course, the justfile is bash/Linux specific as well and I guess Windows support is for later.

PS: Is pinning the compiler version etc best practice? Maybe some of these could be >=?

@hansenms
Copy link
Contributor

hansenms commented Sep 18, 2023

This environment file is not meant to work on Windows it is to set up a full development environment (for tool and all tests) in a Linux devcontainer. If you just want to work with this in Windows, follow the quickstart instructions for C++ or Python.

And we do need those tools in the devcontainer to have a fully functional development environment.

If you just want to use the yardl too, grab a release from here: https://github.com/microsoft/yardl/releases/

@hansenms hansenms added the wontfix This will not be worked on label Sep 18, 2023
@KrisThielemans
Copy link
Author

aha! Great. My mistake. I was reading https://github.com/microsoft/yardl/tree/v0.3.0#building-the-code-in-this-repo, but didn't read the quickstart. (Usual problem with documentation: people read the wrong bit 😉 ).

Maybe the README.md could say "if you want to use yardl, get the executable from the latest release"?

@johnstairs
Copy link
Member

Going to close this. In the README, we have a "getting started" link (appearing before the contributing section)

@johnstairs johnstairs closed this as not planned Won't fix, can't repro, duplicate, stale Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

3 participants