Michael Olbrich wrote: > [...] > > > +config KMOD_ZLIB > > + bool > > + prompt "use system zlib" > > + help > > + Use the system zlib to handle compressed modules > > s/system // > > otherwise it's not clear that this means on vs. off and not system vs. > builtin.
Okay. > [...] > > > +KMOD_CONF_TOOL := autoconf > > +KMOD_CONF_OPT := > > break > > $(CROSS_AUTOCONF_USR) \ > > > + --enable-silent-rules \ > > > > + --disable-dependency-tracking \ > > remove Okay. > [...] > > > + --disable-static \ > > + --enable-shared > > + > > +ifdef PTXCONF_KMOD_TOOLS > > +KMOD_CONF_OPT += --enable-tools > > +else > > +KMOD_CONF_OPT += --disable-tools > > +endif > > + > > +ifdef PTXCONF_KMOD_DEBUG > > +KMOD_CONF_OPT += --enable-debug > > +else > > +KMOD_CONF_OPT += --disable-debug > > +endif > > + > > +ifdef PTXCONF_KMOD_ZLIB > > +KMOD_CONF_OPT += --enable-zlib > > +else > > +KMOD_CONF_OPT += --disable-zlib > > +endif > > + > > +ifdef PTXCONF_KMOD_LOGGING > > +KMOD_CONF_OPT += --enable-logging > > +else > > +KMOD_CONF_OPT += --disable-logging > > +endif > > please use ptx/endis. It's shorter and you can sort all parameters as > listed in "configure --help" that makes updates much easier. Okay. Didn't really noticed this macro yet. > [...] > > > +ifdef PTXCONF_KMOD_TOOLS > > + @for file in kmod-insmod kmod-lsmod kmod-modinfo kmod-modprobe > > kmod-rmmod; \ > > + do \ > > + $(call install_copy, kmod, 0, 0, 0755, -, > > /usr/bin/$$file); \ > > + done > > +endif > > How do these compare to the 'real' tool? While this is nice for testing, > it would be nice if we could only install these. They should replace the current used tools in the long term. Main goal for this package is the library, as it will be used in the future by 'udev'. > I'd use $(foreach ...) but that's no big issue. Another macro I'm not a aware off? Juergen -- Pengutronix e.K. | Juergen Beisert | Linux Solutions for Science and Industry | http://www.pengutronix.de/ | -- ptxdist mailing list [email protected]
