Skip to content

chore(deps): replace ubi-reader fork by official PyPi version.#504

Merged
qkaiser merged 2 commits intomainfrom
345-bump-ubireader
Feb 8, 2023
Merged

chore(deps): replace ubi-reader fork by official PyPi version.#504
qkaiser merged 2 commits intomainfrom
345-bump-ubireader

Conversation

@qkaiser
Copy link
Copy Markdown
Contributor

@qkaiser qkaiser commented Feb 6, 2023

We cannot use git hosted dependencies if we want to publish unblob to PyPi. We therefore moved from our fork of ubi-reader to the official package of ubi-reader.

Our fork was synced with upstream except for one UI change we did to get rid of superfluous directories created by ubireader_extract_images. Since the maintainer of ubi-reader does not want to include it upstream, we moved that logic into the UBIExtractor class.

See onekey-sec/ubi_reader#67 for more details.

@qkaiser qkaiser added the dependencies Pull requests that update a dependency file label Feb 6, 2023
We cannot use git hosted dependencies if we want to publish unblob to
PyPi. We therefore moved from our fork of ubi-reader to the official
package of ubi-reader.

Our fork was synced with upstream except for one UI change we did to get
rid of superfluous directories created by ubireader_extract_images.
Since the maintainer of ubi-reader does not want to include it upstream,
we moved that logic into the UBIExtractor class.

See onekey-sec/ubi_reader#67 for more details.
@qkaiser qkaiser force-pushed the 345-bump-ubireader branch from b1da7c9 to 21c51b8 Compare February 6, 2023 19:14
@qkaiser qkaiser self-assigned this Feb 6, 2023
@qkaiser qkaiser force-pushed the 345-bump-ubireader branch from 449c251 to 3ead9da Compare February 6, 2023 19:30
Comment thread unblob/handlers/filesystem/ubi.py
@qkaiser qkaiser merged commit 0036aaf into main Feb 8, 2023
@qkaiser qkaiser deleted the 345-bump-ubireader branch February 8, 2023 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants