<p>disable_fix_foo are poor grammar, and not only 'disable' found to be consitent with was %_disable_source_fetch..</p>
<p>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)?</p>
<p>For pkgconfig, I made it more than evidently clear it being useful (and never dropped from <a href="https://github.com/rpm-software-management/rpm/pull/122" class="issue-link js-issue-link" data-url="https://github.com/rpm-software-management/rpm/issues/122" data-id="200204902" data-error-text="Failed to load issue title" data-permission-text="Issue title is private">#122</a>).</p>
<p>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..</p>
<p>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.</p>
<p>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..</p>
<p>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.</p>
<p>Changing single comments in scripts fully authored by for no technical/practical reasons is what to be considered as poor etiquette..</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/rpm-software-management/rpm/pull/230#issuecomment-305853299">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ANb803OQlECFeyPd5ulh1KYUl-rdIKFVks5sAEEXgaJpZM4NuKjx">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ANb80xrMotiAESuv45kXGBsLAOZj1nPiks5sAEEXgaJpZM4NuKjx.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/rpm-software-management/rpm/pull/230#issuecomment-305853299"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/rpm-software-management/rpm","title":"rpm-software-management/rpm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/rpm-software-management/rpm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@proyvind in #230: disable_fix_foo are poor grammar, and not only 'disable' found to be consitent with was %_disable_source_fetch..\r\n\r\nwhy changing \"wuzz out\"  to \"quit\" in comment? in order to not offend some unlikely person, or just plain political correctness (not really mutually exclusive)?\r\n\r\nFor pkgconfig, I made it more than evidently clear it being useful (and never dropped from #122).\r\n\r\nIt'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..\r\n\r\nThe 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.\r\n\r\nFixing 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..\r\n\r\nI don't see the rationale for creating a new PR rather than just commenting on the existing one, nor much for the changes either.\r\n\r\nChanging single comments in scripts fully authored by for no technical/practical reasons is what to be considered as poor etiquette.."}],"action":{"name":"View Pull Request","url":"https://github.com/rpm-software-management/rpm/pull/230#issuecomment-305853299"}}}</script>