Skip to content

feat(newserver): Military Conflict: Vietnam#4902

Merged
dgibbs64 merged 9 commits intodevelopfrom
feature/mcv
Apr 24, 2026
Merged

feat(newserver): Military Conflict: Vietnam#4902
dgibbs64 merged 9 commits intodevelopfrom
feature/mcv

Conversation

@dgibbs64
Copy link
Copy Markdown
Member

@dgibbs64 dgibbs64 commented Apr 24, 2026

Description

Add support for the Military Conflict: Vietnam dedicated game server.

Supersedes #4594 (original WIP PR from @knoxed — completing the implementation).

Changes

  • Added lgsm/config-default/config-lgsm/mcvserver/_default.cfg with full default configuration (appid 1136190, Source engine, srcds_run_x64, game dir vietnam, default map mcv_siege, querytype protocol-valve).
  • Added mcv,mcvserver,Military Conflict: Vietnam,ubuntu-24.04 entry to lgsm/data/serverlist.csv.
  • Added mcv handler block in lgsm/modules/install_config.sh.

Type of change

  • Bug fix (a change which fixes an issue).
  • New feature (a change which adds functionality).
  • New Server (new server added).
  • Refactor (restructures existing code).
  • Comment update (typo, spelling, explanation, examples, etc).

Checklist

PR will not be merged until all steps are complete.

  • This pull request links to an issue.
  • This pull request uses the develop branch as its base.
  • This pull request subject follows the Conventional Commits standard.
  • This code follows the style guidelines of this project.
  • I have performed a self-review of my code.
  • I have checked that this code is commented where required.
  • I have provided a detailed enough description of this PR.
  • I have checked if documentation needs updating.

Documentation

No documentation update required beyond the standard new server entry.

Thank you for your Pull Request!

Add support for Military Conflict: Vietnam (mcvserver) dedicated server.
- appid: 1136190
- engine: source (srcds_run_x64)
- game dir: vietnam
- default map: mcv_siege
- querytype: protocol-valve

Closes #4594
Copilot AI review requested due to automatic review settings April 24, 2026 17:56
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds initial LinuxGSM support for the Military Conflict: Vietnam dedicated server by introducing a default config, registering the server in the server list, and wiring config installation for the new shortname.

Changes:

  • Added default configuration for mcvserver (appid 1136190, Source engine settings, start parameters, directories, query settings).
  • Registered mcv in serverlist.csv.
  • Added an mcv handler block to install_config.sh for config installation.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
lgsm/modules/install_config.sh Adds a new mcv shortname branch to install default configs.
lgsm/data/serverlist.csv Registers mcv/mcvserver in the canonical server list.
lgsm/config-default/config-lgsm/mcvserver/_default.cfg Introduces the default config template for the new server.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread lgsm/modules/install_config.sh Outdated
Comment thread lgsm/config-default/config-lgsm/mcvserver/_default.cfg Outdated
Comment thread lgsm/config-default/config-lgsm/mcvserver/_default.cfg Outdated
- Remove non-existent fn_fetch_default_config call from install_config.sh
- Add +tv_port ${sourcetvport} to startparameters
- Remove unused workshop parameters section (wsapikey/wscollectionid/wsstartmap)
Add mcv entry to all 22 distro CSV dependency files.
No extra packages required - uses srcds_run_x64 (64-bit).
The serverlist-validate.sh scripts use ${datadir} to cd into the
correct directory, but the workflow never set this variable.
This caused tail to fail with "No such file or directory" as it
ran from the repo root instead of lgsm/data/.
Add squad44 and tf2c to distro CSV files where they were missing.
squad44 was missing from RHEL/CentOS/AlmaLinux/Rocky + ubuntu-24.04.
tf2c was missing from all distro CSV files.

Remove orphaned ps (GConf2 deps) from 11 distro CSV files.
ps was removed from serverlist.csv at some point but left behind
in the distro dependency files.

All 22 distro CSV files now match serverlist.csv count (139).
Update all workflows to use actions/checkout@v6 to stay current with the latest GitHub Actions standards and Node.js runner requirements.
@dgibbs64 dgibbs64 added this to the v26.2.0 milestone Apr 24, 2026
@dgibbs64 dgibbs64 merged commit 5a3c210 into develop Apr 24, 2026
5 checks passed
@dgibbs64 dgibbs64 deleted the feature/mcv branch April 24, 2026 19:56
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.

2 participants