Skip to content

macos: record renames - #30

Draft
LalitMaganti wants to merge 1 commit into
macos-record-3from
macos-record-4
Draft

LalitMaganti wants to merge 1 commit into
macos-record-3from
macos-record-4

Conversation

@LalitMaganti

@LalitMaganti LalitMaganti commented Sep 16, 2026 •

Copy link
Copy Markdown
Owner

Summary

Records renames on macOS.

Fourth of seven steps towards #2. Stacked on #29.

Why

Build tools often write an output under a temporary name and then rename it into place. Without renames, the output would only appear under a name that nothing reads.

Changes

  • The kernel reports a rename as two lookups, first the source and then the destination. Both are resolved the same way as the path of an open.

Testing

A unit test checks that a rename is reported with both paths resolved.

@LalitMaganti
LalitMaganti added this pull request to stack #32 September 16, 2026 12:57
@LalitMaganti
LalitMaganti force-pushed the macos-record-4 branch 2 times, most recently from d9001e2 to cd0bb4d Compare September 16, 2026 13:31
@LalitMaganti
LalitMaganti removed this pull request from stack #32 October 18, 2026 02:40
@LalitMaganti
LalitMaganti added this pull request to stack #48 October 18, 2026 02:40
@LalitMaganti
LalitMaganti force-pushed the macos-record-4 branch 2 times, most recently from 867d0bf to ee8e230 Compare October 18, 2026 04:44
@LalitMaganti
LalitMaganti force-pushed the macos-record-4 branch 2 times, most recently from 5048b83 to 64a189f Compare October 18, 2026 06:52
@LalitMaganti LalitMaganti changed the title [DNM] macos: record renames macos: record renames Oct 18, 2026
Build tools often write an output under a temporary name and then rename
it into place. Without renames, the output appears only under a name
that nothing reads. The kernel reports a rename as a lookup of the
source and then one of the destination. Both are resolved the same way
as an open.

This branch has not been deployed

No deployments
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