differential-privacy
differential-privacy copied to clipboard
Build error in Windows
Receiving following error after running "bazel build differential_privacy/…"
This is because we add a
copts = ["-Wno-sign-compare"],
line to every build rule. I guess that compiler flag isn't available on windows. It's not necessary, it just suppresses compiler warnings, so I'll look into making it platform-dependent. In the mean time, if you need a workaround you can remove it from the rules you're trying to build.
Is it available for windows now? I am still getting this error. If not, then how to remove it from rules.
Nope, we have not made any progress on this yet. The workaround is to remove the copts = ["-Wno-sign-compare"],
from the build rules you want to use on Windows.
Do @BenjaminDev's patches solve this error, @Dipinka, or is the workaround still necessary?
@TedTed as far as I'm aware my patches solved the issue. PyDP have the windows build as part of the ci and no one is complaining there. Just a note that PyDP is normally a few commits behind.
@Dipinka if you getting build issues take a look at the GitHub Actions for PyDP as it might shed some light on how we build it.
GitHub Actions for PyDP, which is not exist
It seems that file was moved/merged into https://github.com/OpenMined/PyDP/blob/dev/.github/workflows/publish.yml
I've not been working on PyDP in a while and a lot has changed. If google/differential-privacy
wants to support windows "officially" I'd suggest we add some form of CI that builds for windows - otherwise there will forever be breaks that go unnoticed until PyDP tries to do a release.
I'd be happy to have a go at making a google/differential-privacy/.github/workflows/windows.yml
which does a windows build. Then we can catch breaks when they happen. Just want to be sure we want to support windows in this way i.e if it breaks windows we fix before landing on main
.
We have an internal CI job that runs on Windows. However, it builds the C++ library without the postgres
and accounting
folders, i.e. using the following line:
bazelisk build -- ... -postgres/... -accounting/...
Could you provide more information about the error? Which commit of the library are you using and which targets cause the issue?
We have an internal CI job that runs on Windows
Oh, cool. I just happened to do the initial work to get the windows builds going and thought I could assist. If CI is happy then all good. Thanks @dibakch
I see. Thanks :)
Feel free to re-open if the windows build is failing. Everything under cc/postgres/
and cc/accounting
is not tested and is not supported for Windows.