Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

composer.json doesn't list dependencies #26

Closed
richard-damon opened this issue Jul 27, 2014 · 3 comments
Closed

composer.json doesn't list dependencies #26

richard-damon opened this issue Jul 27, 2014 · 3 comments

Comments

@richard-damon
Copy link

The module zetacomponets/mail can not be used without having also loaded zetacomponents/base, but does not have a requires section in its composer.json file listing this dependency. This means that a project that lists zetacomponents/mail as a dependency may error out unless it also adds zetacomponen/base as a dependency. This goes against the philosophy of composer where every module is supposed to declare all of its requirements so that composer can make sure they are all met.

@derickr
Copy link
Member

derickr commented Jul 27, 2014

Could you perhaps create a patch?

@derickr
Copy link
Member

derickr commented Jul 27, 2014

It looks that dep is in there though:

    "type": "library",
        "require": {
            "zetacomponents/base": "*"
        },

@richard-damon
Copy link
Author

It looks like it is an item fixed in 1.18-beta as part of adding unit tests. 1.17 was released April 2012, the fix was committed May 2012, and is part of the 1.18-Beta release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants