MantisBT - Doomseeker
View Issue Details
0003536Doomseeker[All Projects] Bugpublic2018-10-06 10:582018-10-13 16:55
WubTheCaptain 
Pol M 
normalblockalways
closedfixed 
amd64Debian GNU/Linuxbuster/sid
1.2 
1.21.2 
0003536: Regression: Using --rcon is no longer supported by itself
Using --rcon by itself was added in hg changeset 382:41ddf4377133 (Tue Apr 13 22:31:16 2010 +0000) by Blzut3.

Since sometime in 1.2~beta, it now requires two parameters to use and there's now dead code lying around. This is a regression in functionality.
$ doomseeker --rcon # Doomseeker 1.1
[10:55:50] Starting Doomseeker. Hello World! :)
[10:55:50] Setting up data directories.
[10:55:50] Attempting to load plugins from directory: /home/wub/.doomseeker/engines
[10:55:50] Attempting to load plugins from directory: /usr/bin/engines
[10:55:50] Attempting to load plugins from directory: engines
[10:55:50] Attempting to load plugins from directory: /usr/share/doomseeker/engines
[10:55:50] Loaded plugin: "Zandronum"!
[10:55:50] Listening to Zandronum's LAN servers broadcasts on port 15101.
[10:55:50] Initializing configuration file.
[10:55:50] Setting INI file: /home/wub/.doomseeker/doomseeker.ini
[10:55:50] Loading translations definitions
[10:55:50] Loading translation "en_EN".
[10:55:50] Translation loaded.
[10:55:50] Initializing IP2C database.
[10:55:50] Initializing passwords configuration file.
[10:55:50] Initializing configuration for plugins.
[10:55:50] Initializing IRC configuration file.
[10:55:50] Setting IRC INI file: /home/wub/.doomseeker/doomseeker-irc.ini
[10:55:50] Starting RCon client.


$ doomseeker --rcon # Doomseeker 1.2~beta-181002-1931
doomseeker: expected 2 argument(s) in option --rcon

Available command line parameters:
[...]
Prevents testing of rcon dialog → "block" severity.
No tags attached.
related to 0003373closed Zalewa "No RCon support" dialog message is not printed to stderr 
Issue History
2018-10-06 10:58WubTheCaptainNew Issue
2018-10-06 10:58WubTheCaptainRelationship addedrelated to 0003373
2018-10-06 11:27Pol MAssigned To => Pol M
2018-10-06 11:27Pol MStatusnew => assigned
2018-10-06 11:33Pol MNote Added: 0019935
2018-10-06 11:39Pol MNote Edited: 0019935bug_revision_view_page.php?bugnote_id=19935#r12120
2018-10-06 11:41Pol MNote Edited: 0019935bug_revision_view_page.php?bugnote_id=19935#r12121
2018-10-06 11:55Pol MNote Added: 0019937
2018-10-06 11:55Pol MStatusassigned => needs review
2018-10-06 17:25ZalewaNote Added: 0019956
2018-10-06 17:25ZalewaStatusneeds review => needs testing
2018-10-06 18:02WubTheCaptainTarget Version => 1.2
2018-10-06 18:05WubTheCaptainStatusneeds testing => resolved
2018-10-06 18:05WubTheCaptainFixed in Version => 1.2
2018-10-06 18:05WubTheCaptainResolutionopen => fixed
2018-10-13 16:55WubTheCaptainStatusresolved => closed

Notes
(0019935)
Pol M   
2018-10-06 11:33   
(edited on: 2018-10-06 11:41)
This was done in this commit done by me. I'll remove the unnecessary stuff.
EDIT: oh, I was trying to achieve that when an option that required arguments was passed without them it failed. The affected options were "--connect", "--datadir" and "--rcon".
EDIT2: Based on the "--help" page, --datadir should also not error if no path is given.

(0019937)
Pol M   
2018-10-06 11:55   
pr
(0019956)
Zalewa   
2018-10-06 17:25   
PR merged through squash & rebase:'https://bitbucket.org/Doomseeker/doomseeker/commits/12c46bcc9cf04f97452e5561154398b6fff68d1e [^]'