Re: CVS commit: src/usr.bin/gzip

[Martin Husemann <martin%duskware.de@localhost>]

On Sat, Jun 01, 2024 at 11:17:11PM +0900, Izumi Tsutsui wrote:
> > Log Message:
> > Fix typo in previous
> 
> ---
> @@ -1126,7 +1126,7 @@ copymodes(int fd, const struct stat *sbp
> 	if (fchmod(fd, sb.st_mode) < 0)
> 		maybe_warn("couldn't fchmod: %s", file);
>  
> -#if !HAVE_NBTOOL_CONFIG_H
> +#ifdef !HAVE_NBTOOL_CONFIG_H
[..]
> If HAVE_NBTOOL_CONFIG_H is not defined, it's treated as 0 so
> !HAVE_NBTOOL_CONFIG_H still works.

Your diff is backwards and I guess that is the cause for the confusion.

The change was from #ifdef (which does not work with !) to #if, to
unbreak the build.

The !HAVE_NBTOOL_CONFIG_H is still there and everything should work as
you intended.

Martin