Skip to content

Conversation

@nullsystem
Copy link
Collaborator

@nullsystem nullsystem commented Nov 17, 2025

Description

  • Keeps two hidden client convars: cl_neo_cfg_version_major and cl_neo_cfg_version_minor to keep track of which version the config was set for. Default value of "0" so it can recognize config previous to cl_neo_cfg_version_.... Major/minor versions are checked and set on client startup and will only set the new keybinds if the bind wasn't set, and the default key wasn't set.
  • The config version follows NT;RE major.minor version and extracts the version from the git tag with cmake, which puts it into neo_version_number.h header
  • Reviewers/testers can set cl_neo_cfg_version_... and unset relevant keybinds to emulate upgrading from older config to current
  • Currently sets mp3, kdinfo, ping, and +aim -> toggle_aim if config version < 22.0/unset
  • Updated github workflow to fully fetch tags and history as it's required now

Toolchain

  • Linux GCC Distro Native Arch/GCC 15

Linked Issues

@nullsystem nullsystem requested a review from a team November 17, 2025 21:48
@nullsystem nullsystem force-pushed the GH-1332_UpdateBindingOnUpdate branch 4 times, most recently from 903c2f7 to c7b9096 Compare November 17, 2025 22:09
Masterkatze
Masterkatze previously approved these changes Nov 18, 2025
Copy link
Contributor

@Masterkatze Masterkatze left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

sunzenshen
sunzenshen previously approved these changes Nov 19, 2025
@nullsystem
Copy link
Collaborator Author

@Masterkatze @sunzenshen Added +aim -> toggle_aim handling

* Keeps two hidden client convars: `cl_neo_cfg_version_major` and
  `cl_neo_cfg_version_minor` to keep track of which version the
  config was set for. Default value of "0" so it can recognize
  config previous to `cl_neo_cfg_version_...`. Major/minor versions
  are checked and set on client startup and will only set the new
  keybinds if the bind wasn't set, and the default key wasn't set.
* The config version follows NT;RE `major.minor` version and extracts
  the version from the git tag with cmake, which puts it into
  `neo_version_number.h` header
* Reviewers/testers can set `cl_neo_cfg_version_...` and unset
  relevant keybinds to emulate upgrading from older config to current
* Currently sets mp3, kdinfo, ping, and aim keybind if config version
  < 22.0/unset
* fixes NeotokyoRebuild#1332
@nullsystem nullsystem force-pushed the GH-1332_UpdateBindingOnUpdate branch from 1e309c0 to f0b655e Compare November 21, 2025 19:12
@nullsystem nullsystem merged commit 0352dc3 into NeotokyoRebuild:master Nov 22, 2025
7 checks passed
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.

Add binding on update

3 participants