[symfony-devs] Re: [PATCH] Support for defining services with annotations

2010-09-09 Thread Handsome Greg
+1 for XML bundle configs - especially in redistributed code. Until a couple of weeks ago I LOATHED XML and always used YAML in my Symfony2 experiments. As my applications grow, I've come to rely on XML schema errors to alert me when I fat finger a config, which has bitten me in the ass with YAML a

Re: [symfony-devs] Re: [PATCH] Support for defining services with annotations

2010-09-07 Thread Fabien Potencier
On 9/7/10 2:05 PM, Jordi Boggiano wrote: On 07.09.2010 13:57, Fabien Potencier wrote: I have the answer for redistributable bundles: they must use XML as their only way to provide configuration (as explained in the documentation). Annotations must only be used for end-user code. Which means yo

Re: [symfony-devs] Re: [PATCH] Support for defining services with annotations

2010-09-07 Thread Jordi Boggiano
On 07.09.2010 13:57, Fabien Potencier wrote: > I have the answer for redistributable bundles: they must use XML as > their only way to provide configuration (as explained in the > documentation). Annotations must only be used for end-user code. Which means you can't easily take your yml-configured

Re: [symfony-devs] Re: [PATCH] Support for defining services with annotations

2010-09-07 Thread Fabien Potencier
On 9/7/10 1:42 PM, Thomas Rabaix wrote: I just keep thinking about people writing bundles and integrating these bundles. Bundles should be able to work out of the box (with DI or annotation), but all settings should be easily changes without extended too much classes. I have the answer for red

Re: [symfony-devs] Re: [PATCH] Support for defining services with annotations

2010-09-07 Thread Lukas Kahwe Smith
On 07.09.2010, at 13:42, Thomas Rabaix wrote: > I just keep thinking about people writing bundles and integrating these > bundles. Bundles should be able to work out of the box (with DI or > annotation), but all settings should be easily changes without extended too > much classes. do you th

Re: [symfony-devs] Re: [PATCH] Support for defining services with annotations

2010-09-07 Thread Thomas Rabaix
Annotations should be used to set default values and the AnnotationReader should be able to get these settings from an external configuration files and merge them together. I guess I can write my own AnnotationReader, but this should be done by the core team so all symfony projects will use the sam

Re: [symfony-devs] Re: [PATCH] Support for defining services with annotations

2010-09-07 Thread Lukas Kahwe Smith
On 07.09.2010, at 12:59, Navid Mitchell wrote: >> anyway, it was just an example about a general concern i have with >> annotations. for one they need to still enable extending, that being said in >> many cases extending will obviously be limited by whats possible without >> breaking the "is a

[symfony-devs] Re: [PATCH] Support for defining services with annotations

2010-09-07 Thread Navid Mitchell
Thanks everyone for taking the time to give me their feedback >> I'm not sure however if the multiplication of configuration options >> (xml/yaml/php/annotations) isn't gonna bite us in the ass at some point >> int he future.. It's nice and all that things are so flexible but aren't >> we abusing