Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add an install target #55

Closed
wants to merge 1 commit into from
Closed

Add an install target #55

wants to merge 1 commit into from

Conversation

lpapp
Copy link

@lpapp lpapp commented Jul 26, 2022

No description provided.

@wjakob wjakob force-pushed the master branch 5 times, most recently from 1c17434 to a2b8b20 Compare October 14, 2022 15:25
@wjakob wjakob force-pushed the master branch 6 times, most recently from 42ce5e7 to 0106656 Compare October 27, 2022 13:05
@wjakob wjakob force-pushed the master branch 2 times, most recently from c41d953 to f935f93 Compare November 9, 2022 20:41
@wjakob wjakob force-pushed the master branch 7 times, most recently from d5b6905 to 9af019b Compare November 22, 2022 08:50
@wjakob wjakob force-pushed the master branch 7 times, most recently from 4280c39 to 50e761a Compare February 2, 2023 11:02
@wjakob wjakob force-pushed the master branch 5 times, most recently from ec3373c to b5ed696 Compare February 13, 2023 15:09
@wjakob wjakob force-pushed the master branch 4 times, most recently from c88cb4e to 32e860a Compare February 23, 2023 22:03
@wjakob
Copy link
Owner

wjakob commented Feb 27, 2023

Hi @lpapp ,

after thinking about it (for a long time, sorry!) I decided to close the PR. The official way to install nanobind is through Pip or Conda. Custom solutions like what you propose here are better handed in a specific project, or via a fork.

Wenzel

@wjakob wjakob closed this Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants