Skip to content

✨ Compression libraries - episode III: add support for libbz2 & liblzma to python3 #2097

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

Merged
merged 3 commits into from
Mar 24, 2020

Conversation

opacam
Copy link
Member

@opacam opacam commented Mar 24, 2020

This is the third part and last of a set of recipes that will add add support for libbz2 & liblzma to python3 recipe, allowing us to build some extra python modules:

  • _bz2.cpython-38.so
  • _lzma.cpython-38.so

Also I put up-to-date the python3 recipe documentation 📝

These libraries are required by some python packages, like pandas, which I plan to create a PR later 😉

Notes:

AndreMiras
AndreMiras previously approved these changes Mar 24, 2020
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.

very nice thanks

.. note:: This recipe can be built only against API 21+. Also, in order to
build certain python modules, we need to add some extra recipes to our
build requirements:
The python 3 recipe can be built with some extra python modules, but do so,
Copy link
Member

Choose a reason for hiding this comment

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

but do so -> to do so

build certain python modules, we need to add some extra recipes to our
build requirements:
The python 3 recipe can be built with some extra python modules, but do so,
we need some libraries. Per default, we ship the python3 recipe with some
Copy link
Member

Choose a reason for hiding this comment

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

Per default -> By default

build requirements:
The python 3 recipe can be built with some extra python modules, but do so,
we need some libraries. Per default, we ship the python3 recipe with some
common libraries, defined at ``depends``.We also support some optional
Copy link
Member

Choose a reason for hiding this comment

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

I would say defined in but I suck at grammar. Also missing a space between the dot and "We"

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm not sure about this but what I'm sure is that your English is a lot better than mine...so let's go for in, thanks!!! 😄

The python 3 recipe can be built with some extra python modules, but do so,
we need some libraries. Per default, we ship the python3 recipe with some
common libraries, defined at ``depends``.We also support some optional
libraries, which are less common that the ones defined at ``depends``, so
Copy link
Member

Choose a reason for hiding this comment

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

defined in

@opacam opacam force-pushed the feature-python-compression-libs branch from 0bbb62b to 1117908 Compare March 24, 2020 20:56
@opacam opacam merged commit c427344 into kivy:develop Mar 24, 2020
zworkb pushed a commit to zworkb/python-for-android that referenced this pull request Apr 15, 2020
…ma to python3 (kivy#2097)

* ✨ Add `bz2` and `lzma` support to `Python3`

* 📝 Update python3 recipe docstrings

* 📝 Fix typos and grammar errors
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