-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37860/#review96782
-----------------------------------------------------------



specs/rpm/aurora.spec (line 100)
<https://reviews.apache.org/r/37860/#comment152429>

    Can you explain the rationale behind this change - unpinned mesos runs the 
risk of not working with the executor it was compiled against. Much as we might 
wish that this will work, we don't know that it will and it's much easier to 
debug an error from yum than a runtime linker issue. If a user is convinced 
that we're wrong here they can always override with `--nodeps`.


- Kevin Sweeney


On Aug. 27, 2015, 3:50 p.m., Bill Farner wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37860/
> -----------------------------------------------------------
> 
> (Updated Aug. 27, 2015, 3:50 p.m.)
> 
> 
> Review request for Aurora and Kevin Sweeney.
> 
> 
> Repository: aurora-packaging
> 
> 
> Description
> -------
> 
> Rpm: fix install dep on daemonize, be permissive with newer mesos versions.
> 
> 
> Diffs
> -----
> 
>   specs/rpm/aurora.spec 3e7b8bef2272d935f3efb3e3ea6e4ac510026b7d 
> 
> Diff: https://reviews.apache.org/r/37860/diff/
> 
> 
> Testing
> -------
> 
> Successfully ran what jenkins runs:
> ```
> ./build-artifact.sh builder/rpm/centos-7 ~/snapshot.tar.gz 
> 0.10.0snapshot.2015.08.27
> ```
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Reply via email to