Skip to content
Snippets Groups Projects

Fix perforce permissions interpretation for rules where there is a wildcard in the depot name

Merged Warren Gifford requested to merge mlw-fix-sub-repo-perms-test-failure into main

Created by: mollylogue

Issues arise with rules like //A/.../C/foo/bar/etc when the specified depot is //A/B/C/. The rule ends up being A/**/C/foo/bar/etc, when in reality it should strip out the part that matches the depot and be something like foo/bar/etc.

This PR aims to detect this particular case and adjust the rules accordingly. There's definitely still some cleanup needed and more tests that should be added.

WIP

Test plan

Existing and new unit tests should pass Locally generate permissions using this new code, and test that we can view/not view the code we'd expect.

Merge request reports

Merged by avatar (Jul 12, 2025 9:57pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading