Skip to content

Add ws_data_dir to wheel#57

Merged
danielg55 merged 1 commit intomasterfrom
feature/compile_with_datadir
Mar 7, 2021
Merged

Add ws_data_dir to wheel#57
danielg55 merged 1 commit intomasterfrom
feature/compile_with_datadir

Conversation

@danielg55
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown
Collaborator

@yehonatanz yehonatanz left a comment

Choose a reason for hiding this comment

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

I'd like to see a test verifying your changes if that's not too expensive, and of course I assume you have manually tried that and it works

@yehonatanz
Copy link
Copy Markdown
Collaborator

Also I think you should modify our setup.py and perhaps prefer having a single .ws dir with libs and data inside

@danielg55 danielg55 force-pushed the feature/compile_with_datadir branch 5 times, most recently from 90d6df1 to acdcd09 Compare March 7, 2021 14:20
@danielg55 danielg55 force-pushed the feature/compile_with_datadir branch from 2c9a6e0 to 35fd44d Compare March 7, 2021 14:37
@danielg55
Copy link
Copy Markdown
Collaborator Author

I can't think of a way to test this change, since it should only affect plugins.

@danielg55 danielg55 merged commit 8e9bbf1 into master Mar 7, 2021
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.

2 participants