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

Dropping Gaufrette requirement #4

Closed
garak opened this issue Jan 30, 2015 · 1 comment
Closed

Dropping Gaufrette requirement #4

garak opened this issue Jan 30, 2015 · 1 comment

Comments

@garak
Copy link
Contributor

garak commented Jan 30, 2015

While I consider Gaufrette a nice library, it's an overkill for most applications.
This bundle requires KnpGaufretteBundle, but it could work without it with some minor tweaks.
If you merge current pending PRs, I can do a new PR about this (it's ready made)

@marlonbaeten
Copy link
Member

Hi garak. Of course your tweaks to drop the Gaufrette dependancy are welcome! While Gaufrette might be overkill for many applications, it makes switching to different a server architecture a breeze (like using a CDN to serve your images). However, I do not know the exact performance overhead Gaufrette introduces for a local filesystem as storage, therefore a PR is welcome :-)

@garak garak mentioned this issue Feb 4, 2015
@garak garak closed this as completed Feb 24, 2015
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