[libcxx-commits] [PATCH] D139270: [libc++] Rename __tuple to __tuple2 to avoid file collision

[Tom Honermann via Phabricator via libcxx-commits]

tahonermann added a comment.

I've hit this issue too and it was a source of frustration. Though the directory name is an internal detail that we should be free to change, I don't like the name `__tuple2` as a permanent replacement. Unfortunately, any name we choose will be inconsistent with the names used for other such detail directories; the only directory currently present there that doesn't map directly to a header name is `__debug_utils`.

The core problem is that the install step does not attempt to delete any existing `__tuple` file before attempting to create the `__tuple` directory and copy files. Can we not just add a step to the install target to force that to happen?


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

https://reviews.llvm.org/D139270