The "DOS" ARS keyword was always evaluating to false, on all platforms, for the C version of chk_ar(), which is used for populating JS *_area objects and for the User.compare_ars() implementation.

Unfortunately, the startup (sbbs.ini) "NO_DOS" option is not recognized here (yet), so it'll report true (e.g. for Win64 or Linux systems with DOSemu) even if/when the NO_DOS option is set.

e.g. don't log the error "term !ERROR resolve_ip example.com failed with error -2", just let the script deal with the failure and log an error if it wishes.

Pre-increment would've worked too.

prextrn.js keys off of the current node action to detect if the user is
currently in the logon process. Defeat that check in this case (there is no
logon process for auto-xtrn's).

add global head and tail for both section menu and external program menu

See merge request !49

use options.clear_screen for clear instead of hardcoded value in SelectItemHdr
remove redundant code for input in main section menu

... or when passed an invalid external program code or number, rather just just returning false. The underlying exec_xtrn() C++ function may return false (failure) for other reasons, so let's not obfuscate the other potential failure causes.

Previously, the xtrn*_tail menu was only displayed in the programmatic menu
mode.

Also, using the P_NOERROR mode flag for bbs.menu() to simplify all the if/else
logic.

And use require() instead of load().

Nelgin/xtrn tail

See merge request !48

Nigel Reed authored 4 years ago and Rob Swindell committed 4 years ago

display the file after the list of external programs.

cmdstr() is often called for non-executable command-lines (e.g. for @include: file name/path arguments), so don't default to native unless a mode value was specified. Hopefully addresses issue reported by bobrob without breaking anything else (again).

This reverts commit 1597e9a9.

Always check the command-line for a '*' or '?' prefix and compare the executed filename against the native program list. For DOSEMU magic drive/path name command-line specifier substitution, but only when invoking DOS programs. Reported by bobrob via IRC.

xmodem_cancel() would return an error if putcom() failed, *before*
setting the "cancelled" [sic] flag. xmodem_cancel() in response to the
receipt of a remote cancel request (^X character).

Safe-string handling in C/C++.

Bug fix for net type when forwarding messages to internet email or fidonet netmail

See merge request !43

Eric Oulashin authored 4 years ago and Rob Swindell committed 4 years ago

When forwarding a message, it now correctly sets sets the to_net_type property in the
message header to FidoNet or internet for those types of message destinations

This can be used to defeat the automatic inclusion of the sbbs3
"setcap" build target when building on Linux. This should address
issue #175.

Update game.js - the game is showing 2 pauses when exiting. This code is not...

See merge request !46

Don't do the DOSEMU drive-letter dance for xtrn startup directory if the xtrn's command line is implicitly native (e.g. Baja or JS).

Also, recognize Baja command-lines as native in cmdstr() - for %n, %!, etc. DOSemu expansion hack.

If a node's configuration can't be loaded (e.g. nodeX/node.cnf is missing), fall-back and load the "first node" (e.g. Node1) configuration. On recent *nix installs (since the migration to Git), the node2+/node.cnf files were not copied/created, so let's solve that soon-to-be-FAQ now.

This resolves the issue of node2+/node.cnf file missing.

Update README.txt - website defunct

See merge request !44

Update sysop.doc - cvs is gone, and these files haven't changed in a long time...

See merge request !42

Fixes issue introduced in last commit to this file (the each-node-has-its-own-config-in-memory enhancement) that would prevent users from logging in because their time was reduced for an upcoming event that had already run. The change assumes the event thread is running (its the thing that reads the event last-run info from time.dab). Thanks to mlong for the report.

Update game.js - the game is showing 2 pauses when exiting. This code is not even needed since you can set "pause after execution" in scfg. Now if that is enabled, there is only one pause shown. And if not enabled, there is no forced wait/press a key. Seems it would benefit sysops who want it either way.

Update sysop.doc - cvs is gone, and these files haven't changed in a long time so its really not needed in the instructions

... between all nodes of  a instance sbbs instance. That's now no longer true.

Previously, all nodes shared the same copy of the configuration in memory. This prevented any node from seeing an updated configuration until all nodes were offline or waiting for connection and could be recycled. Now, no recycling of the server is needed for a node to load a new config. A node in-use *still* cannot reload configuration until the user disconnects, but all you have to do is logon to another unused node and you (or that user) will get an updated configuration. Bots or server recycling problems will no longer prevent the sysop or user from getting a current configuration when connecting to any node.

This is one of those instances of giving the sysop what he wants rather than what he asked for. I should've done this a long time ago, but the idea just occurred to me. You're welcome. :-)

Let UIFC do the truncation as necessary to fit windows/forms, when applicable.
Thanks for jc and nelgin for pointing this out via IRC.

The name length is limited to 25 chars, so we don't need a 50-char wide menu here.

Thanks for pointing that out mlong.

(emulating the syslog date/timestamp format). Makes debugging file transfer issues easier.

And a couple of minor cosmetic adjustments.

misspellings in jsdocs

See merge request !40