On Tue, 17 May 2011 12:01:12 -0500 Shirish Pargaonkar <shirishpargaonkar@xxxxxxxxx> wrote: [...] > >> +if test $enable_cifsidmap != "no"; then > >> + Â Â AC_CHECK_HEADERS([keyutils.h], , [ > >> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â if test "$enable_cifsidmap" = "yes"; then > >> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â AC_MSG_ERROR([keyutils.h not found, consider installing keyutils-libs-devel.]) > >> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â else > >> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â AC_MSG_WARN([keyutils.h not found, consider installing keyutils-libs-devel. Disabling cifs.idmap.]) > >> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â enable_cifsidmap="no" > >> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â fi > >> + Â Â Â Â Â Â Â Â Â Â ]) > >> +fi > >> + > > > > We're already doing a AC_CHECK_HEADERS for keyutils.h -- I don't think > > we want to do it again. Just use the same check for both cifs.upcall > > and cifs.idmap. > > Could someone build just cifs.idmap and not cifs.upcall? > Sure, I don't see why not. -- Jeff Layton <jlayton@xxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
