[Top][All Lists]

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

Re: [groff] 14/15: [font/devpdf]: Revise path component separation.

From: Deri
Subject: Re: [groff] 14/15: [font/devpdf]: Revise path component separation.
Date: Sun, 05 Jun 2022 00:14:38 +0100

On Friday, 3 June 2022 10:04:52 BST G. Branden Robinson wrote:

> commit afa7128d7b939ca5c8cd082f2547f42e4e563674

> Author: G. Branden Robinson <>

> AuthorDate: Fri Jun 3 03:02:52 2022 -0500


>     [font/devpdf]: Revise path component separation.


>     * font/devpdf/ (font/devpdf/util/BuildFoundries): Generate

>       script using the `PATH_SEPARATOR` Automake macro.


>     * font/devpdf/util/ Add `pathsep` scalar to house the

>       build-time path separator.


>       (LocateFile): Use it.


>       (LoadFoundry, CheckFoundry): Stop using spaces as part of the path

>       separation delimiter.  It is not idiomatic.


>     Also add editor aid comment and correct inconsistent indentation.

This may cause a problem. If you do a gs -h you will notice that the search paths use the space-colon to delimit paths. So by changing the separator to just colon some paths include a leading or trailing space. The space-colon is also used in the file for additional paths, so perhaps changes are required here as well.

I'm also not sure if using PATH_SEPARATOR is proper here, did you check whether gs -h uses alternative separators on other operating systems? Also, shouldn't have some fiddling as well?



reply via email to

[Prev in Thread] Current Thread [Next in Thread]