<!DOCTYPE html>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
<div class="moz-cite-prefix">On 19-10-2023 03:13, Dave Mielke wrote:<br>
</div>
<blockquote type="cite"
cite="mid:ZTCCpC9IqJI%2Fjk2k@beta.private.mielke.cc"><span
style="white-space: pre-wrap">Thanks. My Windows build with those two PRs reverted is fine. Without creating the installer, i.e. bdist_wininst -> bdist_egg, how should this line in the generated README be reworded?</span>
<pre class="moz-quote-pre" wrap="">
- Python bindings are provided by: Brlapi-${BRLAPI_RELEASE}.win32.exe</pre>
</blockquote>
<p><br>
</p>
<p>Ugh, it looks like eggs can't be installed the modern way with
pip.</p>
<p><br>
</p>
<p>I'd personally do the following:</p>
<p><br>
</p>
<p>1. In the winsetup, add logic to install wheel from pip, similar
to how Cython is installed:</p>
<p><br>
</p>
<p>installWheel() {<br>
local python<br>
<br>
if findCommand python python py<br>
then<br>
export PYTHONIOENCODING="utf-8"<br>
local directory="${python%/*}"<br>
<br>
findCommand wheel wheel || {<br>
logMessage task "installing Wheel"<br>
"${dryRun}" || executeCommand pip -q install wheel<br>
}<br>
else<br>
logWarning "Python not found"<br>
fi<br>
}<br>
<br>
2. swap bdist_wininst for bdist_wheel</p>
<p>3. In the documentation, i'm not exactly sure about the file
naming:<br>
</p>
<p><br>
</p>
<pre class="moz-quote-pre" wrap="">- Python bindings are provided by: Brlapi-${BRLAPI_RELEASE}.win32.whl
- They can be installed using pip: pip install
Brlapi-${BRLAPI_RELEASE}.win32.whl
</pre>
<p></p>
</body>
</html>