On 06/20/2018 07:49 PM, Michael Haubenwallner wrote:
> Hi,
>
> please review these patches we have in prefix-overlay,
> where patches 3-5 are for Gentoo Prefix on Cygwin.
Thanks for the reviews, will reorder to account for EAPI 7 paths first.
One minor thing still unclear to me, beyond no _leading_ double slash:
Do we try hard to avoid redundant slashes _in between_ too?
Thoughts behind this question:
Beyond the eventual trailing slash, the {,E,SYS,ESYS,B}ROOT variables
may be empty, so Cygwin really requires the ${%/} modifier for them
to not end up with a network location by accident (in EAPI before 7).
But D,ED are not expected to be empty, and the ${%/} modifier will avoid
an extra slash _in between_ only, not a leading one.
So with D,ED the ${%/} modifier feels needless, reducing readability only.
Just wondering if we have some guidance here,
/haubi/
