NetBSD-Bugs archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: PR/60198 CVS commit: src/external/mit/ctwm/libexec



> On May 2, 2026, at 12:30 PM, Rob Whitlock via gnats <gnats-admin%NetBSD.org@localhost> wrote:
> 
> The following reply was made to PR xsrc/60198; it has been noted by GNATS.
> 
> From: Rob Whitlock <rwhitlock22%gmail.com@localhost>
> To: gnats-bugs%netbsd.org@localhost
> Cc: xsrc-manager%netbsd.org@localhost,
> gnats-admin%netbsd.org@localhost,
> netbsd-bugs%netbsd.org@localhost
> Subject: Re: PR/60198 CVS commit: src/external/mit/ctwm/libexec
> Date: Sat, 2 May 2026 12:29:10 -0400
> 
>> On Apr 17, 2026, at 7:10 AM, Nia Alarie <nia%netbsd.org@localhost> wrote:
>> =20
>> The following reply was made to PR xsrc/60198; it has been noted by =
> GNATS.
>> =20
>> From: "Nia Alarie" <nia%netbsd.org@localhost>
>> To: gnats-bugs%gnats.NetBSD.org@localhost
>> Cc:=20
>> Subject: PR/60198 CVS commit: src/external/mit/ctwm/libexec
>> Date: Fri, 17 Apr 2026 11:07:12 +0000
>> =20
>> Module Name:	src
>> Committed By:	nia
>> Date:		Fri Apr 17 11:07:12 UTC 2026
>> =20
>> Modified Files:
>> 	src/external/mit/ctwm/libexec: ctwm_app_menu
>> =20
>> Log Message:
>> ctwm_app_menu: Handle whitespace around equals sign in ini files
>> =20
>> from Robert Whitlock
>> =20
>> PR xsrc/60198 ctwm_app_menu doesn't consder whitespace
>> =20
>> =20
>> To generate a diff of this commit:
>> cvs rdiff -u -r1.7 -r1.8 src/external/mit/ctwm/libexec/ctwm_app_menu
>> =20
>> Please note that diffs are not public domain; they are subject to the
>> copyright notices on the relevant files.
> 
> Thanks for the quick commit. This PR may be closed.
> 
> P.S.: I think the whitespace before the v parameter was intentional,
> where it is meant to indicate that v is used as a local variable
> and not a parameter that you pass when the value function is called.
> But that=E2=80=99s not a big deal. See
> 
> https://www.gnu.org/software/gawk/manual/html_node/Variable-Scope.html
> 
> and page 54 of The AWK Programming Language:
> 
> =E2=80=9CWe put several blanks between the arguments and the local
> variables so they can be distinguished."=

Actually, I think this needs to be pulled up to netbsd-11.




Home | Main Index | Thread Index | Old Index