https://bugzilla.redhat.com/show_bug.cgi?id=1861839 Remi Collet <fedora@xxxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |fedora@xxxxxxxxxxxxxxxxx Doc Type|--- |If docs needed, set a value Flags| |fedora-review? --- Comment #1 from Remi Collet <fedora@xxxxxxxxxxxxxxxxx> --- > # The library has a "constants.php" file and relies on autoloading it, Indeed, in composer.json "autoload": { "classmap": ["lib"], "files": ["lib/constants.php"] }, Simpler way is to include this file from the generated autoloader, Ex: cat << 'EOF' >>autoload.php require_once "lib/constants.php"; EOF IMHO, the "lib" directory is not needed in the package tree >From phpcompatinfo analysis, and per PHP Guidelines, you need to require: php-simplexml php-date php-pcre -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component _______________________________________________ package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx
