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

QC, DOCS: Added infokey for disabling backpacks in deathmatch #191

Merged
merged 1 commit into from
Nov 2, 2024

Conversation

Zungrysoft
Copy link
Collaborator

@Zungrysoft Zungrysoft commented Oct 18, 2024

Description of Changes


I've always disliked the backpack mechanic in Quake 1 deathmatch. In duels, it punishes the out-of-control player for picking up ammo since often it will do little more than deliver the in-control player free ammo. I've added an infokey that lets you disable it. They're still enabled by default, of course.

Checklist


  • I have read the LibreQuake contribution guidelines
  • I have thoroughly tested my changes to the best of my ability
  • I confirm I have not contributed anything that would impact LibreQuake's licensing and usage
  • This Pull Request fixes a critical issue that should be reviewed and merged as soon as possible

@Nolcoz
Copy link
Collaborator

Nolcoz commented Oct 19, 2024

How does this work? Is it a command you have to put in console?

@Zungrysoft
Copy link
Collaborator Author

Zungrysoft commented Oct 19, 2024

@Nolcoz QuakeWorld engines support infokeys, which are additional settings that change the way the game plays. The original QuakeWorld had infokeys like "dq", which controls whether or players drop the Quad Damage if they die while holding it or if it's just gone for good. I've added some new ones to let players customize their deathmatch play a bit more. They're all optional and must be enabled in the console. The console command to enable this particular infokey would be:

serverinfo disable_backpacks 1

Copy link
Collaborator

@Nolcoz Nolcoz left a comment

Choose a reason for hiding this comment

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

Ahh I see. Not really experienced with deathmatch much but its optional so its fine

Copy link
Owner

@lavenderdotpet lavenderdotpet left a comment

Choose a reason for hiding this comment

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

looks good to me merge when ur ready @Zungrysoft

@Zungrysoft Zungrysoft merged commit 20b5305 into main Nov 2, 2024
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.

3 participants