[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