Skip to content

docs(okf): add the six missing carried-patch rows, refresh seven stale upstream statuses - #563

Merged
gsdali merged 1 commit into
mainfrom
docs/okf-upstream-pr-only
Jul 31, 2026
Merged

docs(okf): add the six missing carried-patch rows, refresh seven stale upstream statuses#563
gsdali merged 1 commit into
mainfrom
docs/okf-upstream-pr-only

Conversation

@gsdali

@gsdali gsdali commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

The PR-only rule is already in the policy, added on 2026-07-30 with dpasukhi's quote and link. What was missing is everything downstream of it.

What this fixes

okf/references/carried-occt-patches.md stopped at 0012 while Scripts/patches/ has run to 0018. Six patches were absent from the ecosystem-level view of what we carry and why:

0013 ShapeUpgrade_UnifySameDomain null-pcurve dereference (#348)
0014 CDF_Application cached-driver reentrancy (#349)
0015 CDM_Application::myMetaDataLookUpTable (#353)
0016 Resource_Manager::Debug + Storage_Schema current data (#374, redesigned per #518)
0017 ShapeFix_ComposeShell/ShapeUpgrade_WireDivide null context (#484)
0018 GCPnts point counts (#555)

Seven upstream statuses were also stale. Re-checked every referenced item against the GitHub API rather than trusting the previous note, and OCCT#1323, #1331, #1374, #1377, #1380, #1382 and #1318 have all merged since the file was written. The two still listed as open, #1386 and #1388, were re-checked and are accurate. #1374 is merged but not yet in a release, so the 0003 retire-when footnote now says that rather than implying we are waiting on review.

The rule the table now straddles

0018 is the first fix we have filed as a PR with no companion reproducer issue, which is what maintainer dpasukhi asked for on OCCT#1409:

In case if you preparing PR, no needs to create Issue. Issue is recommended to create when you are not working on PR

Rows 0001 to 0017 still show the older repro-issue-then-fix-PR pairs. Those stay as the accurate historical record rather than being retitled after the fact; the intro now says which side of the change each row is on, instead of claiming the whole table predates it.

Why this is separate from #519

#519 is the kernel rebuild and the three patches riding it, targeting refactor/381-pass1b. This is a reference doc with no bearing on that binary, so it goes to main on its own and does not wait on Pass 1b landing. 0018's own filing note stays in #519 where the patch lives.

Docs only, no code paths touched.

Refs #555

🤖 Generated with Claude Code

…s stale

The reference stopped at 0012 while Scripts/patches/ has run to 0018, so the four
thread-safety patches from the #344 to #374 chain, the #484 null-context guard and the
new #555 GCPnts fix were all missing from the ecosystem-level view of what we carry.

Adds 0013 to 0018, and refreshes every upstream status against the GitHub API rather
than from the previous note: OCCT#1323, #1331, #1374, #1377, #1380, #1382 and #1318 have
all merged since this file was written. The two still shown as open, #1386 and #1388,
were re-checked and are accurate.

0018 is our first fix filed as a PR with no companion reproducer issue, which is the
practice maintainer dpasukhi asked for on OCCT#1409:

  "In case if you preparing PR, no needs to create Issue. Issue is recommended to
   create when you are not working on PR"

The policy already records that rule, added on 2026-07-30. This only reconciles the
table's intro with it, since the table now has a row on each side of the change.

Refs #555

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@gsdali
gsdali merged commit 069203d into main Jul 31, 2026
2 checks passed
@gsdali
gsdali deleted the docs/okf-upstream-pr-only branch July 31, 2026 01:24
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.

1 participant