s_copyright: search explicit source directories instead of excluding build directories

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Fixed
    • Priority: Major - P3
    • WT12.0.0, 9.0.0-rc0
    • Affects Version/s: None
    • Component/s: Build
    • None
    • Storage Engines, Storage Engines - Transactions
    • 0.001
    • SE Transactions - 2026-06-05
    • None

      Description:

      The s_copyright script previously used find [a-z]* to search all lowercase top-level directories and then relied on sed filters to exclude generated/build directories (build/, cmake/, venv/). This opt-out approach means new build or third-party directories added in the future could silently be scanned unless a corresponding exclusion is remembered and added.

      Change:

      Replace the broad find [a-z]* glob with an explicit list of known source directories: bench, dist, docs, examples, ext, lang, oss, src, test, tools

      Remove the now-redundant sed exclusions for ^build/, ^cmake/, and ^venv/ since those directories are no longer in the search path.

      Motivation:

      • Makes the intent of the script self-documenting — it is immediately clear which directories are subject to copyright checking.
      • New build artifacts or tool-generated directories won't be accidentally scanned.
      • Reduces the chance of false positives or unnecessary failures in non-standard build setups (e.g., cbuild/, alternate CMake output dirs).

      Testing:

      Run dist/s_copyright from the repo root and verify it reports the same set of copyright violations as before with no new errors introduced by the directory change.

            Assignee:
            Ravi Giri
            Reporter:
            Ravi Giri
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: