A place to cache linked articles (think custom and personal wayback machine)
You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

index.html 50KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806
  1. <!doctype html><!-- This is a valid HTML5 document. -->
  2. <!-- Screen readers, SEO, extensions and so on. -->
  3. <html lang="fr">
  4. <!-- Has to be within the first 1024 bytes, hence before the <title>
  5. See: https://www.w3.org/TR/2012/CR-html5-20121217/document-metadata.html#charset -->
  6. <meta charset="utf-8">
  7. <!-- Why no `X-UA-Compatible` meta: https://stackoverflow.com/a/6771584 -->
  8. <!-- The viewport meta is quite crowded and we are responsible for that.
  9. See: https://codepen.io/tigt/post/meta-viewport-for-2015 -->
  10. <meta name="viewport" content="width=device-width,initial-scale=1">
  11. <!-- Required to make a valid HTML5 document. -->
  12. <title>Mercurial's Journey to and Reflections on Python 3 (archive) — David Larlet</title>
  13. <!-- That good ol' feed, subscribe :). -->
  14. <link rel="alternate" type="application/atom+xml" title="Feed" href="/david/log/">
  15. <!-- Generated from https://realfavicongenerator.net/ such a mess. -->
  16. <link rel="apple-touch-icon" sizes="180x180" href="/static/david/icons2/apple-touch-icon.png">
  17. <link rel="icon" type="image/png" sizes="32x32" href="/static/david/icons2/favicon-32x32.png">
  18. <link rel="icon" type="image/png" sizes="16x16" href="/static/david/icons2/favicon-16x16.png">
  19. <link rel="manifest" href="/static/david/icons2/site.webmanifest">
  20. <link rel="mask-icon" href="/static/david/icons2/safari-pinned-tab.svg" color="#07486c">
  21. <link rel="shortcut icon" href="/static/david/icons2/favicon.ico">
  22. <meta name="msapplication-TileColor" content="#f0f0ea">
  23. <meta name="msapplication-config" content="/static/david/icons2/browserconfig.xml">
  24. <meta name="theme-color" content="#f0f0ea">
  25. <!-- Documented, feel free to shoot an email. -->
  26. <link rel="stylesheet" href="/static/david/css/style_2020-06-19.css">
  27. <!-- See https://www.zachleat.com/web/comprehensive-webfonts/ for the trade-off. -->
  28. <link rel="preload" href="/static/david/css/fonts/triplicate_t4_poly_regular.woff2" as="font" type="font/woff2" media="(prefers-color-scheme: light), (prefers-color-scheme: no-preference)" crossorigin>
  29. <link rel="preload" href="/static/david/css/fonts/triplicate_t4_poly_bold.woff2" as="font" type="font/woff2" media="(prefers-color-scheme: light), (prefers-color-scheme: no-preference)" crossorigin>
  30. <link rel="preload" href="/static/david/css/fonts/triplicate_t4_poly_italic.woff2" as="font" type="font/woff2" media="(prefers-color-scheme: light), (prefers-color-scheme: no-preference)" crossorigin>
  31. <link rel="preload" href="/static/david/css/fonts/triplicate_t3_regular.woff2" as="font" type="font/woff2" media="(prefers-color-scheme: dark)" crossorigin>
  32. <link rel="preload" href="/static/david/css/fonts/triplicate_t3_bold.woff2" as="font" type="font/woff2" media="(prefers-color-scheme: dark)" crossorigin>
  33. <link rel="preload" href="/static/david/css/fonts/triplicate_t3_italic.woff2" as="font" type="font/woff2" media="(prefers-color-scheme: dark)" crossorigin>
  34. <script type="text/javascript">
  35. function toggleTheme(themeName) {
  36. document.documentElement.classList.toggle(
  37. 'forced-dark',
  38. themeName === 'dark'
  39. )
  40. document.documentElement.classList.toggle(
  41. 'forced-light',
  42. themeName === 'light'
  43. )
  44. }
  45. const selectedTheme = localStorage.getItem('theme')
  46. if (selectedTheme !== 'undefined') {
  47. toggleTheme(selectedTheme)
  48. }
  49. </script>
  50. <meta name="robots" content="noindex, nofollow">
  51. <meta content="origin-when-cross-origin" name="referrer">
  52. <!-- Canonical URL for SEO purposes -->
  53. <link rel="canonical" href="https://gregoryszorc.com/blog/2020/01/13/mercurial%27s-journey-to-and-reflections-on-python-3/">
  54. <body class="remarkdown h1-underline h2-underline h3-underline hr-center ul-star pre-tick">
  55. <article>
  56. <h1>Mercurial's Journey to and Reflections on Python 3</h1>
  57. <nav>
  58. <p class="center">
  59. <a href="/david/" title="Aller à l’accueil" tabindex="1">🏠</a>
  60. </p>
  61. </nav>
  62. <hr>
  63. <h2><a href="https://gregoryszorc.com/blog/2020/01/13/mercurial%27s-journey-to-and-reflections-on-python-3/">Source originale du contenu</a></h2>
  64. <p>Mercurial 5.2 was released on November 5, 2019. It is the first version
  65. of Mercurial that supports Python 3. This milestone comes nearly 11 years
  66. after Python 3.0 was first released on December 3, 2008.</p>
  67. <p>Speaking as a maintainer of Mercurial and an avid user of Python, I
  68. feel like the experience of making Mercurial work with Python 3 is
  69. worth sharing because there are a number of lessons to be learned.</p>
  70. <p>This post is logically divided into two sections: a mostly factual recount
  71. of Mercurial's Python 3 porting effort and a more opinionated commentary
  72. of the transition to Python 3 and the Python language ecosystem as a whole.
  73. Those who don't care about the mechanics of porting a large Python project
  74. to Python 3 may want to skip the next section or two.</p>
  75. <h2>Porting Mercurial to Python 3</h2>
  76. <p>Let's start with a brief history lesson of Mercurial's support for
  77. Python 3 as told by its own commit history.</p>
  78. <p>The Mercurial version control tool was first released in April 2005
  79. (the same month that Git was initially released). Version 1.0 came out
  80. in March 2008. The first reference to Python 3 I found in the code base
  81. was in <a href="https://www.mercurial-scm.org/repo/hg/rev/8fee8ff13d37">September 2008</a>.
  82. Then not much happens for a while until
  83. <a href="https://www.mercurial-scm.org/repo/hg/rev/4494fb02d549">June 2010</a>, when
  84. someone authors a bunch of changes to make the Python C extensions
  85. start to recognize Python 3. Then things were again quiet for a while
  86. until <a href="https://www.mercurial-scm.org/repo/hg/rev/56ef99fbd6f2">January 2013</a>,
  87. when a handful of changes landed to remove 2 argument <code>raise</code>. There were
  88. a handful of commits in 2014 but nothing worth calling out.</p>
  89. <p>Mercurial's meaningful journey to Python 3 started in 2015. In code,
  90. the work started in
  91. <a href="https://www.mercurial-scm.org/repo/hg/rev/af6e6a0781d7">April 2015</a>, with
  92. effort to make Mercurial's test harness run with Python 3. Part of
  93. this was a <a href="https://www.mercurial-scm.org/repo/hg/rev/fefc72523491">decision</a>
  94. that Python 3.5 (to be released several months later in September 2015)
  95. would be the minimum Python 3 version that Mercurial would support.</p>
  96. <p>Once the Mercurial Project decided it wanted to port to Python 3 (as opposed
  97. to another language), one of the earliest decisions was how to perform that
  98. port. <strong>Mercurial's code base was too large to attempt a flag day conversion</strong>
  99. where there would be a Python 2 version and a Python 3 version and one day
  100. everyone would switch from Python 2 to 3. <strong>Mercurial needed a way to run the
  101. same code (or as much of the same code) on both Python 2 and 3.</strong> We would
  102. maintain a single code base and users would gradually switch from running with
  103. Python 2 to Python 3.</p>
  104. <p>In <a href="https://www.mercurial-scm.org/repo/hg/rev/e1fb276d4619">May 2015</a>,
  105. Mercurial dropped support for Python 2.4 and 2.5. Dropping support for
  106. these older Python versions was critical, as it was effectively impossible to
  107. write Python code that ran on this wide gamut of versions because of
  108. incompatibilities in syntax and language features. For example, you needed
  109. Python 2.6 to get <code>print()</code> via <code>from __future__ import print_function</code>.
  110. The project's late start at a Python 3 port can be significantly attributed
  111. to Python 2.4 and 2.5 compatibility holding us back.</p>
  112. <p>The main goal with Mercurial's early porting work was just getting the code base
  113. to a point where <code>import mercurial</code> would work. There were a myriad of places
  114. where Mercurial used syntax that was invalid on Python 3 and Python 3
  115. couldn't even parse the source code, let alone compile it to bytecode and
  116. execute it.</p>
  117. <p>This effort began in earnest in
  118. <a href="https://www.mercurial-scm.org/repo/hg/rev/e93036747902">June 2015</a>
  119. with global source code rewrites like using modern octal syntax,
  120. modern exception catching syntax (<code>except Exception as e</code> instead of
  121. <code>except Exception, e</code>), <code>print()</code> instead of <code>print</code>, and a
  122. <a href="https://www.mercurial-scm.org/repo/hg/rev/1a6a117d0b95">modern import convention</a>
  123. along with the use of <code>from __future__ import absolute_import</code>.</p>
  124. <p>In the early days of the port, our first goal was to get all source code
  125. parsing as valid Python 3. The next step was to get all the modules <code>import</code>ing
  126. cleanly. This entailed fixing code that ran at <code>import</code> time to work on
  127. Python 3. Our thinking was that we would need the code base to be <code>import</code>
  128. clean on Python 3 before seriously thinking about run-time behavior. In reality,
  129. we quickly ported a lot of modules to <code>import</code> cleanly and then moved on
  130. to higher-level porting, leaving a long-tail of modules with <code>import</code> failures.</p>
  131. <p>This initial porting effort played out over months. There weren't many
  132. people working on it in the early days: a few people would basically hack on
  133. Python 3 as a form of itch scratching and most of the project's energy was
  134. focused on improving the existing Python 2 based product. You can get a rough
  135. idea of the timeline and participation in the early porting effort through the
  136. <a href="https://www.mercurial-scm.org/repo/hg/log/081a77df7bc6/tests/test-check-py3-compat.t?revcount=960">history of test-check-py3-compat.t</a>.
  137. We see the test being added in <a href="https://www.mercurial-scm.org/repo/hg/rev/40eb385f798f">December 2015</a>,
  138. By June 2016, most of the code base was ported to our modern import convention
  139. and we were ready to move on to more meaningful porting.</p>
  140. <p>One of the biggest early hurdles in our porting effort was how to overcome
  141. the string literals type mismatch between Python 2 and 3. In Python 2, a
  142. <code>''</code> string literal is a sequence of bytes. In Python 3, a <code>''</code> string literal
  143. is a sequence of Unicode code points. These are fundamentally different types.
  144. And in Mercurial's code base, <strong>most of our <em>string</em> types are binary by design:
  145. use of a Unicode based <code>str</code> for representing data is flat out wrong for our use
  146. case</strong>. We knew that Mercurial would need to eventually switch many string
  147. literals from <code>''</code> to <code>b''</code> to preserve type compatibility. But doing so would
  148. be problematic.</p>
  149. <p>In the early days of Mercurial's Python 3 port in 2015, Mercurial's project
  150. maintainer (Matt Mackall) set a ground rule that the Python 3 port shouldn't overly
  151. disrupt others: he wanted the Python 3 port to more or less happen in the background
  152. and not require every developer to be aware of Python 3's low-level behavior in order
  153. to get work done on the existing Python 2 code base. This may seem like a questionable
  154. decision (and I probably disagreed with him to some extent at the time because I was
  155. doing Python 3 porting work and the decision constrained this work). But it was the
  156. correct decision. Matt knew that it would be years before the Python 3 port was either
  157. necessary or resulted in a meaningful return on investment (the value proposition of
  158. Python 3 has always been weak to Mercurial because Python 3 doesn't demonstrate a
  159. compelling advantage over Python 2 for our use case). What Matt was trying to do was
  160. minimize the externalized costs that a Python 3 port would inflict on the project.
  161. He correctly recognized that maintaining the existing product and supporting
  162. existing users was more important than a long-term bet in its infancy.</p>
  163. <p>This ground rule meant that a mass insertion of <code>b''</code> prefixes everywhere
  164. was not desirable, as that would require developers to think about whether
  165. a type was a <code>bytes</code> or <code>str</code>, a distinction they didn't have to worry about
  166. on Python 2 because we practically never used the Unicode-based string type in
  167. Mercurial.</p>
  168. <p>In addition, there were some other practical issues with doing a bulk <code>b''</code>
  169. prefix insertion. One was that the added <code>b</code> characters would cause a lot of lines
  170. to grow beyond our length limits and we'd have to reformat code. That would
  171. require manual intervention and would significantly slow down porting. And
  172. a sub-issue of adding all the <code>b</code> prefixes and reformatting code is that it would
  173. <em>break</em> annotate/blame more than was tolerable. The latter issue was addressed
  174. by teaching Mercurial's annotate/blame feature to <em>skip</em> revisions. The project
  175. now has a convention of annotating commit messages with <code># skip-blame &lt;reason&gt;</code>
  176. so structural only changes can easily be ignored when performing an
  177. annotate/blame.</p>
  178. <p>A stop-gap solution to the <code>b''</code> everywhere issue came in
  179. <a href="https://www.mercurial-scm.org/repo/hg/rev/1c22400db72d">July 2016</a>, when I
  180. introduced a custom Python module importer that rewrote source code as part
  181. of <code>import</code> when running on Python 3. (I have
  182. <a href="/blog/2017/03/13/from-__past__-import-bytes_literals/">previously blogged</a>
  183. about this hack.) What this did was transparently add <code>b''</code> prefixes to all
  184. un-prefixed string literals as well as modify how a few common functions were
  185. called so that we wouldn't need to modify source code so things would run natively
  186. on Python 3. The source transformer allowed us to have the benefits of progressing
  187. in our Python 3 port without having to rewrite tens of thousands of lines of
  188. source code. The solution was hacky. But it enabled us to make significant
  189. progress on the Python 3 port without externalizing a lot of cost onto others.</p>
  190. <p>I thought the source transformer would be relatively short-lived and would be
  191. removed shortly after the project inevitably decided to go all in on Python 3.
  192. To my surprise, others built additional transforms over the years and the source
  193. transformer persisted all the way until
  194. <a href="https://www.mercurial-scm.org/repo/hg/rev/d783f945a701">October 2019</a>, when
  195. I removed it just before the first non-alpha Python 3 compatible version
  196. of Mercurial was released.</p>
  197. <p>A common problem Mercurial faced with making the code base dual Python 2/3 native
  198. was dealing with standard library differences. Most of the problems stemmed
  199. from changes between Python 2.7 and 3.5+. But there are changes within the
  200. versions of Python 3 that we had to wallpaper over as well. In
  201. <a href="https://www.mercurial-scm.org/repo/hg/rev/6041fb8f2da8">April 2016</a>, the
  202. <code>mercurial.pycompat</code> module was introduced to export aliases or wrappers around
  203. standard library functionality to abstract the differences between Python
  204. versions. This file <a href="https://www.mercurial-scm.org/repo/hg/log/66af68d4c751/mercurial/pycompat.py?revcount=240">grew over time</a>
  205. and <a href="https://www.mercurial-scm.org/repo/hg/file/66af68d4c751/mercurial/pycompat.py">eventually became</a>
  206. Mercurial's version of <a href="https://six.readthedocs.io/">six</a>. To be honest, I'm
  207. not sure if we should have used <code>six</code> from the beginning. <code>six</code> probably would
  208. have saved some work. But we had to eventually write a lot of shims for
  209. converting between <code>str</code> and <code>bytes</code> and would have needed to invent a
  210. <code>pycompat</code> layer in some form anyway. So I'm not sure <code>six</code> would have saved
  211. enough effort to justify the baggage of integrating a 3rd party package into
  212. Mercurial. (When Mercurial accepts a 3rd party package, downstream packagers
  213. like Debian get all hot and bothered and end up making questionable patches
  214. to our source code. So we prefer to minimize the surface area for
  215. problems by minimizing dependencies on 3rd party packages.)</p>
  216. <p>Once we had a source transforming module importer and the <code>pycompat</code>
  217. compatibility shim, we started to focus in earnest on making core
  218. functionality actually work on Python 3. We established a convention of
  219. annotating changesets needed for Python 3 with <code>py3</code>, so a
  220. <a href="https://www.mercurial-scm.org/repo/hg/log?rev=desc(py3)&amp;revcount=4000">commit message search</a>
  221. yields a lot of the history. (But it isn't a full history since not every Python 3
  222. oriented change used this convention). We see from that history that after
  223. the source importer landed, a lot of porting effort was spent on things
  224. very early in the <code>hg</code> process lifetime. This included handling environment
  225. variables, loading config files, and argument parsing. We introduced a
  226. <a href="https://www.mercurial-scm.org/repo/hg/log/@/tests/test-check-py3-commands.t">test-check-py3-commands.t</a>
  227. test to track the progress of <code>hg</code> commands working in Python 3. The very early
  228. history of that file shows the various error messages changing, as underlying
  229. early process functionality was slowly ported to work on Python 3. By
  230. <a href="https://www.mercurial-scm.org/repo/hg/rev/2d555d753f0e">December 2016</a>, we
  231. had <code>hg version</code> working on Python 3!</p>
  232. <p>With basic <code>hg</code> command dispatch ported to Python 3 at the end of 2016,
  233. 2017 represented an inflection point in the Python 3 porting effort. With the
  234. early process functionality working, different people could pick up different
  235. commands and code paths and start making code work with Python 3. By
  236. <a href="https://www.mercurial-scm.org/repo/hg/rev/52ee1b5ac277">March 2017</a>, basic
  237. repository opening and <code>hg files</code> worked. Shortly thereafter,
  238. <a href="https://www.mercurial-scm.org/repo/hg/rev/ed23f929af38">hg init started working as well</a>.
  239. And <a href="https://www.mercurial-scm.org/repo/hg/rev/935a1b1117c7">hg status</a> and
  240. <a href="https://www.mercurial-scm.org/repo/hg/rev/aea8ec3f7dd1">hg commit</a> did as well.</p>
  241. <p>Within a few months, enough of Mercurial's functionality was working with Python
  242. 3 that we started to <a href="https://www.mercurial-scm.org/repo/hg/rev/7a877e569ed6">track which tests passed on Python 3</a>.
  243. The <a href="https://www.mercurial-scm.org/repo/hg/log/@/contrib/python3-whitelist?revcount=480">evolution of this file</a>
  244. shows a reasonable history of the porting velocity.</p>
  245. <p>In <a href="https://www.mercurial-scm.org/repo/hg/rev/feb910d2f59b">May 2017</a>, we dropped
  246. support for Python 2.6. This significantly reduced the complexity of supporting
  247. Python 3, as there was tons of functionality in Python 2.7 that made it easier
  248. to target both Python 2 and 3 and now our hands were untied to utilize it.</p>
  249. <p>In <a href="https://www.mercurial-scm.org/repo/hg/rev/bd8875b6473c">November 2017</a>, I
  250. landed a test harness feature to report exceptions seen during test runs. I
  251. later <a href="https://www.mercurial-scm.org/repo/hg/rev/8de90e006c78">refined the output</a>
  252. so the most frequent failures were reported more prominently. This feature
  253. greatly enabled our ability to target the most common exceptions, allowing
  254. us to write patches to fix the most prevalent issues on Python 3 and uncover
  255. previously unknown failures.</p>
  256. <p>By the end of 2017, we had most of the structural pieces in place to complete
  257. the port. Essentially all that was required at that point was time and labor.
  258. We didn't have a formal mechanism in place to target porting efforts. Instead,
  259. people would pick up a component or test that they wanted to hack on and then
  260. make incremental changes towards making that work. All the while, we didn't
  261. have a strict policy on not regressing Python 3 and regressions in Python 3
  262. porting progress were semi-frequent. Although we did tend to correct
  263. regressions quickly. And over time, developers saw a flurry of Python 3
  264. patches and slowly grew awareness of how to accommodate Python 3, and the
  265. number of Python 3 regressions became less frequent.</p>
  266. <p>As useful as the source-transforming module importer was, it incurred some
  267. additional burden for the porting effort. The source transformer effectively
  268. converted all un-prefixed string literals (<code>''</code>) to bytes literals (<code>b''</code>)
  269. to preserve string type behavior with Python 2. But various aspects of Python
  270. 3 didn't like the existence of <code>bytes</code>. Various standard library functionality
  271. now wanted unicode <code>str</code> and didn't accept <code>bytes</code>, even though the Python
  272. 2 implementation used the equivalent of <code>bytes</code>. So our <code>pycompat</code> layer
  273. grew pretty large to accommodate calling into various standard library
  274. functionality. Another side-effect which we didn't initially anticipate
  275. was the <code>**kwargs</code> calling convention. Python allows you to use <code>**</code>
  276. with a dict with string keys to turn those keys into named arguments
  277. in a function call. But Python 3 requires these <code>dict</code> keys to be
  278. <code>str</code> and outright rejects <code>bytes</code> keys, even if the <code>bytes</code> instance
  279. is ASCII safe and has the same underlying byte representation of the
  280. string data as the <code>str</code> instance would. So we had to invent support
  281. functions that would convert <code>dict</code> keys from <code>bytes</code> to <code>str</code> for
  282. use with <code>**kwargs</code> and another to convert a <code>**kwargs</code> dict from
  283. <code>str</code> keys to <code>bytes</code> keys so we could use <code>''</code> syntax to access keys
  284. in our source code! Also on the string type front, we had to sprinkle
  285. the codebase with raw string literals (<code>r''</code>) to force the use of
  286. <code>str</code> irregardless of which Python version you were running on (our
  287. source transformer only changed unprefixed string literals, so existing
  288. <code>r''</code> strings would be preserved as <code>str</code>).</p>
  289. <p>Blind transformation of all string literals to <code>bytes</code> was less than ideal
  290. and it did impose some unwanted side-effects. But, again, most <em>strings</em>
  291. in Mercurial are bytes by design, so we thought it would be easier to
  292. <em>byteify</em> all strings then selectively undo that where native strings
  293. were actually warranted (like keys in most <code>dict</code>s) than to take the
  294. up-front cost to examine every string and make an intelligent determination
  295. as to what type it should be. I go back and forth as to whether this was the
  296. correct call. But when you factor in that the source transforming
  297. module importer unblocked Python 3 porting at a time in the project's
  298. history when there was so much focus on improving the core product and it
  299. did so without externalizing many costs onto the people doing the critical
  300. core product work, I think it was the right call.</p>
  301. <p>By mid 2019, the number of test failures in Python 3 had been whittled
  302. down to a reasonable, less daunting number. It felt like victory was
  303. in grasp and inevitable. But a few significant issues lingered.</p>
  304. <p>One remaining question was around addressing differences between Python
  305. 3 versions. At the time, Python 3.5, 3.6, and 3.7 were released and 3.8
  306. was scheduled for release by the end of the year. We had a surprising
  307. number of issues with differences in Python 3 versions. Many of us
  308. were running Python 3.7, so it had the fewest failures. We had to spend
  309. extra effort to get Python 3.5 and 3.6 working as well as 3.7. Same for
  310. 3.8.</p>
  311. <p>Another task we deferred until the second half of 2019 was standing up
  312. robust CI for Python 3. We had some coverage, but it was minimal. Wanting
  313. a distraction from PyOxidizer for a bit and wanting to overhaul Mercurial's
  314. CI system (which is officially built on Buildbot), I cobbled together a
  315. <em>serverless</em> CI system built on top of AWS DynamoDB and S3 for storage,
  316. Lambda functions and CloudWatch events for all business logic, and EC2 spot
  317. instances for job execution. This CI system executed Python 3.5, 3.6, 3.7,
  318. and 3.8 variants of our test harness on Linux and Python 3.7 on Windows.
  319. This gave developers insight into version-specific failures. More
  320. importantly, it also gave insight into Windows failures, which was
  321. previously not well tested. It was discovered that Python 3 on Windows was
  322. lagging significantly behind POSIX.</p>
  323. <p>By the time of the Mercurial developer meetup in October 2019, nearly
  324. all tests were passing on POSIX platforms and we were confident that
  325. we could declare Python 3 support as at least beta quality for the
  326. Mercurial 5.2 release, planned for early November.</p>
  327. <p>One of our blockers for ripping off the alpha label on Python 3 support
  328. was removing our source-transforming module importer. It had performance
  329. implications and it wasn't something we wanted to ship because it felt
  330. too hacky. A blocker for this was we wanted to automatically format
  331. our source tree with <a href="https://black.readthedocs.io/en/stable/">black</a>
  332. because if we removed the source transformer, we'd have to rewrite
  333. a lot of source code to apply changes the transformer was performing,
  334. which would necessitate wrapping a lot of lines, which would involve a lot
  335. of manual effort. We wanted to <em>blacken</em> our code base first so that
  336. mass rewriting source code wouldn't involve a lot of tedious reformatting
  337. since <code>black</code> would handle that for us automatically. And rewriting the
  338. source tree with <code>black</code> was blocked on a specific feature landing in
  339. <code>black</code>! (We did not agree with <code>black</code>'s behavior of
  340. unwrapping comma-delimited lists of items if they could fit on a single
  341. line. So one of our core contributors wrote a patch to <code>black</code> that
  342. changed its behavior so a trailing <code>,</code> in a list of items will force
  343. items to be formatted on multiple lines. I personally find the multiple line
  344. formatting much easier to read. And the behavior is arguably better for
  345. code review and <em>annotation</em>, which is line based.) Once this feature
  346. landed in <code>black</code>, we reformatted our source tree and started ripping
  347. out the source transformations, starting by inserting <code>b''</code> literals
  348. everywhere. By late October, the source transformer was no more and
  349. we were ready to release beta quality support for Python 3 (at least
  350. on UNIX-like platforms).</p>
  351. <p>Having described a mostly factual overview of Mercurial's port to Python
  352. 3, it is now time to shift gears to the speculative and opinionated
  353. parts of this post. <strong>I want to underscore that the opinions reflected
  354. here are my own and do not reflect the overall Mercurial Project or even
  355. a consensus within it.</strong></p>
  356. <h2>The Future of Python 3 and Mercurial</h2>
  357. <p>Mercurial's port to Python 3 is still ongoing. While we've shipped
  358. Python 3 support and the test harness is clean on Python 3, I view shipping
  359. as only a milestone - arguably <em>the</em> most important one - in a longer
  360. journey. There's still a lot of work to do.</p>
  361. <p>It is now 2020 and Python 2 support is now officially dead from the
  362. perspective of the Python language maintainers. Linux distributions are
  363. starting to rip out Python 2. Packages are dropping Python 2 support in
  364. new versions. The world is moving to Python 3 only. But <strong>Mercurial still
  365. officially supports Python 2</strong>. And it is still yet to be determined how
  366. long we will retain support for Python 2 in the code base. We've only had
  367. one release supporting Python 3. Our users still need to port their
  368. extensions (implemented in Python). Our users still need to start widely
  369. using Mercurial with Python 3. Even our own developers need to switch to
  370. Python 3 (old habits are hard to break).</p>
  371. <p>I anticipate a long tail of random bugs in Mercurial on Python 3. While
  372. the tests may pass, our code coverage is not 100%. And even if it were,
  373. Python is a dynamic language and there are tons of invariants that aren't
  374. caught at compile time and can only be discovered at run time. <strong>These
  375. invariants cannot all be detected by tests, no matter how good your test
  376. coverage is.</strong> This is a <em>feature</em>/<em>limitation</em> of dynamic languages. Our
  377. users will likely be finding a long tail of miscellaneous bugs on Python
  378. 3 for <em>years</em>.</p>
  379. <p>At present, our code base is littered with tons of random hacks to bridge
  380. the gap between Python 2 and 3. Once Python 2 support is dropped, we'll
  381. need to remove these hacks and make the source tree Python 3 native, with
  382. minimal shims to wallpaper over differences in Python 3 versions. <strong>Removing
  383. this Python version bridge code will likely require hundreds of commits and
  384. will be a non-trivial effort.</strong> It's likely to be deemed a low priority (it
  385. is glorified busy work after all), and code for the express purpose of
  386. supporting Python 2 will likely linger for years.</p>
  387. <p>We are also still shoring up our packaging and distribution story on
  388. Python 3. This is easier on some platforms than others. I created
  389. <a href="https://github.com/indygreg/PyOxidizer">PyOxidizer</a> partially because
  390. of the poor experience I had with Python application packaging and
  391. distribution through the Mercurial Project. The Mercurial Project has
  392. already signed off on using PyOxidizer for distributing Mercurial in
  393. the future. So look for an <em>oxidized</em> Mercurial distribution in the
  394. near future! (You could argue PyOxidizer is an epic yak shave to better
  395. support Mercurial. But that's for another post.)</p>
  396. <p>Then there's Windows support. A Python 3 powered Mercurial on Windows
  397. still has a handful of known issues. It may require a few more releases
  398. before we consider Python 3 on Windows to be stable.</p>
  399. <p>Because we're still on a code base that must support Python 2, our
  400. adoption of Python 3 features is very limited. The only Python 3
  401. feature that Mercurial developers seem to almost universally get excited
  402. about is type annotations. We already have some people playing around
  403. with <code>pytype</code> using comment-based annotations and <code>pytype</code> has already
  404. caught a few bugs. We're eager to go all in on type annotations and
  405. uncover lots of dynamic typing bugs and poorly implemented APIs.
  406. Beyond type annotations, I can't name any feature that people are screaming
  407. to adopt and which makes a lot of sense for Mercurial. There's a long
  408. tail of minor features I'm sure will get utilized. But none of the
  409. marquee features that define major language releases seem that interesting
  410. to us. Time will tell.</p>
  411. <h2>Commentary on Python 3</h2>
  412. <p>Having described Mercurial's ongoing journey to Python 3, I now want to
  413. focus more on Python itself. Again, the opinions here are my own and
  414. don't reflect those of the Mercurial Project.</p>
  415. <p><strong>Succinctly, my experience porting Mercurial and other projects to
  416. Python 3 has significantly soured my perceptions of Python. As much as
  417. I have historically loved Python - from the language to the welcoming
  418. community - I am still struggling to understand how Python could manage
  419. to inflict so much hardship on the community by choosing the transition
  420. plan that they did.</strong> I believe Python's choices represent a terrific
  421. example of what not to do when managing a large project or ecosystem.
  422. Maintainers of other largely-deployed systems would benefit from taking
  423. the time to understand and reflect on Python's missteps.</p>
  424. <p>Python 3.0 was released on December 3, 2008. And it took the better part of
  425. a decade for the community to embrace it. <strong>This should be universally
  426. recognized as a failure.</strong> While hindsight is 20/20, many of the issues
  427. with Python 3 were obvious at the time and could have been mitigated had
  428. the language maintainers been more accommodating - and dare I say
  429. empathetic - to its users.</p>
  430. <p>Initially, Python 3 had a rather cavalier attitude towards backwards and
  431. forwards compatibility. In the early years of Python 3, the attitude of
  432. Python's maintainers was <em>Python 3 is a new, better language: you should
  433. target it explicitly</em>. There were some tools and methods to ease the
  434. transition. But nothing super polished, especially in the early years.
  435. Adoption of Python 3 in the overall community was slow. Python developers
  436. in the wild justifiably complained that the value proposition of Python 3
  437. was too weak to justify porting effort. Not helping was that the early
  438. advice for targeting Python 3 was to rewrite the source code to become
  439. Python 3 native. This is in contrast with using the same source to run on both
  440. Python 2 and 3. For library and application maintainers, this potentially
  441. meant maintaining separate versions of your code or forcing end-users to
  442. make a giant leap, which would realistically orphan users on an old version,
  443. fragmenting your user base. Neither of those were great alternatives, so
  444. you can understand why many projects didn't bite.</p>
  445. <p>For many projects of non-trivial size, flag day transitions from Python 2 to
  446. 3 were simply not viable: the pathway to Python 3 was to make code dual
  447. Python 2/3 compatible and gradually switch over the runtime to Python 3.
  448. But initial versions of Python 3 made this effectively impossible! Let me
  449. give a few specific examples.</p>
  450. <p>In Python 2, a string literal <code>''</code> is effectively an array of bytes. In
  451. Python 3, it is a series of Unicode code points - a fundamentally different
  452. type! In Python 2, you could write <code>b''</code> to be explicit that a string literal
  453. was bytes or you could write <code>u''</code> to indicate a Unicode literal, mimicking
  454. Python 3's behavior. In Python 3, you could write <code>b''</code> to create a <code>bytes</code>
  455. instance. But for whatever reason, Python 3 initially removed the <code>u''</code> syntax,
  456. meaning there wasn't as easy way to explicitly denote the type of each
  457. string literal so that it was consistent between Python 2 and 3! Python 3.3
  458. (released September 2012) restored <code>u''</code> support, making it more viable to
  459. write Python source code that worked on both Python 2 and 3. <strong>For nearly 4
  460. years, Python 3 took away the consistent syntax for denoting bytes/Unicode
  461. string literals.</strong></p>
  462. <p>Another feature was <code>%</code> formatting of strings. Python 2 allowed use of the
  463. <code>%</code> formatting operator on both its string types. But Python 3 initially
  464. removed the implementation of <code>%</code> from <code>bytes</code>. Why, I have no clue. It
  465. is perfectly reasonable to splice byte sequences into a buffer via use of
  466. a formatting string. But the Python language maintainers insisted otherwise.
  467. And it wasn't until the community complained about its absence loudly enough
  468. that this feature was
  469. <a href="https://docs.python.org/3/whatsnew/3.5.html#whatsnew-pep-461">restored in Python 3.5</a>,
  470. which was released in September 2015. Fun fact: the lack of this feature was
  471. once considered a blocker for Mercurial moving to Python 3 because
  472. Mercurial uses <code>bytes</code> almost universally, which meant that nearly every use
  473. of <code>%</code> would have to be changed to something else. And to this day, Python
  474. 3's <code>bytes</code> still doesn't have a <code>format()</code> method, so the alternative was
  475. effectively string concatenation, which is a massive step backwards from the
  476. expressiveness of <code>%</code> formatting.</p>
  477. <p><strong>The initial approach of Python 3 mirrors a folly that many developers
  478. and projects make: attempting a rewrite instead of performing incremental
  479. evolution.</strong> For established projects, large scale rewrites often go poorly.
  480. And Python 3 is no exception. Yes, from a code level, CPython (and likely
  481. other Python implementations) were incremental changes over Python 2 using
  482. the same code base. But from a language and standard library level, the
  483. differences in Python 3 were significant enough that I - and even Python's
  484. core maintainers - considered it a new language, and therefore a rewrite.
  485. When your random project attempts a rewrite and fails, the blast radius of that is
  486. often contained to that project. Maybe you don't publish a new release
  487. as soon as you otherwise would. <strong>But when you are powering an ecosystem,
  488. the ripple effects from a failed rewrite percolate throughout that ecosystem
  489. and last for years and have many second order effects. We see this with
  490. Python 3, where poor choices made in the late 2000s are inflicting significant
  491. hardship still in 2020.</strong></p>
  492. <p>From the initial restrained adoption of Python 3, it is obvious that the
  493. Python ecosystem overwhelmingly rejected the initial boil the oceans approach
  494. of Python 3. Python's maintainers eventually got the message and started
  495. restoring features like <code>u''</code> and <code>bytes</code> <code>%</code> formatting back into the
  496. language to placate the community. All the while Python 3 had been accumulating
  497. new features and the cumulative sum of those features was compelling enough
  498. to win over users.</p>
  499. <p>For many projects (including Mercurial), Python 3.4/3.5 was the first viable
  500. porting target for Python 3. Python 3.5 was released in September 2015, almost
  501. 7 years after Python 3.0 was released in December 2008. <strong>Seven. Years.</strong>
  502. An ecosystem that falters for that long is generally not healthy. What may have
  503. saved Python from total collapse here is that Python 2 was still going strong and
  504. people were generally happy with it. I really do think Python dodged a bullet
  505. here, because there was a massive window where the language could have
  506. hemorrhaged a critical amount of its user base and been relegated to an
  507. afterthought. One could draw an analogy to Perl, which lost out to PHP,
  508. Python, and Ruby, and whose fall from grace aligned with a lengthy
  509. transition from Perl 5 to 6.</p>
  510. <p>If you look back at the early history of Python 3, <strong>I think you are forced
  511. to conclude that Python effectively kneecapped itself for 5-7 years
  512. through questionable implementation choices that prevented users from
  513. incurring incremental transitions between the major language versions. 2008
  514. to 2013-2015 should be known as the <em>lost years of Python</em> because so much
  515. opportunity and energy was squandered.</strong> Yes, Python is still healthy today
  516. and Python 3 is (finally) being adopted at scale. But had earlier versions
  517. of Python 3 been more <em>empathetic</em> towards Python 2 users porting to it,
  518. Python and Python 3 in 2020 would be even stronger than it is. The community
  519. was artificially hindered for years. And we won't know until 2023-2025 what
  520. things could have looked like in 2020 had the Python core language team
  521. spent more time paving a smoother road between the major language versions.</p>
  522. <p>To be clear, I do think Python 3 is generally a better language than Python 2.
  523. It has fewer warts, more compelling features, and better performance (except
  524. for startup time, which is still slower than Python 2). I am ecstatic the
  525. community is finally rallying around Python 3! For my Python coding, it has
  526. reached the point where I curse under my breath when I need to support
  527. Python 2 or even older versions of Python 3, like 3.5 or 3.6: I just wish
  528. the world would move on and adopt the future already!</p>
  529. <p>But I would be remiss if I failed to mention some of my gripes with Python
  530. 3 beyond the transition shenanigans.</p>
  531. <p>Perhaps my least favorite <em>feature</em> of Python 3 is its insistence that the
  532. world is Unicode. In Python 2, the default string type was backed by
  533. bytes. In Python 3, the default string type is backed by Unicode code
  534. points. As part of that transition, large parts of the standard library
  535. now operate in the Unicode space instead of the domain of bytes. I understand
  536. why Python does this: they want <em>strings</em> to be Unicode and don't want
  537. users to have to spend that much energy thinking about when to use
  538. <code>str</code> versus <code>bytes</code>. This approach is admirable and somewhat defensible
  539. because it takes a stand on a solution that is arguably <em>good enough</em> for
  540. most users. However, <strong>the approach of assuming the world is Unicode is
  541. flat out wrong and has significant implications for systems level
  542. applications</strong> (like version control tools).</p>
  543. <p>There are a myriad of places in Python's standard library where Python
  544. insists on using the Unicode-backed <code>str</code> type and rejects <code>bytes</code>. For
  545. example, various networking modules refuse to accept <code>bytes</code> for hostnames
  546. or URLs. HTTP libraries won't accept <code>bytes</code> for HTTP header names or values.
  547. Functions that are proxies to POSIX-defined functions won't accept <code>bytes</code>
  548. even though the POSIX function it calls into is using <code>char *</code> and isn't
  549. Unicode aware. Then there's filename handling, where Python assumes the
  550. existence of a global encoding for filenames and uses this encoding to convert
  551. between <code>str</code> and <code>bytes</code>. And it does this despite POSIX filesystem paths
  552. being a bag of bytes where the only rules are that <code>\0</code> terminates the
  553. filename and <code>/</code> is special.</p>
  554. <p>In cases like Python refusing to accept <code>bytes</code> for things like HTTP
  555. header names (which will just be spit out over the wire as bytes), Python's
  556. pendulum has swung too far towards Unicode only. In my opinion, Python needs
  557. to be more accommodating and allow <code>bytes</code> when it makes sense. I hope the
  558. pendulum knocks some sense into people when it swings back towards a more
  559. reasonable solution that better acknowledges the realities of the world we
  560. live in.</p>
  561. <p>For areas like filename handling, the world is more complicated. Python
  562. is effectively an abstraction layer over the operating system APIs exposing
  563. this functionality. And there is often an impedance mismatch between operating
  564. systems. For example, POSIX (Linux) tends to use <code>char *</code> for everything
  565. and doesn't care about encoding and Windows tends to use 16 bit character
  566. types where the encoding is... a can of worms.</p>
  567. <p><strong>The reality here is that it is impossible to abstract over differences
  568. between operating system behavior without compromises that can result in data
  569. loss, outright wrong behavior, or loss of functionality. But Python 3 attempts
  570. to do it anyway, making Python 3 unsuitable (or at least highly undesirable) for
  571. certain systems level applications that rely on it</strong> (like a version control
  572. tool).</p>
  573. <p>In fairness to Python, it isn't the only programming language that gets
  574. this wrong. The only language I've seen <em>properly</em> implement higher-order
  575. abstractions on top of operating system facilities is Rust, whose approach can
  576. be generalized as <em>use Python 3's solution of normalizing to Unicode/UTF-8 by
  577. default</em>, but expose <em>escape hatches</em> which allow access to the raw underlying
  578. types and APIs used by the operating system for the advanced consumers who
  579. require it. For example, Rust's <code>Path</code> type which represents a filesystem path
  580. <a href="https://doc.rust-lang.org/std/path/struct.Path.html#method.as_os_str">allows access</a>
  581. to the raw <a href="https://doc.rust-lang.org/std/ffi/struct.OsStr.html">OsStr</a> value
  582. used by the operating system, not a normalization of it to bytes or Unicode,
  583. which may be lossy. This allows consumers to e.g. create and retrieve
  584. OS-native filesystem paths without data loss. This functionality is critical
  585. in some domains. Python 3's awareness/insistence that the world is
  586. Unicode (which it isn't universally) reduces Python's applicability in these
  587. domains.</p>
  588. <p>Speaking of Rust, at the Mercurial developer meetup in October 2019, we were
  589. discussing the use of Rust in Mercurial and one of the core maintainers blurted
  590. out something along the lines of <em>if Rust were at its current state 5 years ago,
  591. Mercurial would have likely ported from Python 2 to Rust instead of Python 3</em>.
  592. As crazy as it initially sounded, I think I agree with that assessment. With the
  593. benefit of hindsight, having been a key player in the Python 3 porting effort,
  594. seeing all the complications and headaches Python 3 is introducing, and
  595. having learned Rust and witnessed its benefits for performance, control,
  596. and correctness firsthand, porting to Rust would likely have been the correct
  597. move for the project at that point in time. 2020 is not 2014, however, and I'm
  598. not sure if I would opt for a rewrite in Rust today. (Most rewrites are follies
  599. after all.) But I know one thing: I certainly wouldn't implement a new version
  600. control tool in Python 3 and I would probably choose Rust as an implementation
  601. language for most new projects in the systems level space or with an expected
  602. shelf life of 10+ years. (I really should blog about how awesome Rust is.)</p>
  603. <p>Back to the topic of Python itself, <strong>I'm really soured on Python at this
  604. point in time. The effort required to port to Python 3 was staggering. For
  605. Mercurial, Python 3 introduces a ton of problems and doesn't really solve
  606. many. We effectively sludged through mud for several years only to wind
  607. up in a state that feels strictly worse than where we started. I'm sure it will
  608. be strictly better in a few years. But at that point, we're talking about a
  609. 5+ year transition. To call the Python 3 transition disruptive and
  610. distracting for the project would be an understatement. As a project maintainer,
  611. it's natural to ask what we could have accomplished if we weren't forced
  612. to carry out this sideshow.</strong></p>
  613. <p>I can't shake the feeling that a lot of the pain afflicted by the Python 3
  614. transition could have been avoided had Python's language leadership made
  615. a different set of decisions and more highly prioritized the transition
  616. experience. (Like not initially removing features like <code>u''</code> and <code>bytes %</code>
  617. and not introducing gratuitous backwards compatibility breaks, like with
  618. <code>items()/iteritems()</code>. I would have also liked to see a feature like
  619. <code>from __future__</code> - maybe <code>from __past__</code> - that would make it easier for
  620. Python 3 code to target semantics in earlier versions in order to provide
  621. a more turnkey on-ramp onto new versions.) I simultaneously see Python 3
  622. losing its position as a justifiable tool in some domains (like systems
  623. level tooling) due to ongoing design decisions and poor implementation (like
  624. startup overhead problems). (In contrast, I see Rust excelling where Python
  625. is faltering and find Rust code surprisingly expressive to write and maintain
  626. given how low-level it is and therefore feel that Rust is a compelling
  627. alternative to Python in a surprisingly large number of domains.)</p>
  628. <p>Look, I know it is easy for me to armchair quarterback and critique with the
  629. benefit of hindsight/ignorance. I'm sure there is a lot of nuance here. I'm
  630. sure there was disagreement within the Python community over a lot of these
  631. issues. Maintaining a large and successful programming language and community
  632. like Python's is hard and you aren't going to please all the people all the
  633. time. And speaking as a maintainer, I have mad respect for the people leading
  634. such a large community. But niceties aside, everyone knows the Python 3
  635. transition was rough and could have gone better. It should not have taken 11
  636. years to get to where we are today.</p>
  637. <p><strong>I'd like to encourage the Python Project to conduct a thorough postmortem on
  638. the transition to Python 3.</strong> Identify what went well, what could have gone
  639. better, and what should be done next time such a large language change is wanted.
  640. Speaking as a Python user, a maintainer of a Python project, and as someone in
  641. industry who is now skeptical about use of Python at work due to risks of
  642. potentially company crippling high-effort migrations in the future, a postmortem
  643. would help restore my confidence that Python's maintainers learned from the
  644. various missteps on the road to Python 3 and these potentially ecosystem
  645. crippling mistakes won't be made again.</p>
  646. <p>Python had a wildly successful past few decades. And it can continue to
  647. thrive for several more. But the Python 3 migration was painful for all
  648. involved. And as much as we need to move on and leave Python 2 behind us,
  649. there are some important lessons to be learned. I hope the Python community
  650. takes the opportunity to reflect and am confident it will grow stronger by
  651. taking the time to do so.</p>
  652. </article>
  653. <hr>
  654. <footer>
  655. <p>
  656. <a href="/david/" title="Aller à l’accueil">🏠</a> •
  657. <a href="/david/log/" title="Accès au flux RSS">🤖</a> •
  658. <a href="http://larlet.com" title="Go to my English profile" data-instant>🇨🇦</a> •
  659. <a href="mailto:david%40larlet.fr" title="Envoyer un courriel">📮</a> •
  660. <abbr title="Hébergeur : Alwaysdata, 62 rue Tiquetonne 75002 Paris, +33184162340">🧚</abbr>
  661. </p>
  662. <template id="theme-selector">
  663. <form>
  664. <fieldset>
  665. <legend>Thème</legend>
  666. <label>
  667. <input type="radio" value="auto" name="chosen-color-scheme" checked> Auto
  668. </label>
  669. <label>
  670. <input type="radio" value="dark" name="chosen-color-scheme"> Foncé
  671. </label>
  672. <label>
  673. <input type="radio" value="light" name="chosen-color-scheme"> Clair
  674. </label>
  675. </fieldset>
  676. </form>
  677. </template>
  678. </footer>
  679. <script type="text/javascript">
  680. function loadThemeForm(templateName) {
  681. const themeSelectorTemplate = document.querySelector(templateName)
  682. const form = themeSelectorTemplate.content.firstElementChild
  683. themeSelectorTemplate.replaceWith(form)
  684. form.addEventListener('change', (e) => {
  685. const chosenColorScheme = e.target.value
  686. localStorage.setItem('theme', chosenColorScheme)
  687. toggleTheme(chosenColorScheme)
  688. })
  689. const selectedTheme = localStorage.getItem('theme')
  690. if (selectedTheme && selectedTheme !== 'undefined') {
  691. form.querySelector(`[value="${selectedTheme}"]`).checked = true
  692. }
  693. }
  694. const prefersColorSchemeDark = '(prefers-color-scheme: dark)'
  695. window.addEventListener('load', () => {
  696. let hasDarkRules = false
  697. for (const styleSheet of Array.from(document.styleSheets)) {
  698. let mediaRules = []
  699. for (const cssRule of styleSheet.cssRules) {
  700. if (cssRule.type !== CSSRule.MEDIA_RULE) {
  701. continue
  702. }
  703. // WARNING: Safari does not have/supports `conditionText`.
  704. if (cssRule.conditionText) {
  705. if (cssRule.conditionText !== prefersColorSchemeDark) {
  706. continue
  707. }
  708. } else {
  709. if (cssRule.cssText.startsWith(prefersColorSchemeDark)) {
  710. continue
  711. }
  712. }
  713. mediaRules = mediaRules.concat(Array.from(cssRule.cssRules))
  714. }
  715. // WARNING: do not try to insert a Rule to a styleSheet you are
  716. // currently iterating on, otherwise the browser will be stuck
  717. // in a infinite loop…
  718. for (const mediaRule of mediaRules) {
  719. styleSheet.insertRule(mediaRule.cssText)
  720. hasDarkRules = true
  721. }
  722. }
  723. if (hasDarkRules) {
  724. loadThemeForm('#theme-selector')
  725. }
  726. })
  727. </script>
  728. </body>
  729. </html>