Skip to content

Added option to support custom shared libraries with <uses-library> tag#1684

Merged
AndreMiras merged 1 commit intokivy:masterfrom
pax0r:master
Mar 7, 2019
Merged

Added option to support custom shared libraries with <uses-library> tag#1684
AndreMiras merged 1 commit intokivy:masterfrom
pax0r:master

Conversation

@pax0r
Copy link
Contributor

@pax0r pax0r commented Feb 6, 2019

@pax0r pax0r force-pushed the master branch 2 times, most recently from a9bc663 to 9409cf7 Compare February 6, 2019 07:48
Copy link
Member

@AndreMiras AndreMiras left a comment

Choose a reason for hiding this comment

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

Looking good, thanks!

@AndreMiras
Copy link
Member

@inclement you have some ongoing "requested changes" that I think got addressed. Do you mind doing a final check and merging?

@AndreMiras
Copy link
Member

@pax0r since @inclement seems to be afk I think we can merge soon. Have you been using this change for a while without issues? Do you think it's OK for a merge now?

@pax0r
Copy link
Contributor Author

pax0r commented Mar 7, 2019

@AndreMiras Yes, we build application using some 3rd party library and had no issues. After merge I'll also provide patch for buildozer with support for this option.

@AndreMiras AndreMiras dismissed inclement’s stale review March 7, 2019 12:38

comments about using default=[] were addressed, now merging

@AndreMiras AndreMiras merged commit 287d191 into kivy:master Mar 7, 2019
@AndreMiras
Copy link
Member

Merged thanks!
@inclement comments were addressed, but the his review was still in "request changes" and that was blocking the merge, so I had to dismiss them

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