You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Benno Evers <be...@mesosphere.com> on 2019/08/14 09:58:26 UTC
Re: Review Request 71206: Removed old mesos-style and references.
-----------------------------------------------------------
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
>
>