Re: [PATCH] build: Use $(PATH_SEPARATOR) as a path separator

[Josh Marshall]

Tiny install base, deprecated, non-standard.  In my unauthoritative opinion, the case to support needs to be made first.  This is to say, the balance between supporting more users and each additional step of complexity and indirection needs to be weighed.

On Thu, Oct 19, 2023, 9:59 AM KO Myung-Hun <komh78@gmail.com> wrote:

Josh Marshall wrote:
> Who still uses OS/2?
>

I do.

> On Wed, Oct 18, 2023, 10:32 AM KO Myung-Hun <komh78@gmail.com> wrote:
>
>> ':' is not a path separator on OS/2.
>>
>> * gettext-tools/examples/po/Makefile.am (USE_BUILT_PROGS): Use
>> $(PATH_SEPARATOR) instead of ':'.
>> ---
>>  gettext-tools/examples/po/Makefile.am | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/gettext-tools/examples/po/Makefile.am
>> b/gettext-tools/examples/po/Makefile.am
>> index bf2027fa4..23a35de41 100644
>> --- a/gettext-tools/examples/po/Makefile.am
>> +++ b/gettext-tools/examples/po/Makefile.am
>> @@ -237,7 +237,7 @@ $(DOMAIN).pot-update: $(POTFILES_DEPS) $(SMALLPOTS)
>> remove-potcdate.sed
>>  # The environment variable 'gettext_datadir' is used by autopoint.
>>  # The environment variable 'GETTEXTDATADIR' is used by xgettext.
>>  USE_BUILT_PROGS = \
>> -  PATH="`pwd`/../../src:`pwd`/../../misc:$$PATH"; \
>> +
>> PATH="`pwd`/../../src$(PATH_SEPARATOR)`pwd`/../../misc$(PATH_SEPARATOR)$$PATH";
>> \
>>    gettext_datadir="`pwd`/../../misc"; export gettext_datadir; \
>>    GETTEXTDATADIR="`cd '$(srcdir)/../..' && pwd`"; export GETTEXTDATADIR;
>>
>> --
>> 2.42.0
>>
>>
>>
>

--
KO Myung-Hun

Korean OS/2 User Community : https://www.os2.kr/