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



After applying this series for testing purposes, I ran into a problem where 
`mesos-style.py` was still referenced from a git hook, breaking my ability to 
commit without non-trivial investigation of the problem. (and I had the 
advantage of *knowing* that I just applied this patch series, if we commit it 
to master some other developers will be caught by suprise, no matter how 
heavily it is advertised.)

I'd suggest instead of removing `mesos-style.py` outright, we replace it with a 
simple bash script that just prints something like `WARNING: 'mesos-style.py' 
is not supported anymore and does nothing, please fix whatever was calling this 
script. See MESOS-9630.`

- Benno Evers


On July 30, 2019, 9:01 p.m., Benjamin Bannier wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/71206/
> -----------------------------------------------------------
> 
> (Updated July 30, 2019, 9:01 p.m.)
> 
> 
> Review request for mesos and Till Toenshoff.
> 
> 
> Bugs: MESOS-9360
>     https://issues.apache.org/jira/browse/MESOS-9360
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> This patch removes references to `support/mesos-style.py` which was
> replaced with a pre-commit setup in a previous commit. We also remove
> the tool itself.
> 
> 
> Diffs
> -----
> 
>   docs/c++-style-guide.md 8a48afe780f23736c9b7abeb7337977521cecfa5 
>   support/build-virtualenv 7dc03b054f7663979e4eb4b11ad51d759b7f1ad3 
>   support/hooks/commit-msg a0c218deee3fb4b7594fe39b76c1025045ba0725 
>   support/hooks/post-rewrite 1ab14abf711d1923a7ae69beb33581317009a94a 
>   support/hooks/pre-commit 519567bf5f20a74b273c8d8514577fe4342dc45d 
>   support/mesos-style.py cd490bd3deeb8477295bbad010f36d984ce17d27 
> 
> 
> Diff: https://reviews.apache.org/r/71206/diff/1/
> 
> 
> Testing
> -------
> 
> n/a
> 
> 
> Thanks,
> 
> Benjamin Bannier
> 
>

Reply via email to