# HG changeset patch
# User Matt Harbison <matt_harbison@yahoo.com>
# Date 1736638248 18000
# Node ID 022c91426022b8a012b69e9acb5edf89815a7458
# Parent  f5091286b10c12d1cd83374506bf787b03e9a676
pyoxidizer: fix building with PEP 517 support

I'm not sure when this broke, because f6b045910d82 explicitly says it's fixing
something, but a recent attempt to build pyoxidizer on Windows specifically
flagged `--no-use-pep517` as a problem.  Now that we have PEP 517 support for
building, this goes away and pyoxidizer 0.17.0 with rust 1.54.0 works again
(yeah, these should be updated).

Note that `py -3.9 run-tests.py --pyoxidized` fails to get the version info on a
zstd repo, and thus fails build and install hg to run the tests.  A zlib repo
works, and since there's a PR that changes obtaining the version info from
`setuptools-scm`, I'm not bothering to fix this.  With the zlib test run, the
following are broken:

    Failed test-commit-interactive.t: output changed
    Failed test-i18n.t: output changed
    Failed test-log.t:

The diffs all look to be related to not applying the requested i18n locale.  IDR
if this was the case when this was being tested 4 years ago (but I suspect it
was not).

diff -r f5091286b10c -r 022c91426022 contrib/heptapod-ci.yml
--- a/contrib/heptapod-ci.yml	Wed Jan 08 05:07:00 2025 +0100
+++ b/contrib/heptapod-ci.yml	Sat Jan 11 18:30:48 2025 -0500
@@ -530,8 +530,10 @@
 
 windows-pyox:
     extends: .windows-runtests
-    when: manual  # pyoxidizer builds seem broken with --no-use-pep517
+    when: manual
     variables:
+        RUNTEST_ARGS: ""
+        WHEEL_ARG: ""
         FLAVOR: "--pyoxidized"
 
 macos:
diff -r f5091286b10c -r 022c91426022 rust/hgcli/pyoxidizer.bzl
--- a/rust/hgcli/pyoxidizer.bzl	Wed Jan 08 05:07:00 2025 +0100
+++ b/rust/hgcli/pyoxidizer.bzl	Sat Jan 11 18:30:48 2025 -0500
@@ -142,7 +142,7 @@
     )
 
     # Add Mercurial to resources.
-    exe.add_python_resources(exe.pip_install(["--verbose", "--no-use-pep517", ROOT]))
+    exe.add_python_resources(exe.pip_install(["--verbose", ROOT]))
 
     # On Windows, we install extra packages for convenience.
     if IS_WINDOWS: