<p>Yup, %{_libdir} is wrong on x86_64 and technically like with %{_prefix}, there's no actual guarantee that python shares the same prefix as rpm. So I guess the really right thing to do would be querying all those different library paths out of python itself: we have %python_sitelib and %python_sitearch but nothing for the standard lib, and I'm not sure they're technically required to overlap in the way the do in eg Fedora.</p>
<p>Might also be that's all just being too perfect for anybodys good, and %{_prefix} be close enough for this purpose. Dunno.</p>
<p>The 'pyd' addition should be in a separate commit, and explained what it is. But for that too, the technically most correct solution would be fishing the extension out of python itself:</p>
<pre><code>>>> import distutils.sysconfig
>>> distutils.sysconfig.get_config_vars('SHLIB_EXT')
['".so"']
</code></pre>

<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/346#issuecomment-341652505">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ANb8067ZYgBHBK3RTvUo9RDi3Ums4-MOks5sytnWgaJpZM4QP61H">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ANb8035OE6vDUh0ud9dUolc-joGrmF38ks5sytnWgaJpZM4QP61H.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/346#issuecomment-341652505"></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":"@pmatilai in #346: Yup, %{_libdir} is wrong on x86_64 and technically like with %{_prefix}, there's no actual guarantee that python shares the same prefix as rpm. So I guess the really right thing to do would be querying all those different library paths out of python itself: we have %python_sitelib and %python_sitearch but nothing for the standard lib, and I'm not sure they're technically required to overlap in the way the do in eg Fedora.\r\n\r\nMight also be that's all just being too perfect for anybodys good, and %{_prefix} be close enough for this purpose. Dunno.\r\n\r\nThe 'pyd' addition should be in a separate commit, and explained what it is. But for that too, the technically most correct solution would be fishing the extension out of python itself:\r\n```\r\n\u003e\u003e\u003e import distutils.sysconfig\r\n\u003e\u003e\u003e distutils.sysconfig.get_config_vars('SHLIB_EXT')\r\n['\".so\"']\r\n```"}],"action":{"name":"View Pull Request","url":"https://github.com/rpm-software-management/rpm/pull/346#issuecomment-341652505"}}}</script>