Watching the output of configure scroll before my eyes once again, this came back to me...
On Sun, Nov 6, 2011 at 11:35 PM, Monty Taylor <mord...@inaugust.com> wrote: > On 11/06/2011 01:19 PM, Henrik Ingo wrote: >> On Sun, Nov 6, 2011 at 8:37 PM, Monty Taylor <mord...@inaugust.com> wrote: >>> I disagree that required= should default to yes. I think that the only >>> plugins that should be required are ones that will cause the server to >>> malfunction if they are not built. I do think that we should build as >>> many as we can based on deps. >> >> From a drizzled point of view I agree. The idea with being modular is >> exactly this, you should end up with a minimal build if that's where >> you are. >> >> But from a developer point of view I see this as similar to building >> everything in a "warnings are errors" kind of mode. There's a lot of >> output when you run ./configure, so if your new and shiny plugin is >> just silently skipped because of a missing dependency, it takes some >> time to realize what is going on. >> >> So essentially I'm saying that the default for pandora-plugin should >> be required=yes, but almost all drizzle modules should specify >> required=no. >> >> I think I might be contradicting myself there, so I'll shut up and >> leave the decision to you :-) > > Hrm. You are making a good point here. Let's keep thinking about this. One approach would be to let things stay as they are now, but also add a new configure option named --with-all-default-plugins. Using this option you would end up with an identical build as the default ./configure does today, except that all of the default plugins would be guaranteed to be included. If a dependency check failed, the configure would error out instead of silently excluding some plugin. It would then be easy to document the existence of this option in README and the manual and could be good practice to actually use it most of the time. henrik -- henrik.i...@avoinelama.fi +358-40-8211286 skype: henrik.ingo irc: hingo www.openlife.cc My LinkedIn profile: http://www.linkedin.com/profile/view?id=9522559 _______________________________________________ Mailing list: https://launchpad.net/~drizzle-discuss Post to : drizzle-discuss@lists.launchpad.net Unsubscribe : https://launchpad.net/~drizzle-discuss More help : https://help.launchpad.net/ListHelp
