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 system PATH overflow with modular boost library paths #910

Open
db4 opened this issue Aug 12, 2019 · 4 comments
Open

Windows system PATH overflow with modular boost library paths #910

db4 opened this issue Aug 12, 2019 · 4 comments
Labels

Comments

@db4
Copy link

db4 commented Aug 12, 2019

Can we exclude this:

https://github.com/bincrafters/conan-boost_base/blob/06b161a93da699d70d8fe7df162b30e7078a243c/conanfile.py#L716

at least for Windows static build? PATH size is quite limited there (see Limitation to the length of the System PATH variable), and numerous modular boost lib paths easily overflow it. And once it's overflown everything silently stops working.

Of course, it's only a workaround, not a real solution. If someone can propose a real one (maybe on conan.io level?) I would love to know it.

@db4 db4 added the bug label Aug 12, 2019
@SSE4
Copy link
Member

SSE4 commented Aug 13, 2019

I think it should be safe to remove for static.
@uilianries @solvingj @grafikrobot WDYT?

@uilianries
Copy link
Member

@db4
Copy link
Author

db4 commented Aug 15, 2019

So short paths is not enough?

Looks like that.

What about enabling long paths?

They won't help. The problem is that max PATH environment variable size is quite limited, even under Windows 10 (see the Intel article that I mentioned above).

@SSE4
Copy link
Member

SSE4 commented Aug 18, 2019

I suggest to make proposed change, as it's useless to pollute binpaths for static builds anyway

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants