[PATCH] D34617: LNT's MANIFEST.in file is missing required client & server requirement files

Chris Matthews via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 26 07:56:19 PDT 2017


Looks good. Thanks!

> On Jun 26, 2017, at 7:20 AM, Przemyslaw Wirkus via Phabricator <reviews at reviews.llvm.org> wrote:
> 
> PrzemekWirkus created this revision.
> 
> This patch updates LNT's MANIFEST.in file with two missing entries which are required to correctly specify the files needed to distribute LNT as redistributable Python source distribution.
> If we don’t supply an explicit list of files, packaging command puts a minimal default set of files into the source distribution (omitting requirements.client.txt & requirements.server.txt). Both .txt files store pinned versions of dependencies for LNT client and server.
> 
> Missing MANIFEST.in file entries with requirements are:
> 
> - requirements.client.txt
> - requirements.server.txt
> 
> 
> https://reviews.llvm.org/D34617
> 
> Files:
>  MANIFEST.in
> 
> 
> Index: MANIFEST.in
> ===================================================================
> --- MANIFEST.in
> +++ MANIFEST.in
> @@ -1,4 +1,5 @@
> include README.txt CHANGELOG.txt
> +include requirements.client.txt requirements.server.txt
> recursive-include docs *
> recursive-include tests *
> recursive-include examples *
> 
> 
> <D34617.103941.patch>


More information about the llvm-commits mailing list