Fix buildworld when cross-building and MK_INFO == yes

[bapt]

Summary

Fix buildworld when cross-building and MK_INFO == yes

build-tools doesn't have consistent install logic, so to avoid disrupting that
I chose to move compiling texinfo from build-tools to bootstrap-tools. This is
a misnomer and I'd like to resolve that item sometime in the near future if
possible.

This commit adds a new knob, MK_INFO_SUPPORT, and builds texinfo depending on
whether or not that knob is set

Test Plan

% make tinderbox -DMAKE_JUST_WORLDS JFLAG=-j6 TARGETS="mips powerpc" \
               TARGET_ARCHES_mips=mipsel TARGET_ARCHES_powerpc=powerpc

[ngie]

Update the diff post-bapt's commits to remove texinfo from base on head

This diff doesn't make sense on head, but will make sense as a direct
commit to stable/9 and stable/10

[ngie]

Remove orphaned build knob from tools/build/options

[imp]

OBE. You should abandon this, no?

[imp]

Even in -stable, you need to define the _SUPPORT variables correctly....

Makefile.inc1

252

This is bogus. You need to define in the same file that MK_INFO is defined.

267

This isn't needed.

1313

This doesn't appear to be defined in src.opts.mk or bsd.opts.mk. Why not?

[ngie]

It seems like some revisions weren't backported to stable/10 for setting MK_INFO=no / -DNO_INFO, so I'm going to abandon/simplify this revision.

Makefile.inc1

252

Ok

1313

I was trying to keep the diff minimal, but I can add it to bsd.own.mk on stable/10.

[bapt]

May be it should not be named MK_INFO_SUPPORT as it is not a real option but something to workaround the fact that MK_INFO=no is part of NO_FUN

Rename MK_INFO_SUPPORT into an internal _BOOTSTRAP_MAKEINFO or something like this so it does not appear as an option but really looks like an internal variable

@imp what do you think?

[ngie]

In D1410#16, @bapt wrote:

May be it should not be named MK_INFO_SUPPORT as it is not a real option but something to workaround the fact that MK_INFO=no is part of NO_FUN

Rename MK_INFO_SUPPORT into an internal _BOOTSTRAP_MAKEINFO or something like this so it does not appear as an option but really looks like an internal variable

@imp what do you think?

I don't think it's necessary because bootstrap-tools isn't build with MK_INFO=no / -DNO_INFO on stable/10...

[bapt]

Here is what I was talking about

[bapt]

From my testing MK_INFO is always set to know on stable/10

[ngie]

Makefile.inc1

252

This isn't needed. Just put it in BMAKE.

263

Please capitalize _bootstrap_makeinfo for consistency.

1329

Why ?=?

1356

Please put _texinfo before that so it's easier to MFC changes and is in alphabetical order.

[ngie]

Makefile.inc1

252

Please ignore this comment. It's needed...

[bapt]

I'll update the patch

Makefile.inc1

252

This is needed otherwise _bootstrap_makeinfo is not defined while hitting the .if the first time resulting in a make(1) failure

[bapt]

Update according to @ngie comment

[bapt]

@imp approved by irc

[ngie]

LGTM!