[mc-devel] [PATCH] Prefer $() to backticks in sh script

[Yury V. Zaytsev]

Sorry, clicked the wrong reply button and somehow dropped the list.

> On 2. Nov 2024, at 20:04, Yury V. Zaytsev <yury at shurup.com> wrote:
> 
> 
>> On 2. Nov 2024, at 12:22, Matěj Cepl <mcepl at cepl.eu> wrote:
>> 
>> On Sat Nov 2, 2024 at 12:02 PM CET, Yury V. Zaytsev wrote:
>>> Notably, the default /bin/sh on Solaris is not XPG4-compliant.
>> 
>> Aside from my very little will to support
>> purchase of yet another island by Larry,
>> https://solarisfacts.blogspot.com/p/working-with-shell.html
>> claims that shell on Solaris 10 (the oldest still supported
>> version) supports $().
>> 
>> Is it true?
> 
> As I said, /bin/sh doesn't support it. If you read the blog post you linked more carefully, you'll see that the author is referring to ksh, which does support $().
> 
>> And if you tell me, that you want to keep mc development hostage
>> to people who would like to run master branch mc on the ancient
>> unsupported Solaris, then I would call it … Let’s not call it.
> 
> Again, I have to repeat myself: I see no reason to intentionally break systems if keeping them supported is not too much of a hassle. As one of the mc maintainers, it's up to me to decide if it's too cumbersome for me or not. In this case, I see no problem with continuing to use backticks for scripts that need to run under /bin/sh instead of bash, ksh, etc. for the time being.
> 
> I'm a little concerned about you, though, because it's not clear to me what's causing so much aggression on your part. According to our exchange so far, your eyes hurt, you're worried about supporting Larry, you think mc development is being held hostage by e.g. the Illumos people… That all doesn't sound very healthy. I would suggest chilling out a bit. It's just software, after all, some 0's and 1's on some storage medium.
> 
> Thanks for bringing to my attention that we are not consistent about backticks, though. I fixed it for the scripts I tried a few months ago, but I didn't go through all the extfs helpers.
>