chore: Do not require dexdump to install package

Exodus-Privacy / exodus-core

Core functionality of εxodus

GNU Affero General Public License v3.0

18 stars 17 forks source link

Closed pnu-s closed 7 months ago

pnu-s commented 8 months ago

Remove dexdump presence check during package installation because, if you have exodus-core in your requirements, it prevents to:

install requirements and run tests/lints/etc (workaround: install dexdump even if you don't need it :shrug:)
install requirements to upgrade them for Dependabot for instance (workaround: none :-1:)

U039b commented 7 months ago

Why do we still need and use dexdump when analyzing XAPK?

pnu-s commented 7 months ago

Discussed offline, we want to remove dexdump but for the time being, let's move this presence check.