On Mon, 2011-02-14 at 22:23 +0900, Masatake YAMATO wrote: > Thank you for commenting.
Check out: http://intranet.corp.redhat.com/ic/intranet/IanKent.html If you want to discus this (at least I'll try to remember the details) find me on our internal irc at #fs. > > > On Fri, 2011-02-11 at 11:04 +0800, Ian Kent wrote: > >> On Fri, 2011-02-11 at 05:27 +0900, Masatake YAMATO wrote: > >> > No comment? > >> > > >> > I've got one comment privately: use a suffix instead of prefix "auto." > >> > to filter included files. I'm reflected the comment in my local source > >> > tree. Before posting it to this list, I'd like to get more feedbacks. > >> > >> I seem to have missed this, but looking at it, it doesn't appear to > >> adhere to the semantics of + included maps. How does this detect when a > >> map of the same name is included so that the files source can be skipped > >> and the next nss source used instead? > >> > >> > > >> > Masatake YAMATO > >> > > >> > > `dir' map-type is for including files under a directory into master > >> > > map. > >> > > `file' map-type can be used for including a file with + notation like: > >> > > > >> > > +/etc/auto.mine > > > > Within a file map (only) any map type can be + included by giving just > > the map name and setting nsswitch to the sources you want to look in. If > > a file map of the same name as the included map exists, at the moment > > that means within /etc, then it will be used, if not then the next > > nsswitch source will be consulted. If a map of the same name as the + > > included map is included then the nsswitch files source will be ignored > > and subsequent sources consulted. > > > > I've not understood the relationship between my patch and your comment well > yet. > Could you give me a concrete example? > > I'm not convinced that I understood your comment. But I guess you suggest > inconsistent evaluation of following autom.master: > > $ cd /etc; cat auto.master > +file:/etc/autofs.d/auto.foo > +dir:/etc/autofs.d > > In this case auto.foo will be read as a file type master source twice: > once via +file line of auto.master and once via +dir line of auto.master. > > Is my guessing correct? > (In my patch `dir' type is not added to `nss_source' of > autofs/lib/nss_parse.y.) > > In addition I have a question about autofs (without my patch). > Consider following auto.master with no nsswitch setting: > > $ cd /etc; cat auto.master > +file:/etc/auto.bar > +auto.bar > > In this case my understanding is that /etc/auto.bar will be read twice as a > file type source. Is my understanding right? > > Masatake YAMATO _______________________________________________ autofs mailing list [email protected] http://linux.kernel.org/mailman/listinfo/autofs
