Arguably, removing the extra expansion for all of the above expansions 
returning strings might be less surprising (after thought): they are all string 
manipulations without side effects, and so the extra expansion isn't needed, 
and passing along '%%' is likelier more useful (at least with %dirname).

 Not doing the expansion is rather easy to do with the existing subroutines 
(from nearly 20yo memory of the implementation ;-).

Still the issue is an obscure corner case and there are other lurking issues 
with escaping like escaped '/' characters in paths (which are more constrained 
than strings), and with non-uglix M$_DOS path alternatives like 
"C:\HELLO\%%%%\WORLD' (capitals deliberately used).

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/313#issuecomment-326723449
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to