Skip to content

Commit

Permalink
repo: Update build directives to fix warnings
Browse files Browse the repository at this point in the history
A number of errors are occurring in the build process due to using
old setup.py directives, and the conversion to using the toml
file has left a number of directories reporting that they would be
ignored as the namespace resolution was not being done.

Change to use a narrow namespace resolution to only include the
desired directories with the distribution to fix these errors.
This also required some inti.py file additions to ensure some
directories are included

Change-Id: Icad43d1a9bc85feabe13b846926d1b237cb9df11
  • Loading branch information
cianmcgrath committed Apr 15, 2024
1 parent 4522e04 commit e0b33fa
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 4 deletions.
3 changes: 1 addition & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,7 @@ clean:
rm -r cloudvision.egg-info build dist

dist:
python3 setup.py sdist
python3 setup.py bdist_wheel
python3 -m build

dev-setup:
pip3 install -r requirements-dev.txt
Expand Down
3 changes: 3 additions & 0 deletions cloudvision/Connector/protobuf/__init__.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
# Copyright (c) 2024 Arista Networks, Inc.
# Use of this source code is governed by the Apache License 2.0
# that can be found in the COPYING file.
5 changes: 3 additions & 2 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,9 @@ classifiers = [
"Operating System :: OS Independent",
]

[tool.setuptools]
packages = ["arista", "cloudvision", "fmp"]
[tool.setuptools.packages.find]
include = ["arista*", "cloudvision*", "fmp*"]
namespaces = false

[tool.setuptools.dynamic]
dependencies = { file = ["requirements.txt"] }
Expand Down

0 comments on commit e0b33fa

Please sign in to comment.