I think we can enable support for both NFS and TFTP protocols by default for all architectures and redefine options to LOADER_NO_NFS_SUPPORT and LOADER_NO_TFTP_SUPPORT.
2017-09-23 20:34 GMT+02:00 Warner Losh <i...@bsdimp.com>: > On Sat, Sep 23, 2017 at 9:50 AM, John Baldwin <j...@freebsd.org> wrote: > > > On Saturday, September 23, 2017 12:44:42 PM Mariusz Zaborski wrote: > > > Author: oshogbo > > > Date: Sat Sep 23 12:44:42 2017 > > > New Revision: 323952 > > > URL: https://svnweb.freebsd.org/changeset/base/323952 > > > > > > Log: > > > After the r317886 support for TFTP and NFS can be enable > > simultaneously. > > > > > > The cleanup of this distinction was done in the r318988, but this > > Makefile > > > was omitted. > > > > > > Submitted by: kczekirda@ > > > > > > Modified: > > > head/sys/boot/i386/libi386/Makefile > > > > > > Modified: head/sys/boot/i386/libi386/Makefile > > > ============================================================ > > ================== > > > --- head/sys/boot/i386/libi386/Makefile Sat Sep 23 12:35:46 2017 > > (r323951) > > > +++ head/sys/boot/i386/libi386/Makefile Sat Sep 23 12:44:42 2017 > > (r323952) > > > @@ -12,10 +12,10 @@ SRCS= biosacpi.c bioscd.c biosdisk.c biosmem.c > > biospnp > > > .PATH: ${.CURDIR}/../../zfs > > > SRCS+= devicename_stubs.c > > > > > > -# Enable PXE TFTP or NFS support, not both. > > > .if defined(LOADER_TFTP_SUPPORT) > > > CFLAGS+= -DLOADER_TFTP_SUPPORT > > > -.else > > > +.endif > > > +.if defined(LOADER_NFS_SUPPORT) > > > CFLAGS+= -DLOADER_NFS_SUPPORT > > > .endif > > > > Is LOADER_NFS_SUPPORT defined by default? If not, I think you just > turned > > off NFS > > in the default loader. One option would be to add a new > > 'LOADER_NO_NFS_SUPPORT' > > and include NFS unless that is defined. We could also just always > include > > NFS > > support. Finally, if we want to retain the ability to choose loader > bits, > > these > > should probably change to real src options: WITH/WITHOUT_LOADER_TFTP_ > SUPPORT > > and > > WITH/WITHOUT_LOADER_NFS_SUPPORT and have this Makefile check > > MK_LOADER_TFTP/NFS_SUPPORT. Then we could choose defaults in > src.opts.mk. > > > > I've wanted all the 'ad-hoc' options in the tree to wind up in src.opts.mk > . > These would be a great addition. > > Warner > _______________________________________________ > svn-src-h...@freebsd.org mailing list > https://lists.freebsd.org/mailman/listinfo/svn-src-head > To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org" > _______________________________________________ svn-src-all@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-all To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
