Re: bin/59595: sh(1) in -current breaks pkgsrc/net/libcmis configure

["Robert Elz via gnats" <gnats-admin%NetBSD.org@localhost>]

The following reply was made to PR pkg/59595; it has been noted by GNATS.

From: Robert Elz <kre%munnari.OZ.AU@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: 
Subject: Re: bin/59595: sh(1) in -current breaks pkgsrc/net/libcmis configure
Date: Fri, 15 Aug 2025 04:01:50 +0700

     Date:        Thu, 14 Aug 2025 08:55:00 +0000 (UTC)
     From:        martin%NetBSD.org@localhost
     Message-ID:  <20250814085500.7FAC31A923E%mollari.NetBSD.org@localhost>
 
   | Downgrading /bin/sh to cvs date D2025.06.30.22.00.00
   | works around the bug.
 
 Anything which breaks because cd "" no longer works, is going
 to be broken all over the place now, or soon, most shells which
 used to turn that into "cd ." (which was very common) either have
 already (FreeBSD in early July, bash in 5.3, ...) or will soon
 remove that hack.   So an upstream fix is needed (and in the
 interim, in pkgsrc, a local patch perhaps, as Kimmo provided for
 this case - though that is probably not the correct way, as
 there's no guarantee that ls "" is going to work either.
 
 Rather, change
 
 	cd "$boost_dir" 2>/dev/null
 
 in that script into
 
 	cd "${boost_dir:-.}" 2>/dev/null
 
 so if boost_dir == "" it will "cd ." instead, which is what cd ''
 previously did.
 
 kre
 
 ps: it fails, as "cd ''" now exits with status != 0, which means none
 of the following '&&' commands in that cmdsub get run, the result from
 the cmdsub will be empty, resulting in the following for loop not executing
 its body at all.