MantisBT - Doomseeker
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0003866Doomseeker[All Projects] Suggestionpublic2020-11-10 18:002021-08-17 06:30
ReporterZalewa 
Assigned ToZalewa 
PrioritynoneSeverityfeatureReproducibilityN/A
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version1.3.1 
Target Version1.3.2Fixed in Version1.3.2 
Summary0003866: Add Q-Zandronum plugin
DescriptionQ-Zandronum is a Zandronum fork made and tailored specifically for the Quake Champions: Doom Edition mod. This fork uses a separate master server. Its team member, geNia, desires to have a plugin for Doomseeker for this fork.

Quote from "geNia"

We at QC:DE mod team have been working on a Zandronum fork called Q-Zandronum for the last two months. The goal for the fork is to improve QC:DE gameplay by adding Quake style movement and improve the netcode. However, that makes it incompatible with vanilla Zandronum, so we would like to separate Q-Zandronum servers in Doomseeker to not confuse people.
I think we need to make a new Doomseeker plugin [...]


The plugin has already been made, but hasn't been checked or even reviewed, yet. I'm attaching it as a patch file to this ticket.
Steps To Reproduce
Additional InformationI am unable to find any information about this fork online. Google returns nothing on "Q-Zandronum". There are no projects in geNia's profile on OSDN (https://osdn.net/users/genia/).
TagsNo tags attached.
Relationships
related to 0003838closed Zalewa In Create Game, "Flags" tab does not have a disabled state; the tab either exists, or it doesn't – inconsistency 
Attached Filespatch 0001-Add-Q-Zandronum-plugin.patch (1,032,106) 2020-11-10 18:00
https://zandronum.com/tracker/file_download.php?file_id=2635&type=bug
Issue History
Date ModifiedUsernameFieldChange
2020-11-10 18:00ZalewaNew Issue
2020-11-10 18:00ZalewaFile Added: 0001-Add-Q-Zandronum-plugin.patch
2020-11-10 18:31geNiaNote Added: 0021559
2020-11-11 22:24Pol MNote Added: 0021560
2020-11-12 00:46Pol MAssigned To => Pol M
2020-11-12 00:46Pol MStatusnew => confirmed
2020-11-14 02:54WubTheCaptainNote Added: 0021561
2020-11-14 03:34geNiaNote Added: 0021562
2020-11-14 15:47Pol MNote Added: 0021563
2020-11-14 15:49Pol MNote Edited: 0021563bug_revision_view_page.php?bugnote_id=21563#r13267
2020-11-15 02:41geNiaNote Added: 0021564
2020-11-23 18:19ZalewaNote Added: 0021567
2020-11-23 18:19ZalewaAssigned ToPol M =>
2020-11-23 18:19ZalewaStatusconfirmed => needs testing
2021-08-07 16:44Blzut3Fixed in Version => 1.3.2
2021-08-07 16:44Blzut3Target Version => 1.3.2
2021-08-08 10:34ZalewaStatusneeds testing => resolved
2021-08-08 10:34ZalewaResolutionopen => fixed
2021-08-08 10:34ZalewaAssigned To => Zalewa
2021-08-10 05:47WubTheCaptainNote Added: 0021707
2021-08-10 05:47WubTheCaptainStatusresolved => closed
2021-08-16 19:54WubTheCaptainRelationship addedrelated to 0003838
2021-08-16 19:56WubTheCaptainProduct Version => 1.3.1
2021-08-17 06:30ZalewaNote Added: 0021744

Notes
(0021559)
geNia   
2020-11-10 18:31   
> I am unable to find any information about this fork online. Google returns nothing on "Q-Zandronum". There are no projects in geNia's profile on OSDN (https://osdn.net/users/genia/).

That is because most of the discussion was happening in QC:DE Discord server.
Anyway, you can find the source code at'https://github.com/IgeNiaI/Q-Zandronum [^]'
(0021560)
Pol M   
2020-11-11 22:24   
That's nice to see! I'll review it as soon as possible and check that everything is OK and works on Windows/Linux. I have been lurking on your discord for a while, but haven't had the courage to join any server :P (I wish I was at least as good at playing shooters as I am programming, aka competent enough XD). On the discussion on how this should be distributed (Whether this should be merged into doomseeker's repo, or hosted, compiled and distributed separately) I'm not going to say anything as I don't know what would be best.
(0021561)
WubTheCaptain   
2020-11-14 02:54   
Though I'm not a maintainer of Doomseeker, I would have no interest to maintain this in Doomseeker's repository unless every change for this plugin could be pulled from a remote Git repository for updates between releases. That would assume Q-Zandronum maintains and hosts its own plugin source.
(0021562)
geNia   
2020-11-14 03:34   
I also think that there is no need to add this to official Doomseeker repo. At least not at this moment, because Q-Zandronum is new and doesn't have many players.
(0021563)
Pol M   
2020-11-14 15:47   
(edited on: 2020-11-14 15:49)
Ok! It seems like you guys basically did a rename of most files, I like the zandronumq name to disambiguate the Q of Qt! Some small notes though:
- You have kept the IRC url, which is a service that you do not have. You probably should disable that.
- You have kept the code for the betas of zandronum, which I think you maybe do not provide. If you do provide them, I haven't said anything! But you still would have to change the url because qzandronum.com/downloads/testing/ does not seem to contain that much info.
- There are some extra urls to'https://qzandronum.com/ [^]' which would need to be changed (either the context of the sentence or whatever you would prefer!) like "You may contact Q-Zandronum staff about this through "
        "IRC: irc.qzandronum.com #zandronumq or on "
        "the forum: <a href=\"https://qzandronum.com/forum\">https://qzandronum.com/forum.</a>"

Apart from such minor things, it works as expected :)
(0021564)
geNia   
2020-11-15 02:41   
I pushed new commits and created a pull request at'https://bitbucket.org/Doomseeker/doomseeker/pull-requests/3/add-q-zandronum-plugin. [^]' Please review the changes there.

I think it is pretty much done, except that translation files are pointing at wrong lines because I removed a lot of code (mostly Zandronum 2 support). Also I need to create a new icon for the plugin.
(0021567)
Zalewa   
2020-11-23 18:19   
I have merged the PR. I realize that the plugin in its current form mostly just duplicates the already existing Zandronum plugin but I assume that this is a faux-duplication and the plugins will soon diverge. The plugin still needs to be added to the auto-update system (hint hint: tools\updateinstaller\tools\win32-configs) and then propagated to the users somehow.
(0021707)
WubTheCaptain   
2021-08-10 05:47    
$ apt show doomseeker-zandronumq
Package: doomseeker-zandronumq
Version: 2
Priority: optional
Section: devel
Maintainer: Doomseeker Team <admin@maniacsvault.net>
Installed-Size: 744 kB
Depends: doomseeker, libc6 (>= 2.14), libgcc1 (>= 1:3.0), libqt5core5a (>= 5.9.0~beta), libqt5gui5 
(>= 5.0.2), libqt5network5 (>= 5.0.2), libqt5widgets5 (>= 5.2.0~alpha1), libstdc++6 (>= 4.1.1)

Homepage:'https://doomseeker.drdteam.org [^]'
Download-Size: 152 kB
APT-Sources:'http://debian.drdteam.org [^]' stable/multiverse amd64 Packages
Description: Doomseeker plugin providing support for zandronumq.
  Provides Doomseeker with an implementation for zandronumq
 protocols.
(0021744)
Zalewa   
2021-08-17 06:30   
Quote from "Zalewa"
The plugin still needs to be added to the auto-update system (hint hint: tools\updateinstaller\tools\win32-configs) and then propagated to the users somehow.

This was done as a part of 1.3.2 release. The plugin is a part of the official release and I added the auto-updater configs.