yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #57735
[Bug 1608980] Re: Remove MANIFEST.in as it is not explicitly needed by PBR
Reviewed: https://review.openstack.org/385968
Committed: https://git.openstack.org/cgit/openstack/zun/commit/?id=9b805918882abcf36145ddd9ef6e69889273c5ec
Submitter: Jenkins
Branch: master
commit 9b805918882abcf36145ddd9ef6e69889273c5ec
Author: Iswarya_Vakati <v.iswarya@xxxxxxxxxxxxxxxxxx>
Date: Thu Oct 13 18:21:37 2016 +0530
Drop MANIFEST.in - it's not needed by pbr
Zun already uses PBR:-
setuptools.setup(
setup_requires=['pbr>=1.8'],
pbr=True)
This patch removes `MANIFEST.in` file as pbr generates a
sensible manifest from git files and some standard files
and it removes the need for an explicit `MANIFEST.in` file.
Change-Id: I30830509234c2e37759b97502bb9fd5a9e7036dd
Closes-Bug:#1608980
** Changed in: zun
Status: In Progress => Fix Released
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1608980
Title:
Remove MANIFEST.in as it is not explicitly needed by PBR
Status in craton:
In Progress
Status in Kosmos:
New
Status in Magnum:
Fix Released
Status in masakari:
In Progress
Status in neutron:
In Progress
Status in Neutron LBaaS Dashboard:
Confirmed
Status in octavia:
In Progress
Status in Solum:
In Progress
Status in Tricircle:
Fix Released
Status in OpenStack DBaaS (Trove):
In Progress
Status in watcher:
In Progress
Status in Zun:
Fix Released
Bug description:
PBR do not explicitly require MANIFEST.in, so it can be removed.
Snippet from: http://docs.openstack.org/infra/manual/developers.html
Manifest
Just like AUTHORS and ChangeLog, why keep a list of files you wish to
include when you can find many of these in git. MANIFEST.in generation
ensures almost all files stored in git, with the exception of
.gitignore, .gitreview and .pyc files, are automatically included in
your distribution. In addition, the generated AUTHORS and ChangeLog
files are also included. In many cases, this removes the need for an
explicit ‘MANIFEST.in’ file
To manage notifications about this bug go to:
https://bugs.launchpad.net/craton/+bug/1608980/+subscriptions