Skip to content
This repository has been archived by the owner on Oct 29, 2020. It is now read-only.

Make security-acl an optional dependency #76

Closed
Tobion opened this issue Nov 25, 2015 · 2 comments
Closed

Make security-acl an optional dependency #76

Tobion opened this issue Nov 25, 2015 · 2 comments

Comments

@Tobion
Copy link

Tobion commented Nov 25, 2015

IMO the symfony/security-acl dependency should be moved from require to require-dev (and suggest).

We are trying to remove ACL from the default symfony installation, see symfony/symfony#14718
And symfony/symfony 3.0 does not require security-acl anymore and security-acl is also extracted into standalone repo since 2.8.

But using symfony-standard 3.0 still installs security-acl because the doctrine cache bundle requires it.

@weaverryan
Copy link
Contributor

👍 - it is only "used" by https://github.com/doctrine/DoctrineCacheBundle/blob/master/Acl/Model/AclCache.php, correct? This is holding up DoctrineBundle's compatibility as well doctrine/DoctrineBundle#486 (comment)

@weaverryan
Copy link
Contributor

Pull request #77

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

No branches or pull requests

2 participants