aaron.ballman added a comment.

In D126495#3569863 <https://reviews.llvm.org/D126495#3569863>, 
@LegalizeAdulthood wrote:

> In D126495#3569029 <https://reviews.llvm.org/D126495#3569029>, @njames93 
> wrote:
>
>> It would also be nice if there was a redirect that would dynamically 
>> translate the old links to new links.
>
> You can do that with `.htaccess`, but I don't know if that's considered 
> acceptable in clang documentation.

FWIW, I don't have any idea if this is or isn't acceptable.

> Leaving `.rst` pages in place for all the old locations just recreates the 
> clutter instead of getting rid of it.

+1

I don't feel super comfortable reviewing the Python changes, but I spot checked 
the changes and they all seem reasonable to me. So this has my LG, but I'm not 
accepting the review because of the Python bit.


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

https://reviews.llvm.org/D126495

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to