[PATCH] D112202: Fixed plistlib API usage for both python 2.7 and 3.x

Thomas Preud'homme via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 21 00:20:47 PDT 2021


thopre added a comment.

I like the simplicity of it compared to my solution, but I'm a bit bothered by plistformat2 being in fact the same format. But it is simpler and less ugly than what I did. @tnfchris what is your opinion?

@kpdev42 Are you still using LNT in Python 2 mode? If not maybe we should just drop Python 2 support (https://reviews.llvm.org/D69056)


Repository:
  rLNT LNT

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

https://reviews.llvm.org/D112202



More information about the llvm-commits mailing list