[libcxx-commits] [PATCH] D156242: [libc++] Removes a broken test.

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added subscribers: saugustine, libc++ vendors.
ldionne added a comment.

@saugustine Is there any interest in picking this back up?

Otherwise, if nobody steps up to fix the script and its test, I think the only reasonable thing we can do is remove it (that includes the pretty printer scripts). It's really nice to support that, however if it's broken and unmaintained, then it's just dead weight we're accumulating. As part of this cleanup, we could remove `host-has-gdb-with-python` and clean up some bits of our `Dockerfile` that pertain to these `gdb` tests IIRC.

Don't get me wrong, I think it's better to support this cause it's a nice QOI improvement and I think libc++ is the right place to have this code, but if it's unmaintained, there's no reason to keep it around. I would suggest we make a decision about this in two weeks, on September 5th. If nobody steps up to maintain this, let's remove it.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D156242/new/

https://reviews.llvm.org/D156242