Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • Tor Tor
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 345
    • Issues 345
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 11
    • Merge requests 11
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Artifacts
    • Schedules
  • Packages and registries
    • Packages and registries
    • Container Registry
    • Model experiments
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • The Tor Project
  • Core
  • TorTor
  • Issues
  • #22110

Defining TOR_BUILD_TAG and tor_git_revision violates the version spec

When we removed the git revision in legacy/trac#2988 (moved), we allowed vendors to specify TOR_BUILD_TAG instead. But if we specify both TOR_BUILD_TAG and tor_git_revision, get_version() returns a string like this:

Tor 0.2.9.9 (TOR_BUILD_TAG) (git-tor_git_revision)

This violates the version spec, which only allows one set of brackets for EXTRA_INFO. https://gitweb.torproject.org/torspec.git/tree/version-spec.txt#n22

So instead, we should use:

Tor 0.2.9.9 (TOR_BUILD_TAG,git-tor_git_revision)

(We can't use spaces in the EXTRA_INFO.)

We should also write a unit test that checks that our own version passes the directory authority checks.

This isn't serious, because the only programmatic interface that uses this is GETINFO version.

To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information
Assignee
Assign to
Time tracking