Giter Site home page Giter Site logo

Comments (9)

glandium avatar glandium commented on June 6, 2024

Related to #1730 ?

from sccache.

szymonj99 avatar szymonj99 commented on June 6, 2024

I think the error is unrelated, but the fix is what's causing the issue (correct me if I'm wrong) (I've compiled sccache from source with commit 6bfe2cbdbd4bfceecadd35a9c5b19586eb7161f2 from 19th June, and the issue persisted; I had no issues with the warning that the original PR references (error C4668: '__midl' is not defined as a preprocessor macro, replacing with '0' for '#if/#elif'). I'm running version 0.5.4 right now which has that fix included.

from sccache.

sylvestre avatar sylvestre commented on June 6, 2024

cc @Alexei-Barnes if you have an idea

from sccache.

cctv130 avatar cctv130 commented on June 6, 2024

After testing, you can add comments to wd4668, in msvc.rs, then recompile sccache, after that you can use clang-cl.

from sccache.

szymonj99 avatar szymonj99 commented on June 6, 2024

I'll do more testing on my side and report back; I seem to have fixed the issue now, and I'm not 100% sure why.

from sccache.

cctv130 avatar cctv130 commented on June 6, 2024

I'll do more testing on my side and report back; I seem to have fixed the issue now, and I'm not 100% sure why.

clang-cl, as I know, is not fully compatible with msvc, so it seems that clang-cl will not handle the wd4668 error, while msvc will handle the wd4668 error.

from sccache.

aeubanks avatar aeubanks commented on June 6, 2024

clang-cl handles /wd4668 just fine, the actual problem is that it's getting passed at the very end of the command line after a --, so it gets treated as a file to compile rather than a flag

the same thing happens with MSVC, it's just that MSVC doesn't recognize -- and I see the following when using sccache with cl (and don't see it without sccache)

cl : Command line warning D9002 : ignoring unknown option '--'

so MSVC treats /wd4668 at the end of the command like a flag

from sccache.

glandium avatar glandium commented on June 6, 2024

So, this is a dupe of #1498 then

from sccache.

aeubanks avatar aeubanks commented on June 6, 2024

in some sense yes, fixing #1498 would also fix this, but the currently proposed fix #1856 is also reasonable but would only fix this

from sccache.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.