Remove deprecated `--use-old-naming-style` flag.

Review Request #3427 - Created Feb. 5, 2016 and submitted

Information
John Sirois
pants
jsirois/0.0.71/deprecation_removals
2892
Reviewers
pants-reviews
patricklaw, peiyu, stuhood
This kills support in the `export-classpath` goal for the old style of
classpath segregation that mirrored a target's namespace as a directory
tree in favor of using a flat set of files differentiated by including
the target's id in their names.

 src/python/pants/backend/jvm/tasks/classpath_util.py                                  | 25 ++++--------------
 src/python/pants/backend/jvm/tasks/jvm_compile/jvm_classpath_publisher.py             | 12 +--------
 tests/python/pants_test/backend/jvm/tasks/jvm_compile/test_jvm_classpath_published.py |  2 +-
 tests/python/pants_test/backend/jvm/tasks/test_classpath_util.py                      | 69 ++++++++++++++------------------------------------
 4 files changed, 26 insertions(+), 82 deletions(-)

Locally green ./build-support/bin/ci.sh

CI went green here:
https://travis-ci.org/pantsbuild/pants/builds/107374106

Issues

  • 0
  • 1
  • 0
  • 1
Description From Last Updated
Yi Cheng
Nick Howard (Twitter)
John Sirois
Nick Howard (Twitter)
Peiyu Wang
John Sirois
John Sirois
John Sirois
John Sirois
John Sirois
John Sirois
John Sirois
John Sirois
Review request changed

Status: Closed (submitted)

Change Summary:

Now on master:

git log -1 origin/master
commit e108d581f7db099c6613102bc73a4534d35f92b0
Author: John Sirois <john.sirois@gmail.com>
Date:   Fri Feb 5 21:47:16 2016 -0700

    Remove deprecated `--use-old-naming-style` flag.
    
    This kills support in the `export-classpath` goal for the old style of
    classpath segregation that mirrored a target's namespace as a directory
    tree in favor of using a flat set of files differentiated by including
    the target's id in their names.
    
    Testing Done:
    Locally green `./build-support/bin/ci.sh`
    
    CI went green here:
      https://travis-ci.org/pantsbuild/pants/builds/107374106
    
    Bugs closed: 2892
    
    Reviewed at https://rbcommons.com/s/twitter/r/3427/
Loading...