[PATCH 1/2] aiaiai: move most of the aiaiai-email-test-patchset options to the file
dedekind1 at gmail.com
Thu Mar 6 08:51:08 PST 2014
On Thu, 2014-03-06 at 16:27 +0000, Keller, Jacob E wrote:
> Much better! I can work on this. I hope to resolve why my local mailer
> was not sending out the patches.
> I wasn't really sure how to do this, but I like this method much
> as it removes the necessity for the cfg_get_value function, which I
> not like!
> Quick question, how do you think these changes relate to adding
> for other programs, and will it work for that..?
I do not really understand what you mean, sorry.
> We have some global options which I would like to add that I wasn't
> how to implement (number of validators, for example).
Do you mean the "-J" option of aiaiai-email-dispatcher ? I'd move
everything to the cfg file. At least for the e-mail stuff. So I'd kill
that -J too and had it in the config file instead.
> Also, what about this may have to change once we use the cfg file for
> all programs and not just the email settings?
Off the top of my head...
I guess after doing this for e-mail this can be "pushed down" to
aiaiai-testp-patchset too. Then may be down to helpers.
Although probably the base scripts should supported both, so that I
could either use the cfg file or options (but not mixed). Not sure how
difficult is that, though. The thing is that aiaiai-test-patchset is
used as a command-line tool too.
More information about the aiaiai