[LNT] r307222 - Load TestSuite descriptions from json files

Kristof Beyls via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 6 00:44:24 PDT 2017


On 6 Jul 2017, at 01:34, Matthias Braun via llvm-commits <llvm-commits at lists.llvm.org<mailto:llvm-commits at lists.llvm.org>> wrote:

Modified: lnt/trunk/requirements.client.txt
URL: http://llvm.org/viewvc/llvm-project/lnt/trunk/requirements.client.txt?rev=307222&r1=307221&r2=307222&view=diff
==============================================================================
--- lnt/trunk/requirements.client.txt (original)
+++ lnt/trunk/requirements.client.txt Wed Jul  5 16:34:37 2017
@@ -17,3 +17,4 @@ WTForms==2.0.2
Flask-WTF==0.12
typing
click==6.7
+pyyaml==3.12

I find it odd that this commit adds server functionality, but the pyyaml dependency is added to requirements.client.txt, not requirements.server.txt.
Should this have been added to requirements.server.txt instead?
Assuming this really is a bug, it's a shame the regression tests can't easily catch this kind of bug.

Thanks,

Kristof
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170706/5f72d059/attachment.html>


More information about the llvm-commits mailing list