@swills p.s. Besides the patch being simple, I figured that not calling sed_checked.sh was in line with what is done in case REINPLACE_CMD has been overridden, as changing REINPLACE_ARGS is also a way to alter REINPLACE_CMD.

In D27954#624840, @swills wrote:

In D27954#624793, @swills wrote:

It seems to me, and maybe I'm wrong, that this particular case could have been avoided by checking the output of make makeplist more carefully, it's not perfect, never has been and never will be. Am I missing something?

I am sorry but this is a no. The ports are broken, and the framework is telling you they are, and all you want to do is stop the framework telling you they are broken. If you don't care, just don't build with DEVELOPER set, and everything will be silent again.

In D27954#624718, @mat wrote:

I don't think this is a good idea. Having REINPLACE_ARGS overridable was probably a bad idea, and the ability to change -i is also something that needs to go away.

The change this was based on was backed out in rS363060 to come up with a better strategy. This renders this change pointless, as I assume the next attempt will include this anyway.

@jhb Thanks

In D25488#562868, @imp wrote:

Either of the proposed text changes sound good to me.

Change wording as proposed by 0mp

Screen shot

In D23801#523182, @jbeich wrote:

Michael, does this work fine for you as well?

In D21473#475370, @emaste wrote:

In D21473#473720, @grembo wrote:

That was actually what I called it at first (showconfig or dumpconfig), but then I realized that commands provided by freebsd-update prefer brevity over clarity (e.g. cron actually fetches updates from cron, upgrade fetches upgrades, but doesn't install them, IDS has no verb at all).

Ah, fair enough. Was going to say I'm fine with leaving at as config then, but since you've already changed it let's go with showconfig.

Rebase to rS352542, rename config' to showconfig', update man page date.

In D21473#473711, @emaste wrote:

I'm happy with the config part so far, but probably use showconfig.
I haven't yet reviewed the updatesready.

Change function name to make more sense.

In D21579#473694, @emaste wrote:

Fine with me, although I'd prefer a different name for config_Components_src. This is not directly used for the config file as other config_-prefixed functions are. Maybe something like filter_components?