Profile include support
rra at stanford.edu
Mon Aug 23 18:20:42 EDT 2010
Greg Hudson <ghudson at MIT.EDU> writes:
> After looking at the profile code a bit, I think that's about a 3x
> expansion in scope. The least invasive approach would be to create a
> new variant of profile_init() which takes a struct errinfo * argument,
> and then to revise a bunch of internal APIs so that it can be used by
> the bottom-level code which does the actual parsing.
> I'm still pondering whether I think my schedule can accomodate that
> amount of scope creep.
> I think that would be fine, and I hadn't considered it; I had assumed
> globbing support was the only good way to avoid including .rpmsave and ~
> That approach is probably more code than using glob() (though, of
> course, much less code than implementing glob() on any hypothetical
> platforms which don't have it). It does have the virtue of simplifying
> the error cases.
Feel free to look at and/or copy things from server/config.c in the remctl
distribution  for how I handle this for one project. It's all fairly
obvious, but that will show you what enforcement I do on valid characters.
The last released version has some minor deficiencies in error
Russ Allbery (rra at stanford.edu) <http://www.eyrie.org/~eagle/>
More information about the krbdev