Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,8 @@ readme = "README-wheel.md"
authors = [
{name="PyTorch Team", email="[email protected]"},
]
license = "BSD-3-Clause"
license-files = ["LICENSE"]
license = {text = "BSD-3-Clause"}

keywords = ["pytorch", "machine learning"]
# PyPI package information.
classifiers = [
Expand Down Expand Up @@ -97,6 +97,9 @@ flatc = "executorch.data.bin:flatc"
# TODO(dbort): Could use py_modules to restrict the set of modules we
# package, and package_data to restrict the set up non-python files we
# include. See also setuptools/discovery.py for custom finders.
[tool.setuptools]
license-files = ["LICENSE"]

[tool.setuptools.package-dir]
# Tell setuptools to follow the symlink: src/executorch/* -> * for all first level
# modules such as src/executorch/exir -> exir. This helps us to semi-compliant with
Expand Down
3 changes: 1 addition & 2 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,6 @@
import site
import subprocess
import sys

from distutils import log # type: ignore[import-not-found]
from distutils.sysconfig import get_python_lib # type: ignore[import-not-found]
from pathlib import Path
Expand Down Expand Up @@ -552,7 +551,7 @@ def run(self):
# package subdirectory.
if self.editable_mode:
# In editable mode, the package directory is the original source directory
dst_root = self.get_package_dir(".")
dst_root = self.get_package_dir("executorch")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Curious, is this the symlinked folder or the actual source folder?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the one under src/, so src/executorch

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like tests are failing to build the package with the change.

Copy link
Contributor Author

@larryliu0820 larryliu0820 Nov 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just rebased, let's see

else:
dst_root = os.path.join(self.build_lib, "executorch")
# Create the version file.
Expand Down
Loading