https://bugzilla.redhat.com/show_bug.cgi?id=1861839

Remi Collet <fed...@famillecollet.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |fed...@famillecollet.com
           Doc Type|---                         |If docs needed, set a value
              Flags|                            |fedora-review?



--- Comment #1 from Remi Collet <fed...@famillecollet.com> ---
> # 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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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@lists.fedoraproject.org

Reply via email to