Re: Solaris sed

From: Brad Smith <brad@comstyle.com>
Date: Jun 12, 2025, 03:49 UTC
Message-ID: <f2082cde-7eb9-4927-a01c-e6fb3b355d13@comstyle.com>
In-Reply-To: <87bjqteicd.fsf@gmail.com>

On 2025-06-11 11:42 p.m., Collin Funk wrote:

Show 28 quoted lines

> Hi Brad,
>
> Brad Smith <brad@comstyle.com> writes:
>
>> Building on Solaris I noticed the following two issues with Solaris sed.
>>
>>      GEN version-def.h
>> sed: Missing newline at end of file standard input.
>>
>>      GEN config-list.h
>> sed: illegal option -- E
>> Usage:  sed [-n] script [file...]
>>          sed [-n] [-e script]...[-f script_file]...[file...]
>>
>>
>> https://github.com/git/git/commit/e1b81f54da80267edee2cb8fd0d0f75f03023019
>>
>> The second issue being introduced fairly recently. Not sure what would be
>> appropriate fixes. Just pointing them out if someone has an suggestions for
>> fixes.
> I noticed these as well, but just ignored them since it seems to build
> fine.
>
> The first one seems like just a warning? Probably something to do with
> POSIX defining a "Text File" as "A file that contains characters
> organized into zero or more lines" where a line is "A sequence of zero
> or more non- <newline> characters plus a terminating <newline>
> character."

It looks as if it is just a warning to me. I wasn't worrying about that one as much as I was the second issue.

> The second is more tricky. The '-E' option to use EREs was not added to
> the specification for 'sed' until POSIX.1-2024 [1]. Maybe the script
> could check for the 'gsed' command? All of the (few) Solaris machines I
> use will have many GNU programs installed like that.

I can't comment on that especially as the build bits support pretty old releases and I have no idea how long Sun / Oracle have been shipping GNU bits like this. I do not believe this has always been a thing.

> Collin
>
> [1] https://pubs.opengroup.org/onlinepubs/9799919799/utilities/sed.html
>