community icon indicating copy to clipboard operation
community copied to clipboard

Windows system PATH overflow with modular boost library paths

Open db4 opened this issue 6 years ago • 4 comments
trafficstars

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 avatar Aug 12 '19 09:08 db4

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

SSE4 avatar Aug 13 '19 18:08 SSE4

So short paths is not enough?

What about enabling long paths?

https://docs.microsoft.com/en-us/windows/win32/fileio/naming-a-file#enable-long-paths-in-windows-10-version-1607-and-later

uilianries avatar Aug 14 '19 20:08 uilianries

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).

db4 avatar Aug 15 '19 11:08 db4

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

SSE4 avatar Aug 18 '19 09:08 SSE4