Package Details: edl-git r594.407f1a9-1

Git Clone URL: https://aur.archlinux.org/edl-git.git (read-only, click to copy)
Package Base: edl-git
Description: Inofficial Qualcomm Firehose / Sahara / Streaming / Diag Tools
Upstream URL: https://github.com/bkerler/edl
Keywords: edl firehose flashing qfil qualcomm sahara
Licenses: GPL-3.0
Conflicts: edl
Provides: edl
Submitter: zkm2fw
Maintainer: SelfRef
Last Packager: SelfRef
Votes: 7
Popularity: 0.69
First Submitted: 2022-01-10 10:15 (UTC)
Last Updated: 2025-05-14 20:33 (UTC)

Latest Comments

« First ‹ Previous 1 2 3 4 Next › Last »

SelfRef commented on 2023-09-22 09:09 (UTC)

@nilz3000 That's odd. python-exscript isn't necessary for build process. I cannot reproduce this. Have you tried clean build and using makepkg -s? If you still has this error please post it here.

nilz3000 commented on 2023-09-21 18:45 (UTC) (edited on 2023-09-21 18:49 (UTC) by nilz3000)

Hey @Selfref, the build step fails if python-exscript is not installed. Could you please move it from optdepends to depends. And thanks for keeping this package alive.

SelfRef commented on 2023-09-20 16:28 (UTC) (edited on 2023-09-20 16:28 (UTC) by SelfRef)

Thanks @dreieck. I just updated the PKGBUILD and it now works flawlessly with both makepkg and yay using new PEP 517 install method.

A saw the strip: error: the input file... errors although they look more like warnings. Despite them being printed in output edl-git still contains all loaders listed so this is not a typical issue. Maybe i can resolve this some day but it should not make any difference inside built package.

dreieck commented on 2023-09-20 08:27 (UTC)

Python deprecates direct use of setup.py, consider using python-wheel, python-build, python-installer.

Regards and thanks for maintaining!

dreieck commented on 2023-09-20 08:25 (UTC) (edited on 2023-09-20 08:30 (UTC) by dreieck)

I get some "big fat" warnings during python build:

==> Starting build()...
/usr/lib/python3.11/site-packages/setuptools/config/setupcfg.py:293: _DeprecatedConfig: Deprecated config in `setup.cfg`
!!

        ********************************************************************************
        The license_file parameter is deprecated, use license_files instead.

        By 2023-Oct-30, you need to update your project and remove deprecated calls
        or your builds will no longer be supported.

        See https://setuptools.pypa.io/en/latest/userguide/declarative_config.html for details.
        ********************************************************************************

!!
  parsed = self.parsers.get(option_name, lambda x: x)(value)
[...]
/usr/lib/python3.11/site-packages/setuptools/command/build_py.py:201: _Warning: Package 'Loaders.GM' is absent from the `packages` configuration.
!!

        ********************************************************************************
        ############################
        # Package would be ignored #
        ############################
        Python recognizes 'Loaders.GM' as an importable package[^1],
        but it is absent from setuptools' `packages` configuration.

        This leads to an ambiguous overall configuration. If you want to distribute this
        package, please make sure that 'Loaders.GM' is explicitly added
        to the `packages` configuration field.

        Alternatively, you can also rely on setuptools' discovery methods
        (for example by using `find_namespace_packages(...)`/`find_namespace:`
        instead of `find_packages(...)`/`find:`).

        You can read more about "package discovery" on setuptools documentation page:

        - https://setuptools.pypa.io/en/latest/userguide/package_discovery.html

        If you don't want 'Loaders.GM' to be distributed and are
        already explicitly excluding 'Loaders.GM' via
        `find_namespace_packages(...)/find_namespace` or `find_packages(...)/find`,
        you can try to use `exclude_package_data`, or `include-package-data=False` in
        combination with a more fine grained `package-data` configuration.

        You can read more about "package data files" on setuptools documentation page:

        - https://setuptools.pypa.io/en/latest/userguide/datafiles.html


        [^1]: For Python, any directory (with suitable naming) can be imported,
              even if it does not contain any `.py` files.
              On the other hand, currently there is no concept of package data
              directory, all directories are treated like packages.
        ********************************************************************************

!!
  check.warn(importable)
/usr/lib/python3.11/site-packages/setuptools/command/build_py.py:201: _Warning: Package 'Loaders.HuaQin' is absent from the `packages` configuration.
!!

        ********************************************************************************
        ############################
        # Package would be ignored #
        ############################
        Python recognizes 'Loaders.HuaQin' as an importable package[^1],
        but it is absent from setuptools' `packages` configuration.

        [...]
!!
  check.warn(importable)
[... and a _lot_ of more of this kind. ...]

Regards!

dreieck commented on 2023-09-20 08:22 (UTC) (edited on 2023-09-20 08:30 (UTC) by dreieck)

When makepkg runs strip, I get a lot of errors like
strip: error: the input file '[...]' has no sections:

  -> Stripping unneeded symbols from binaries and libraries...
strip: error: the input file './usr/lib/python3.11/site-packages/Loaders/GM/009510e100310000_098b12634b3b8b17_fhprg.bin' has no sections
strip: error: the input file './usr/lib/python3.11/site-packages/Loaders/HuaQin/000460e115900001_85c752ebecd1b0e6_fhprg_peek.bin' has no sections
[...]
strip: error: the input file './usr/lib/python3.11/site-packages/Loaders/zte/3006000000010000_158d0f5f0552472f_fhprg_peek.bin' has no sections
strip: error: the input file './usr/lib/python3.11/site-packages/Loaders/zte/6000000000010000_0b83c6390385506b_fhprg.bin' has no sections

Regards!

SelfRef commented on 2023-08-02 00:45 (UTC)

@dreieck Thanks for noticing. It looks like source repository have been updated recently with new dependencies added.

In case of python-pycryptodomex I'm not sure if it's needed or it's just a legacy dependency because it doubles with python-pycryptodome (same library, different namespace).

Nevertheless all dependencies are now updated including new optional one for QR codes. There's also a temporary workaround for building because some symlinks are breaking the build. There's a bug for that in the upstream.

dreieck commented on 2023-07-31 22:11 (UTC)

According to namcap, python-pycryptodomex and python-requests are needed as dependencies:

edl-git E: Dependency python-pycryptodomex detected and not included (python modules ['Cryptodome.Cipher.AES', 'Cryptodome.Util.number.bytes_to_long', 'Cryptodome.Util.number.long_to_bytes', 'Cryptodome.Hash.CMAC', 'Cryptodome.Util.Counter'] needed in files ['usr/lib/python3.11/site-packages/edlclient/Tools/Library/cryptutils.py', 'usr/lib/python3.11/site-packages/edlclient/Library/cryptutils.py'])
edl-git E: Dependency python-requests detected and not included (python modules ['requests'] needed in files ['usr/lib/python3.11/site-packages/edlclient/Tools/enableadb', 'usr/bin/boottodwnload', 'usr/lib/python3.11/site-packages/edlclient/Tools/boottodwnload', 'usr/bin/enableadb'])

Maybe they are optional, too, for some specific function?

Regards!

SelfRef commented on 2023-07-22 09:35 (UTC)

@nilz3000 Thanks for noticing. I have packaged missing python-exscript so it's now in the AUR.

However I added it as optional dependency for edl-git because: 1. it's only used for telnet functionality, 2. python-exscript has dependency for python-configparser which right now has broken build script and needs to be updated with this fix.

nilz3000 commented on 2023-07-21 21:37 (UTC)

There is also the newly required Exscript package, which is not available from AUR. I think core functionality is not affected, but additional modules that rely on telnet are.

See this commit and the three following for reference: https://github.com/bkerler/edl/commit/7a1191838507a5d1f051c04777cfbbf1eb5ccc74