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

Embedd the warp binaries and extract the correct binary when the library gets loaded #4

Closed
AdmiringWorm opened this issue May 8, 2019 · 4 comments
Assignees
Labels
Feature New non-breaking functionality
Milestone

Comments

@AdmiringWorm
Copy link
Member

This is possible to do by using one of the Fody Weavers, and would allow the addin to not have any external dependencies (Which would be the best as there is currently no installation for Warp, only download and execute(

@AdmiringWorm AdmiringWorm added this to the 0.1.0 milestone May 8, 2019
@AdmiringWorm AdmiringWorm added the Feature New non-breaking functionality label May 8, 2019
@gep13
Copy link
Member

gep13 commented May 8, 2019

I will be curious to see how you implement this!

@AdmiringWorm
Copy link
Member Author

I was testing something of that sort yesterday, and got something.
But want to finish up all the configurations before I commit that code.

I will open it as a PR so you can have a look at it. Although the code for that is simple, I believe it would be best with a second pair of eyes to make sure.

@gep13
Copy link
Member

gep13 commented May 8, 2019

Sure thing! Keep me posted, in case I miss a notification.

@AdmiringWorm AdmiringWorm self-assigned this May 8, 2019
AdmiringWorm added a commit that referenced this issue May 10, 2019
in such a case, we should not extract the
binary file.
AdmiringWorm added a commit that referenced this issue May 10, 2019
in such a case, we should not extract the
binary file.
AdmiringWorm added a commit that referenced this issue May 10, 2019
in such a case, we should not extract the
binary file.
AdmiringWorm added a commit that referenced this issue May 12, 2019
in such a case, we should not extract the
binary file.
@AdmiringWorm
Copy link
Member Author

Implemented in fc658a0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New non-breaking functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants