Skip to content

Allow lxml 6.x by raising the upper bound - #1

Draft
tonycoder-hub wants to merge 1 commit into
masterfrom
cursor/allow-lxml-6-5c30
Draft

Allow lxml 6.x by raising the upper bound#1
tonycoder-hub wants to merge 1 commit into
masterfrom
cursor/allow-lxml-6-5c30

Conversation

@tonycoder-hub

@tonycoder-hub tonycoder-hub commented Aug 18, 2026

Copy link
Copy Markdown
Owner

Closes codematrixer#51.

lxml = "^5.3.0" is Poetry for >=5.3.0,<6.0.0, which rejects lxml 6.0.2. Environments that already require lxml 6.x cannot install hmdriver2.

This only changes the constraint to >=5.3.0,<7.0.0 so 5.3+ and the 6.x series are both allowed. No lockfile or requirements file is present.

Verified on the host (no Harmony device):

  • pip install lxml==6.0.2 plus editable hmdriver2 resolves as lxml (>=5.3.0,<7.0.0) → 6.0.2
  • import hmdriver2, hmdriver2.driver, hmdriver2._xpath, hmdriver2._client
  • _XPath._json2xml + xpath queries (//*[@text="showDialog"], //root[1]/Row[1]/Button[1]) and control-character sanitization

Existing tests/ need a connected Harmony device and were not run.

Open in Web Open in Cursor 

Poetry caret ^5.3.0 excluded lxml 6.0+, which blocked installs
that already require 6.0.2. Keep 5.3.0 as the floor and allow
the 6.x series (<7).

Co-authored-by: Tony Jin <407243179@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

请考虑升级lxml依赖版本

2 participants