Re: CVS commit: src/usr.bin/crunch/crunchgen

[Joerg Sonnenberger <joerg%bec.de@localhost>]

On Thu, Dec 19, 2019 at 01:46:43PM -0000, Christos Zoulas wrote:
> In article <20191218222723.GA17293%bec.de@localhost>,
> Joerg Sonnenberger  <joerg%bec.de@localhost> wrote:
> >On Wed, Dec 18, 2019 at 03:51:21PM -0000, Christos Zoulas wrote:
> >> In article <20191218152113.GA7514%bec.de@localhost>,
> >> Joerg Sonnenberger  <joerg%bec.de@localhost> wrote:
> >> >On Tue, Dec 17, 2019 at 09:16:04PM -0500, Christos Zoulas wrote:
> >> >> Module Name:	src
> >> >> Committed By:	christos
> >> >> Date:		Wed Dec 18 02:16:04 UTC 2019
> >> >> 
> >> >> Modified Files:
> >> >> 	src/usr.bin/crunch/crunchgen: crunchgen.1 crunchgen.c
> >> >> 
> >> >> Log Message:
> >> >> Also disable ssp and fortify by default.
> >> >
> >> >Why?
> >> 
> >> Size reduction on install media. There are flags to turn it on.
> >
> >Not all users of crunchgen are space constrained, so this seems like a
> >bad default to me.
> 
> I think that there are two uses: install media and rescue. I can reinstate
> it for rescue. Is that ok? Note that by default both ssp and fortify are
> off for most things, so my change was mostly a no-op for the default settings.

We seem to default to off for more things, PIE too for example.

Joerg