disable_fix_foo are poor grammar, and not only 'disable' found to be consitent 
with was %_disable_source_fetch..

why changing "wuzz out"  to "quit" in comment? in order to not offend some 
unlikely person, or just plain political correctness (not really mutually 
exclusive)?

For pkgconfig, I made it more than evidently clear it being useful (and never 
dropped from #122).

It's the root cause for majority of all double slashes in paths with the result 
of different hacks added to debugedit without the actual ability to really 
workaround the issues for all scenarious..

The idea of that trimming away extra slashes in pkgconfig files having a high 
risk of causing breakage (despite of it's triviality), not even if it's clear 
that it's desirable is irrational.

Fixing the pkgconfig files and the actual root cause of most double slash 
issues with debugedit is hard to dispute as being desired over hacks and 
breakages with debugedit caused by not fixing the actual cause, which this 
script does..

I don't see the rationale for creating a new PR rather than just commenting on 
the existing one, nor much for the changes either.

Changing single comments in scripts fully authored by for no 
technical/practical reasons is what to be considered as poor etiquette..

-- 
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/pull/230#issuecomment-305853299
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to