Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 3, 2026

📝 Pull Request

🔄 Port Update

📋 Port Update Checklist

  • Port installs successfully with overlay(vcpkg install --overlay-ports)
  • No unexpected files in port directory(e.g. patch files that are not used in portfile.cmake)
  • Port follows formatting(vcpkg format-manifest --all)
  • Version baseline has been updated

For Reviewers🔎, check the followings.

Changes Made

  • Updated version in vcpkg.json
  • Updated REF in portfile.cmake (uses v${VERSION} pattern)
  • Updated SHA512 hash in portfile.cmake
  • Updated/removed patches: Removed fix-pr-7390.patch (upstreamed), updated fix-cmakelists.patch line numbers (668→534)
  • Merged with main branch to incorporate latest repository changes

Patch changes:

  • fix-pr-7390.patch: Deleted. Upstream v1.20.0+ includes the fix (removal of explicit keyword from OpSchemaRegisterOnce constructor).
  • fix-cmakelists.patch: Updated from line 668 to line 534 to match CMakeLists.txt structure. Patch ensures .proto and .proto3 files are installed.

New SHA512: 4bbc4c09e4bb3eb6049d653ce49200564e8c5dcf1154a30f894f24e15f1986d1f2fe2f4ca32fe383c559e2a0b20681f33d649376bf63e4345df6972a2c78eac8

Validation

  • vcpkg install --overlay-ports=ports onnx succeeds
  • Regression tested in the other ports that use changed port

List the tested triplets:

  • x64-linux

Build time: ~3.2 minutes
Artifacts: libonnx.a, libonnx_proto.a, CMake configs, headers, proto files

Original prompt

/update-port https://github.com/onnx/onnx/releases/tag/v1.20.0


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 3 commits January 3, 2026 12:30
Co-authored-by: luncliff <14101523+luncliff@users.noreply.github.com>
Co-authored-by: luncliff <14101523+luncliff@users.noreply.github.com>
Co-authored-by: luncliff <14101523+luncliff@users.noreply.github.com>
Copilot AI changed the title [WIP] Update port to ONNX version 1.20.0 [onnx] update to v1.20.0 Jan 3, 2026
Copilot AI requested a review from luncliff January 3, 2026 12:39
@luncliff
Copy link
Owner

luncliff commented Jan 3, 2026

@luncliff
Copy link
Owner

@copilot We will use more later version of the onnx project.

  1. merge with the main branch to follow up other files's changes
  2. Use https://github.com/onnx/onnx/releases/tag/v1.20.1 to update the port

Copilot AI and others added 2 commits January 25, 2026 09:48
Co-authored-by: luncliff <14101523+luncliff@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 25, 2026

@copilot We will use more later version of the onnx project.

  1. merge with the main branch to follow up other files's changes
    ...

Completed both tasks:

  1. ✅ Merged with main branch (bc59ca7)
  2. ✅ Updated port to v1.20.1 (d160507)

Installation tested successfully in 3.2 minutes.

Copilot AI changed the title [onnx] update to v1.20.0 [onnx] update to v1.20.1 Jan 25, 2026
Copilot AI requested a review from luncliff January 25, 2026 10:04
@luncliff luncliff marked this pull request as ready for review January 25, 2026 14:08
@luncliff luncliff merged commit b002d80 into main Jan 25, 2026
6 of 16 checks passed
@luncliff luncliff deleted the copilot/update-port-to-v1-20-0 branch January 25, 2026 14:15
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.

2 participants