Skip to content

Add torch_python and _C library to bazel build#38707

Closed
malfet wants to merge 1 commit intopytorch:masterfrom
malfet:malfet/bazel-add-torch-python
Closed

Add torch_python and _C library to bazel build#38707
malfet wants to merge 1 commit intopytorch:masterfrom
malfet:malfet/bazel-add-torch-python

Conversation

@malfet
Copy link
Copy Markdown
Contributor

@malfet malfet commented May 19, 2020

Split generated_sources into cpp_generated_sources and python_generated_sources
Add shm and _C library definitions

Test Plan: `bazel build :_C.so; pushd bazel-bin/; python -c 'import _C;print(dir(_C))'; popd

@malfet malfet requested review from kostmo, salexspb and seemethere May 19, 2020 05:12
@dr-ci
Copy link
Copy Markdown

dr-ci Bot commented May 19, 2020

💊 CI failures summary and remediations

As of commit f29afed (more details on the Dr. CI page):


  • 1/1 failures possibly* introduced in this PR
    • 1/1 non-CircleCI failure(s)

ci.pytorch.org: 1 failed


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker.

See how this bot performed.

This comment has been revised 1 time.

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@malfet is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@malfet malfet deleted the malfet/bazel-add-torch-python branch May 20, 2020 06:06
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@malfet merged this pull request in 9e910a9.

laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
Split `generated_sources` into `cpp_generated_sources` and `python_generated_sources`
Add `shm` and `_C` library definitions
Pull Request resolved: pytorch#38707

Test Plan: `bazel build :_C.so; pushd bazel-bin/; python -c 'import _C;print(dir(_C))'; popd

Differential Revision: D21654868

Pulled By: malfet

fbshipit-source-id: dd5f78c38fe58e5ab4cccd3eee42706f44af7989
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants