-
-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
marathon: 1.3.6 -> 1.4.1 #22935
marathon: 1.3.6 -> 1.4.1 #22935
Conversation
@pradeepchhetri, thanks for your PR! By analyzing the history of the files in this pull request, we identified @rushmorem, @kamilchm and @pSub to be potential reviewers. |
It seems some tests are failing on travis. |
I am having a look at it. Thanks. |
This seems like a more general error with building the Python bindings for Protobuf 2.6. So no program depending on this version will build OK. For example Unfortunately current Mesos doesn't work with newer protobuf versions, apparently it will in an upcoming release (maybe 1.3.0?), see MESOS-5186 and MESOS-6138. |
I tried applying #22061 and then build this. That worked quite well both Mesos and Marathon built successfully. |
So I guess we can merge this once #22061 gets merged into master ? |
Yeah, I would be in favor of that. |
That PR has been merged. @rycee can you take another look? |
c5bd4c4
to
ecca0c9
Compare
Motivation for this change
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)