Skip to content

Commit bc9f7eb

Browse files
committed
onnxsim: keep the patch's Upstream-Status tag on one line
ci_scripts/check_patch.py's Upstream-Status regex matches ^Upstream-Status: *(.*)$ without DOTALL, so it only reads the first physical line after the tag. The previous commit wrapped the "To upstream [...]" comment across four lines, which left the bracket unterminated on line one and would have failed the check. Join it back into a single line, matching every other patch in this repo (e.g. patches/py-spy/0.4.2/*.patch).
1 parent 609f1be commit bc9f7eb

1 file changed

Lines changed: 1 addition & 4 deletions

File tree

‎patches/onnxsim/0.7.3/0001-tests-pin-ir_version-in-test_profiling-s-foldable-model.patch‎

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -35,10 +35,7 @@ Match the existing convention used throughout the suite (ir_version=10
3535
regardless of opset) so the fixture stays valid across the wide range
3636
of onnx versions the unpinned runtime dependency can resolve to.
3737

38-
Upstream-Status: To upstream [no push/issue access to onnxsim/onnxsim from
39-
this environment - only riseproject-dev/python-wheels is attached; the fix
40-
is a one-line ir_version pin matching every sibling test file's existing
41-
convention]
38+
Upstream-Status: To upstream [no push/issue access to onnxsim/onnxsim from this environment - only riseproject-dev/python-wheels is attached; the fix is a one-line ir_version pin matching every sibling test file's existing convention]
4239

4340
Signed-off-by: Ludovic Henry <git@ludovic.dev>
4441
---

0 commit comments

Comments
 (0)