Android cross-compiling from Windows error "cargo:warning=The command line is too long" about physx-rs HOT 14 CLOSED

from physx-rs.

Figured it out! Fix is on it's way rust-lang/cc-rs#572 :)

from physx-rs.

The latest version of cc fixes this issue!

from physx-rs.

@Jasper-Bekkers you were looking into this right?

from physx-rs.

Reduced to roughly 8257 chars when using -no-everything. So close yet so far...

from physx-rs.

@Jasper-Bekkers any further progress on this?

from physx-rs.

rust-lang/cc-rs#547 solves this.

from physx-rs.

Awesome!

from physx-rs.

Alright, I've been looking into this issue quite a bit this week and am majorly confused.

The error @repi shows above shouldn't actually error at all. The reason being, yes the command is 10k+ lines of code but cc spawns the command as a new process, whose command limit is 32767 characters according to this blog. I've compared the command length of that same command on a msvc -> msvc cl.exe invocation and it was also over 10k+ characters long but doesn't error.

Linking would actually fail on msvc -> Android as cc tried to link every .o file in one go, which would result in a command over 32k+ characters. When targeting msvc this was solved by using a response file but not when targeting Android. This issue I fixed by progressive linking, which has been merged in cc.

TLDR: no clue why we get this error :(

from physx-rs.

@maxded ouch. you are able to reproduce the error locally on Windows right though?

from physx-rs.

Yes

from physx-rs.

Would it be possible to reduce the number of include directories when compiling physx-sys itself instead of relying on a fix in cc-rs? My fix mentioned above reintroduces response files to cc-rs when they just recently got completely removed and the maintainer would prefer if they stayed removed.

from physx-rs.

You can try, but the way the physx code works means a lot of includes are needed.

from physx-rs.

Awesome!

from physx-rs.