repository urls for non editables end up in the `install_requires`
list causing setup to fail. In this commit, all usable external urls get
inserted into the `dependency_links` list with the corresponding egg fragments
parsed into package names and versions (maintaining dashes if any) and added to
`install_requires` list.
Introduce parsing by using urlparse to extract properly the url
fragment.
Change-Id: If19576869fdf38e1768190c2d9c33f32504f7c54
Closes-Bug: #1373623
- Use constraints for documentation targets
- Indicate support for Python 3.6, 3.7
- Fix indentation
Change-Id: I486b6f645fa71f0fa519464465bf26411ca4faf4
Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
Currently we rely on the wsgi server to choose its own bind address,
which tends to result in it listening on a DNS name. This means we
have a dependency on that name resolving as expected on the node,
which seems to not always be the case.
In the interest of removing that environment dependency, let's have
the wsgi tests explicitly listen on 127.0.0.1, which should always
work and still allows us to verify our part of the WSGI functionality.
This also includes a fix to use constraints for building docs because
that is also blocking ci on this repo. We need both of these changes
for anything to merge without endless rechecks.
Change-Id: I48438eb31ee9dd102f06e30b13c6d892a93aba3b
Closes-Bug: 1830962
A recent commit [1] broke Windows support by using shlex to
unquote paths.
The issue with shlex.split is that it doesn't work with Windows
paths, treating backslashes as escape characters.
We'll just replace backslashes with slashes before using shlex.split,
converting them back afterwards.
Closes-Bug: #1831242
[1] Id2cc32e4e40c1f834b19756e922118d8526358d3
Change-Id: Icb3abca004a35ab9760db8116fedfa96d012d0d0
This fix allows it to use git-tags with a pre-release suffix that
follows the SemVer specification (e.g. 1.2.3-rc1).
Change-Id: Ie2e3c0e4145a105aff0a35c8dc6ec2a0cc1fc04e
Currently pbr fails if the description file contains unicode
characters. To fix this we need to open the description file as
utf-8 explicitly. Since open() in Python 2 doesn't support an
encoding parameter, use io.open() which works on both 2 and 3.
Co-Authored-By: Hervé Beraud<hberaud@redhat.com>
Change-Id: I1bee502ac84b474cc9db5523d2437a8c0a861c00
Closes-Bug: 1704472
Enables py37 testing while also being more friendly with developer
environments where not all python versions may be available.
Runs the docs jobs last as that one is the least likely to be needed.
Tox min version is needed for the new options used inside.
Change-Id: Icc6cb4979b2523a0edfc6375c5c032bb8be76b9f
Story: #2004073
Task: #27440
When subparser argument is not provided by user
argparse return an error message not really useful for user:
'Namespace' object has no attribute 'func'
This is due to the fact that when we launch the pbr in cli mode
the subparser argument is not mandatory (required) and directly
we try to execute a undefined function.
Set the subparser required is more helpful for users due to the
fact that argparse display the helping message with the available
sub-commands that users can use
These changes provides the following output if the argument is not
passed:
usage: pbr [-h] [-v] {sha,info,freeze} ...
main.py: error: too few arguments
Change-Id: I7982f9d40cb0979ddb89d7bc53964167f8e4b269
When running this locally I needed to install a few more distro
packages to provide build dependencies for pip packages. This change
adds them to the list of things installed in tools/integration.sh.
Change-Id: I2ae04b893c7dfddcb6d94b724f53ee08436bf577
Previously, when using data_files with a glob that matched a file with
whitespace in the name, pip would error with a message that the file
does not exist, e.g:
error: can't copy 'ansible/roles/yatesr.timezone/templates/timezone-Arch': doesn't exist or not a regular file
The problem was that ansible/roles/yatesr.timezone/templates/timezone-Arch
was a truncated form of the actual filename:
ansible/roles/yatesr.timezone/templates/timezone-Arch Linux.j2
Note the space in the filename and that it has been split on this space.
This change allows you to use a glob that matches files with whitespace
in the name. It does this by quoting the path.
Change-Id: Id2cc32e4e40c1f834b19756e922118d8526358d3
Fixes-Bug: 1810934
This was added to setuptools in [1] so we should support passing it
through from pbr.
Change-Id: I46493c18756bcb01c53575ab51ec5e5e97670fc3
1: b2ea3c4a20
Closes-Bug: 1760938
This is a mechanically generated change to replace openstack.org
git:// URLs with https:// equivalents.
This is in aid of a planned future move of the git hosting
infrastructure to a self-hosted instance of gitea (https://gitea.io),
which does not support the git wire protocol at this stage.
This update should result in no functional change.
For more information see the thread at
http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003825.html
Change-Id: I1d81f12d728b628f5f6efc0ffb812246c0c2fef8
It appears this no longer exists on bionic, and we shouldn't need it
for pbr tests. I suspect this may have been some copy-pasta from
another project's integration tests.
Change-Id: Ife631f77a92ee0b34c19e77cad782d94d18f2e74
Closes-Bug: 1820855
When keywords are defined as a list in cfg file the generated
output have errors and breaks metadata generation.
Change-Id: Ie8a5f30d6af1e81ecf3ca40bc94bc460cca38179
Closes-Bug: #1811475
We already skip things like --index-url, but --find-links can also
be present and also shouldn't be included in install_requires.
This also fixes some issues with the existing unit test for this
filtering.
Change-Id: Ie8eca8c19e955d52722feaa71d5843ccd74b0da0
Closes-Bug: 1716808
If a subdirectory contained the source prefix in it's name, this was
replaced globally e.g using share/ansible = ansible/*, with the following
directory structure:
ansible/roles/kolla-ansible/test
would result in the files being installed as follows:
share/ansible/roles/kolla-share/test
whereas we expected:
share/ansible/roles/kolla-ansible/test
This patch changes the behavior so that only the first occurance is
replaced.
Change-Id: I0aab845315dab0aaccd5f67725d2ebcf0fd08aef
Fixes-Bug: 1810804
Adding the comment flake8:noqa in a file will skip linting the entire
file. Most of the time, the intent was just to skip individual lines to
handle exception cases.
This gets rid of the "flake8:" prefix where it was used incorrectly and
fixes a few legitimate errors that were being hidden by the entire file
being skipped.
The behavior is change in flake8 to handle this better, which will
result in pep8 job failures if these are not fixes first. See more
information in the 3.6.0 release notes:
http://flake8.pycqa.org/en/latest/release-notes/3.6.0.html#features
Change-Id: I1af4eb8bef9d0d0e3aa3bbbfd66a2fcb8ab336cb
Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
Previous hyperlink was redirected to root of the new documentation
location which did not contain the right information.
New link points directly to the right location.
Change-Id: I3c83553a50036eadec74e7a00f3b312a7e5afa11