New issue
Advanced search Search tips

Issue 640559 link

Starred by 0 users

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 3
Type: Bug



Sign in to add a comment

Remove class-dump from iossim

Project Member Reported by rohitrao@chromium.org, Aug 24 2016

Issue description

We should not need class-dump any more, since iossim shells out to xcrun and simctl now.
 
This requires all the bots running Xcode 8+
Project Member

Comment 2 by bugdroid1@chromium.org, Sep 21 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/17ca7445f3cd00d605a349152a610b8301b91fba

commit 17ca7445f3cd00d605a349152a610b8301b91fba
Author: justincohen <justincohen@chromium.org>
Date: Wed Sep 21 03:13:03 2016

Remove old iossim code (pre-Xcode 8) and class-dump.

BUG= 640559 
TBR=brettw

Review-Url: https://codereview.chromium.org/2351613006
Cr-Commit-Position: refs/heads/master@{#419962}

[modify] https://crrev.com/17ca7445f3cd00d605a349152a610b8301b91fba/.gitignore
[modify] https://crrev.com/17ca7445f3cd00d605a349152a610b8301b91fba/DEPS
[modify] https://crrev.com/17ca7445f3cd00d605a349152a610b8301b91fba/testing/iossim/BUILD.gn
[modify] https://crrev.com/17ca7445f3cd00d605a349152a610b8301b91fba/testing/iossim/iossim.mm
[delete] https://crrev.com/a7040a0af3abeca232b0b4bf493d30f070e37d86/third_party/class-dump/BUILD.gn
[delete] https://crrev.com/a7040a0af3abeca232b0b4bf493d30f070e37d86/third_party/class-dump/LICENSE
[delete] https://crrev.com/a7040a0af3abeca232b0b4bf493d30f070e37d86/third_party/class-dump/OWNERS
[delete] https://crrev.com/a7040a0af3abeca232b0b4bf493d30f070e37d86/third_party/class-dump/README
[delete] https://crrev.com/a7040a0af3abeca232b0b4bf493d30f070e37d86/third_party/class-dump/README.chromium
[delete] https://crrev.com/a7040a0af3abeca232b0b4bf493d30f070e37d86/third_party/class-dump/class-dump.gni
[delete] https://crrev.com/a7040a0af3abeca232b0b4bf493d30f070e37d86/third_party/class-dump/class-dump.py

Status: Fixed (was: Assigned)

Sign in to add a comment