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
bazel: 0.27.0 -> 0.28.0 #64633
bazel: 0.27.0 -> 0.28.0 #64633
Conversation
@GrahamcOfBorg build bazel.tests |
I dont understand the failure in |
@GrahamcOfBorg eval |
@guibou it was an unrelated issue on master which has been resolved |
@Profpatsch suggested that I used an import from derivation which may be forbidden in nixpkgs. Is that true ? |
Ah, there is indeed something else. I just thought, without really looking it, it was the same as the other PR's. |
Indeed, you are using import from derivation:
|
--distdir is now used in the installCheckPhase for prefetched repositories. That's simpler, more robust and easier to extend in the future. Note that `name` argument of fetchurl was removed because it changed the basename of the generated file and bazel uses this basename for its cache behavior.
I removed the import from derivation refactoring. Sorry I was not aware that it was not allowed in nixpkgs. |
@GrahamcOfBorg build bazel bazel.tests |
@guibou looks like the update broke the protocol buffer test on Darwin.
|
@kalbasit test for tentative fix in progress. @GrahamcOfBorg build bazel bazel.tests (I'm not sure @GrahamcOfBorg is obeying me, so if you have more control over him, please run the tests) |
@GrahamcOfBorg build bazel bazel.tests |
Any darwin user can help me to fix the build failure. I don't have access to a darwin machine. |
@guibou would an ssh connection help you? |
Yes, definitely.
…On Sun, Jul 14, 2019 at 10:23 PM Wael Nasreddine ***@***.***> wrote:
@guibou <https://github.com/guibou> would an ssh connection help you?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#64633?email_source=notifications&email_token=ACKBPDLRUVEDGYFKZWSLD5LP7ODNZA5CNFSM4IBKRKA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZ4MUTI#issuecomment-511232589>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACKBPDIUIGSM4XB7CJRDBZ3P7ODNZANCNFSM4IBKRKAQ>
.
|
Ok let's continue this discussion on Keybase, I'm @YLCodes over there. I'll get my mac ready in the meantime. |
I'm ready whenever you are! |
I tested this branch as it is now (320dd2def24e6dacd1d6b89b6fa705169d10bc43) on my mac and it's working. |
@uri-canva can you try on your mac to build |
Yep,
output
|
The ofborg darwin environment is a bit strange, it's not the same as a regular macOS nix installation, either with or without sandboxing enabled, or the hydra environment. I've hit issues with it before but I could never reproduce them, eventually I think we merged ignoring the ofborg results, and it worked fine on hydra. @groodt do you remember something more about that? |
I've squashed my "fixup" commit to clean a bit my history. Considering that @GrahamcOfBorg is unhappy about |
It's not the best option to merging with an unhappy build. Perhaps @LnL7 or @grahamc may help here? I'm going to test on my Darwin as well. EDIT: It works on my Mac as well. 🤔 |
@GrahamcOfBorg build bazel bazel.tests |
ok ofborg says that it's passing now. |
Thank you! |
Motivation for this change
Bazel bump to 0.28. I also refactored the following:
--distdir
, which removes a lot of boilerplate.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)