Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Stale metadata w/o full rebuild #400

Closed
1 of 2 tasks
Rainyan opened this issue Jun 25, 2024 · 1 comment
Closed
1 of 2 tasks

[BUG] Stale metadata w/o full rebuild #400

Rainyan opened this issue Jun 25, 2024 · 1 comment
Labels
bug Something isn't working tooling Tooling related works (EX: OS fixes, libraries)

Comments

@Rainyan
Copy link
Collaborator

Rainyan commented Jun 25, 2024

Build Info

20240626_a5667e3

Description

The CMake-generated build metadata can go stale on some translation units when not doing a complete rebuild (which can be slow). This can lead to confusion as to which commit exactly the currently running build is based on.

This stuff was implemented in: #349

To Reproduce

  • Checkout to some branch
  • Rebuild the full project (I tested with Visual Studio 2022 on Windows 10)
  • Verify the git HEAD's hash matches what's shown in-game
  • Make a dummy commit: git commit --allow-empty -m test
    • Contents of the change don't matter, but this is the fastest way to test it
  • Build the project (no cleaning and not "full rebuild")

Expected behavior

All the in-game build metadata still matches current latest build info.

Actual behavior

Build info has gone stale. See the git hash discrepancy between main menu and the console:

stale

For git log --oneline HEAD~2..HEAD

1e89e75 (HEAD -> master) test
a5667e3 (origin/master, origin/HEAD) Return mouse back to camera on closing VGUI dialog (#379)

you can see the main menu has picked up the dummy commit, but neo_version hasn't. The CNEOHud_HTA embedded git short hash also suffers from this, and won't necessarily update correctly without a complete rebuild.

Operating System

  • Windows
  • Linux

Version/Distro

10

Build's compiler

MSVC 19.40.33811.0

@Rainyan Rainyan added bug Something isn't working tooling Tooling related works (EX: OS fixes, libraries) labels Jun 25, 2024
@Rainyan Rainyan changed the title [BUG] Stale build metadata [BUG] Stale build metadata w/o full rebuild Jun 25, 2024
@Rainyan Rainyan changed the title [BUG] Stale build metadata w/o full rebuild [BUG] Stale metadata w/o full rebuild Jun 25, 2024
@Rainyan
Copy link
Collaborator Author

Rainyan commented Jul 7, 2024

I'm unable to reproduce this anymore, so closing for now.

@Rainyan Rainyan closed this as not planned Won't fix, can't repro, duplicate, stale Jul 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working tooling Tooling related works (EX: OS fixes, libraries)
Projects
Status: Done
Development

No branches or pull requests

1 participant