ChangeLog 528 KB
Newer Older
Per Cederqvist's avatar
Per Cederqvist committed
1 2
2006-01-02  Per Cederqvist  <ceder@lysator.liu.se>

3 4 5 6 7 8 9 10 11 12
	Update valgrind options to valgrind-2.4.0.
	* src/server/testsuite/lyskomd.supp (i_fopen-2.3.1): New name for
	former i_fopen.
	(i_fopen-2.3.5): New suppression.
	* src/server/testsuite/config/unix.exp (l2g_start): Use valgrind
	option --leak-check=full, not the obsolete variant
	--leak-check=yes.
	(spawn_lyskomd): Ditto.
	(dbck_run): Run with --leak-check=full.

13 14 15 16
	Fixed a type mismatch in adns.
	* src/libraries/adns/src/event.c (adns_processreadable):
	udpaddrlen should be a socklen_t, not int.

Per Cederqvist's avatar
Per Cederqvist committed
17 18 19 20 21 22 23
	Xenofarm warning fixes.
	* src/libraries/adns/src/check.c: Include <stdlib.h>, which is
	needed since abort() is used here.
	* scripts/warnings.sed: gcc 3.4.3 uses 'foo' instead of `foo' to
	mark identifiers.  Adjusted a few warnings that were found by
	various Xenofarm machines.

24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54
2006-01-01  Per Cederqvist  <ceder@lysator.liu.se>

	Since .cvsignore no longer exist, we cannot distribute it.
	* Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* db-crypt/Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* db-crypt/db/Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* doc/Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* doc/man/Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* run-support/Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* scripts/Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* scripts/lyskomd-copyrights: Removed .cvsignore.
	* src/Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* src/include/Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* src/include/server/Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* src/libraries/Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* src/libraries/adns/Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* src/libraries/adns/client/Makefile.am (EXTRA_DIST): Removed
	.cvsignore.
	* src/libraries/adns/src/Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* src/libraries/libansi/Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* src/libraries/libcommon/Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* src/libraries/libeintr/Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* src/libraries/libmisc/Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* src/libraries/liboop/Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* src/server/Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* src/server/testsuite/Makefile.am (EXTRA_DIST): Removed .cvsignore.
	* src/server/testsuite/config/Makefile.am (EXTRA_DIST): Removed
	.cvsignore.
	* src/server/testsuite/lyskomd.0/Makefile.am (EXTRA_DIST): Removed
	.cvsignore.

Per Cederqvist's avatar
Per Cederqvist committed
55 56
2005-12-28  Per Cederqvist  <ceder@lysator.liu.se>

57 58 59 60 61 62 63 64 65 66 67 68 69 70 71
	Cleanup of simple-cache.c.  (Bug 172).  Allocate less memory when
	performing a pattern name match.  
	* src/server/simple-cache.c: Removed obsolete comments.  Fixed
	spelling errors in comments.
	(find_matching_info_compare): Don't overflow if a Conf_no doesn't
	fit in an int.
	(find_matching_info): Removed obsolete comments (bug 172).
	(rebuild_matching_info_entry): Added a comment that explains a
	magic "2" in the code.
	(build_matching_info): Use cached_no_of_existing_conferences()
	instead of next_free_num when deciding how large temporary table
	we need.  Reduce the number of loop induction variables.
	(cached_lookup_name): Made code more readable by introducing a
	temporary variable.

Per Cederqvist's avatar
Per Cederqvist committed
72 73 74 75 76
	Improved test suite logging.
	* src/server/testsuite/config/unix.exp (simple_expect): Log what
	we are looking for and when we are done looking at verbosity level
	2 and above.

77
2005-12-27  Per Cederqvist  <ceder@lysator.liu.se>
78 79 80 81

	Add a testsuite temporary file to svn:ignore.
	* src/server/testsuite: Add locksuite.lock to svn:ignore.

82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98
	Don't overestimate the number of conferences as next_free_num when
	allocating certain data structures.  (Bug 164).
	* src/server/simple-cache.c (existing_confs): New static
	variable.
	(cached_no_of_existing_conferences): Return existing_confs,
	instead of overestimating the number of conferences as
	next_free_num.  (Bug 164).
	(cached_create_conf): Increase existing_confs.
	(cached_delete_conf): Decrease existing_confs.
	(init_cache): Increase existing_confs when a new conference is
	created.
	(free_all_cache): Decrease existing_confs when conferences are
	deallocated.
	(dump_cache_mem_usage): Dump existing_confs.
	* src/server/testsuite/config/unix.exp (check_memory_usage): Check
	the dump of existing_confs.  Fail if allocated_strings or
	allocated_blocks isn't found in the file.
99 100 101 102 103 104 105 106 107 108 109 110

	Simplified the name parsing routing, and save some small amount of
	memory, by removing the priority field of Matching_info.
	* src/libraries/libcommon/parser.c (parse): Don't let the priority
	influence the match.  The priority functionality was poorly
	documented, not used by lyskomd, and made the code more complex.
	* src/libraries/libcommon/parser.h (Matching_info): Removed the
	priority field, which is no longer used.
	* src/server/simple-cache.c (rebuild_matching_info_entry): Don't
	initialize the priority field.
	(build_matching_info): Ditto.

Per Cederqvist's avatar
Per Cederqvist committed
111 112 113 114 115 116 117
2005-12-25  Per Cederqvist  <ceder@lysator.liu.se>

	Minor code cleanup.
	* src/libraries/libcommon/parser.c (parse): Code cleanup: get rid
	of one level of indentation, and avoid using multiple ways to get
	to the next iteration of the main loop.

118 119 120 121 122 123 124 125
2005-12-24  Per Cederqvist  <ceder@lysator.liu.se>

	Writing to a client that was disconnecting and blocked could cause
	the server to crash.  (Bug 1403).
	* src/server/connections.c (check_idle_callback): Don't crash if
	isc_enable() fails.  Emit the session number in the log, mostly
	for debug purposes.

Per Cederqvist's avatar
Per Cederqvist committed
126 127
2005-12-23  Per Cederqvist  <ceder@lysator.liu.se>

128 129 130 131 132 133 134 135 136
	Add automake-generated files to svn:ignore.
	* scripts: Add depcomp, install-sh, mdate-sh, missing,
	mkinstalldirs and texinfo.tex to svn:ignore.
	* src/libraries/adns: Add depcomp, missing,  and mkinstalldirs to
	svn:ignore.
	* src/libraries/libisc-new: Add depcomp to svn:ignore.
	* src/libraries/liboop: Add depcomp, install-sh, missing and
	mkinstalldirs to svn:ignore.

Per Cederqvist's avatar
Per Cederqvist committed
137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156
	Add TAGS to svn:ignore.
	* src/include/server: Add TAGS to svn:ignore.
	* src/include: Ditto.
	* src/libraries/adns/client: Ditto.
	* src/libraries/adns/src: Ditto.
	* src/libraries/adns: Ditto.
	* src/libraries/libansi: Ditto.
	* src/libraries/libcommon: Ditto.
	* src/libraries/libeintr: Ditto.
	* src/libraries/libisc-new/src: Ditto.
	* src/libraries/libisc-new: Ditto.
	* src/libraries/libmisc: Ditto.
	* src/libraries/liboop: Ditto.
	* src/libraries/regex: Ditto.
	* src/libraries: Ditto.
	* src/server/testsuite: Ditto.
	* src/server: Ditto.
	* src: Ditto.
	* .: Ditto.

157 158 159 160 161 162 163 164 165 166
2005-12-22  Per Cederqvist  <ceder@lysator.liu.se>

	Don't hang if "Use DNS" is off.  (Bug 1318.)  Thanks to Rasmus
	Sten who reported the bug and proposed an alternate fix.
	* src/server/connections.c (login_request): Don't hang if the "Use
	DNS" option is false.  (Bug 1318).  Log a warning message if
	isc_resolve_remote() fails.
	(src/server/testsuite/lyskomd.0/bug-1318.exp): New file.  Test the
	"Use DNS: off" configuration option.

167 168 169 170 171 172 173 174 175 176 177 178
2005-12-21  Per Cederqvist  <ceder@lysator.liu.se>

	"make check" without a previous "make" failed, because
	checkkomspace was not built early enough.  Also, the test suite
	should not fail on platforms that don't support fsusage.c.
	* src/server/Makefile.am (CHECKKOMSPACE): New empty variable.
	Append checkkomspace$(EXEEXT) to it if we HAVE_FSUSAGE.
	(check-lyskomd): Depend on $(CHECKKOMSPACE).
	(check-recursive): Ditto.
	* src/server/testsuite/lyskomd.0/50.exp: Report "unsupported" if
	checkkomspace is not built.  (Don't fail with fatal errors.)

179 180
2005-12-19  Per Cederqvist  <ceder@lysator.liu.se>

Per Cederqvist's avatar
Per Cederqvist committed
181 182 183 184 185 186
	Fixed test suite bitrot.
	* src/server/testsuite/lyskomd.0/11.exp: Handle the message that
	is emitted when the debug calls are disabled.  That message was
	introduced 2004-07-21; apparently, nobody has used the test suite
	without debug calls since then.

187 188
	* Switched from CVS to Subversion.

Per Cederqvist's avatar
Per Cederqvist committed
189 190 191 192 193 194 195 196
2005-12-18  Per Cederqvist  <ceder@lysator.liu.se>

	Update copyright headers.
	* <all files>: update the year in the copyright notice.
	* scripts/lyskomd-copyrights: Ignore m4/* and
	src/server/testsuite/lyskomd.0/*.aux.  Updated year in copyright
	statement.  Ignore src/server/fsusage.[hc].
	
197 198 199 200 201 202 203 204 205 206 207 208 209 210 211 212 213 214 215 216 217 218 219 220 221 222
2005-12-18  Per Cederqvist  <ceder@moria.cederqvist>

	Implemented the disable-client debug call, and use it in a test
	suite.  Some minor code refactoring.
	* src/include/services.h (disable_client): New debug request.
	* src/server/fncdef.txt (disable_client): New debug request.
	* src/server/connections.h (Connection) [DEBUG_CALLS]: New field:
	blocked_by_disable_client.
	* src/server/internal-connections.c (init_connection)
	[DEBUG_CALLS]: Initialize blocked_by_disable_client.
	(new_client): Ditto.
	* src/server/connections.c (parse_unparsed) [DEBUG_CALLS]: Don't
	process any request if the client is disabled by disable_client.
	(may_read_more): New method, extracted from read_from_connection.
	Return false if disabled by disable_client.
	(should_be_disabled): New method, extracted from
	read_from_connection and check_idle_callback. Return true if
	disabled by disable_client.
	(read_from_connection): Simplified code by using may_read_more and
	should_be_disabled.
	(check_idle_callback): Simplified code by using should_be_disabled.
	(disable_client): New debug request.
	* src/server/testsuite/lyskomd.0/51.exp: New test suite. Test that
	nothing is read from a disabled client, and that the output queue
	fills up properly.

223 224 225 226 227 228
2005-12-12  Per Cederqvist  <ceder@ingate.com>

	x-face is a broken and historical image format, so don't promote
	it.
	* doc/Protocol-A.texi (Aux-Item Types): Deprecate x-face.

Per Cederqvist's avatar
Per Cederqvist committed
229 230 231 232 233
2005-12-12  Per Cederqvist  <ceder@lysator.liu.se>

	Added on assert.
	* src/server/stats.c (update_one_stat): Added an assert.

234 235 236 237 238 239 240 241 242 243 244 245 246 247 248 249 250 251 252 253
2005-12-11  Per Cederqvist  <ceder@lysator.liu.se>

	The get-members-old request is often better than get-members, so
	remove the "Obsolete" status from it.
	* doc/Protocol-A.texi (Protocol Requests): get-members-old is now
	being promoted from "Obsolete" to "Recommended".
	(get-members-old): Ditto.  Discuss get-members versus
	get-members-old.
	(get-members): Ask client writers to consider using
	get-members-old.

	Clarify how create-conf-old works.
	* doc/Protocol-A.texi (create-conf-old): Document when the ability
	to send an Extended-Conf-Type was introduced, and what it means to
	send a four-bit Conf-Type.

	Doc fixes.
	* doc/Protocol-A.texi (create-text): Typo fixed.
	(Mapping Local to Global Text Numbers): Typo fixed.

Per Cederqvist's avatar
Per Cederqvist committed
254 255 256 257 258 259
2005-12-11  Per Cederqvist  <ceder@moria.cederqvist>

	Doc fixes.
	* doc/Protocol-A.texi (set-membership-type): Typo fixed.
	* HACKING: Update paths to the files on the Lysator web server.  

Per Cederqvist's avatar
Per Cederqvist committed
260 261 262 263 264 265
2005-11-28  Per Cederqvist  <ceder@moria.cederqvist>

	Comment fix.
	* doc/checkargs.py (generate_summary_output): Fixed a typo in the
	generated comment.

Per Cederqvist's avatar
Per Cederqvist committed
266 267 268 269 270 271 272
2005-05-06  Per Cederqvist  <ceder@lysator.liu.se>

	Doc fix.
	* doc/Protocol-A.texi (get-uconf-stat): Don't state that this is
	the only way to get all the flag bits of the conference.  It
	isn't; get-conf-stat works just fine.

Per Cederqvist's avatar
Per Cederqvist committed
273 274 275 276 277 278
2005-04-04  Per Cederqvist  <ceder@lysator.liu.se>

	Doc fix.
	* doc/Protocol-A.texi (local-to-global-reverse): Fixed typo.
	* doc/constructs.expected: Updated.

279 280 281 282 283 284
2004-12-13  Per Cederqvist  <ceder@ingate.com>

	Added a comment.
	* src/server/person.c (get_person_stat): Added a comment that
	explains why get-person-stat is allowed without logging in.

Per Cederqvist's avatar
Per Cederqvist committed
285 286 287 288 289 290
2004-12-02  Per Cederqvist  <ceder@ingate.com>

	Doc fix.
	* doc/Protocol-A.texi (mark-as-read): Document the error code
	long-array.

291 292
2004-11-30  Per Cederqvist  <ceder@ingate.com>

Per Cederqvist's avatar
Per Cederqvist committed
293 294 295 296
	Release doc fix.
	* HACKING (Release generation): Check third-party packages for new
	versions.

297 298 299 300
	Allocate a range of aux-item types to Canidius.  (Bug 1440).
	* doc/Protocol-A.texi (Client-Specific Aux-Item Types): Allocate
	10300-10399 to Canidius.

301 302 303 304 305 306 307 308 309 310 311 312 313 314 315
2004-08-03  Per Cederqvist  <ceder@ceder.dyndns.org>

	Minor cleanup of the multipart-related documentation supplied by
	Rasmus Sten.
	* doc/Protocol-A.texi (Aux-Item Types): Remove inferior
	documentation of multipart handling by allowed-content-type; that
	is now obsoleted by the better documentation that Rasmus Sten
	provided.  Fix markup and language errors in the documentation by
	Rasmus.
	(Multipart (multipart/mixed)): Fixed markup errors.  Clarify that
	text/x-kom-basic is preferred.
	(MHTML (message/rfc822;x-lyskom-variant=rfc2557)): Fixed markup
	errors.  Clarify that it is automatic access to external resources
	that should be stopped.

316 317 318 319 320 321 322 323 324 325 326 327 328 329
2004-08-02  Per Cederqvist  <ceder@ceder.dyndns.org>

	* AUTHORS: Added Rasmus Sten.

2004-08-02  Rasmus Sten  <rasmus@bricole.se> (committed by ceder)

	Document how multipart content should be handled by Protocol A
	clients, and document MHTML and multipart/mixed.
	* doc/Protocol-A.texi (Aux-Item Types): State that
	allowed-content-type should be applied recursively to all content
	types, in much more detail than in the change of 2004-07-18.
	(Multipart (multipart/mixed)): New section.
	(MHTML (message/rfc822;x-lyskom-variant=rfc2557)): New section.

330 331 332 333 334 335 336 337 338 339 340 341 342 343 344 345 346 347 348 349 350 351 352 353 354 355 356 357 358 359 360 361 362 363 364 365 366 367 368 369 370 371 372 373 374 375 376 377
2004-07-21  Per Cederqvist  <ceder@ingate.com>

	Increase coverage of the aux-item definitions parser.  (Bug 64).
	Get rid of a spurious error message found in the process.  Use
	less preprocessor macro magic.
	* src/server/testsuite/lyskomd.0/bug-64.exp: New file.  Increase
	test coverage of aux-item-def-parse.y.
	* src/server/testsuite/lyskomd.0/bug-64.aux: New file.
	* src/server/testsuite/lyskomd.0/bug-64-2.aux: New file.
	* src/server/simple-cache.c (pre_sync): Use VBUG instead of BUG to
	report sync start and stop.
	* src/server/aux-item-def-parse.y (CHK_ASSIGN): Macro rewritten as
	the new chk_assign() function, with a slightly different API.
	This was mainly done to get rid of the error message "invalid
	field name: xyzzy" that occured if the value of xyzzy was bogus,
	even if xyzzy itself was a valid name.  All callers updated.
	(CHK_FLAG_A): Similarly rewritten as chk_flag_a().
	(assign): Check for valid field name and valid field content
	separately, so that the "invalid field name: xyzzy" message isn't
	printed just because the content is bad.
	(chk_assign): New name for former aux_item_def_check_assign.  New
	arguments: found and good; return status via them.
	(chk_flag_a): New function.
	(aux_item_def_check_trigger): New arguments: found and good;
	return status via them.
	(aux_item_def_check_validate): Ditto.
	
	Test suite framework improvements: simple_expect can now expect
	lines that don't contain the line leader, and lyskomd_start and
	lyskomd_fail_start can now handle output sent before the database
	lock is taken.
	* src/server/testsuite/config/unix.exp (simple_expect): If the
	regexp starts with "<<<", remove it, and don't expect a line
	leader.  This is useful for catching messages from lyskomd that
	are created by libraries that we have no control over, and that
	don't emit the expected line leader.
	(set_debug_calls): New proc, extracted from lyskomd_start and 
	lyskomd_fail_start.  Expect the new message that is emitted when
	the debug calls are disabled.
	(lyskomd_start): New optional argument: pre_lock_messages.  Moved
	code to set_debug_calls().
	(lyskomd_fail_start): Ditto.
	* src/server/ramkomd.c (main): Write a message when debug calls
	are disabled, to help the test suite.

	Fixed a comment.
	* src/server/testsuite/timeval-overflow.c: Fixed a comment.

378 379
2004-07-18  Per Cederqvist  <ceder@ingate.com>

380 381 382 383 384 385 386 387 388 389 390 391 392 393 394 395 396 397 398 399 400 401 402 403 404 405 406 407 408 409 410 411 412 413 414 415 416 417 418 419 420 421 422 423 424 425 426 427
	Implement async-text-read and async-invalidate-text-read.  (Bug
	985).
	* src/server/testsuite/lyskomd.0/52.exp: Test async-text-read and
	async-invalidate-text-read.
	* src/server/testsuite/lyskomd.0/03.exp: Handle async-text-read
	and async-invalidate-text-read.
	* src/server/session.c (accept_async): Handle ay_text_read and
	ay_invalidate_text_read.
	* src/server/send-async.h, src/server/send-async.c
	(async_text_read): New function.
	(async_invalidate_text_read): New function.
	* src/server/prot-a-send-async.h, src/server/prot-a-send-async.c
	(prot_a_async_text_read): New function.
	(prot_a_async_invalidate_text_read): Ditto.
	* src/server/membership.c (mark_as_read): Send async-text-read.
	(mark_as_unread): Send async-invalidate-text-read.
	(set_read_ranges): Ditto.
	(set_unread): Ditto.
	(set_last_read): Ditto.
	* src/server/async.h (enum async): Added ay_text_read and
	ay_invalidate_text_read.
	* doc/Protocol-A.texi (Membership Information): Added the Text-Id
	type.
	(async-text-read): New async message.
	(async-invalidate-text-read): New async message.
	* doc/constructs.expected: Updated.

	Added missing newline in kom_log call.
	* src/server/connections.c: (set_time): Added a missing newline in
	a log message.

	Doc fixes.
	* src/server/testsuite/locksuite.py: Doc fixes. 
	* src/server/connections.c: Doc added.
	* doc/Protocol-A.texi (Person Status Types): Added missing closing
	parenthesis.
	(Who Information): Ditto.
	(modify-system-info): Fixed references to argument names in the
	error code descriptions.
	(Aux-Item Types): Don't use @field for anything but Protocol A
	structure fields.  

	Specify how multipart contents should be handled by the aux-item
	allowed-content-type.
	* doc/Protocol-A.texi (Aux-Item Types): State that
	allowed-content-type should be applied recursively to all content
	types.

428 429 430 431 432 433 434 435
	Fix bit decay in the test suite.
	* src/server/testsuite/lyskomd.0/bug-1121.exp: Turn off garbing
	when reading the old bug-1121 database so that the garb doesn't
	remove our test data.
	* src/server/testsuite/lyskomd.0/46.exp: Bison 1.875 uses a
	slightly different error message; update the regexp to accept it
	as well as older versions.

David Byers's avatar
David Byers committed
436 437 438 439 440 441 442 443 444 445 446 447 448 449 450
2004-06-19  David Byers  <byers@lysator.liu.se>

	Documentation improvements:
	* doc/lyskomd.texi (Aux-Item Definition File): Documented
	supervisor-only and author-only more accurately (bug 530).

	* doc/Protocol-A.texi (Aux-Item Types): Document letterbox, text
	and other type parameters of aux-item definitions (bug 9).
	(Person Status Types, Who Information, Session Information):
	Improved documentation of the username field (bug 196).
	(add-member): Fixed documentation of error codes (bug 199).
	(async-i-am-on): Fixed error in the message description (bug 200).
	(Importing and Exporting E-Mail): Improved documentation on what
	aux-items affect importers (bug 201).

451 452 453 454 455 456
2004-04-26  Per Cederqvist  <ceder@ceder.dyndns.org>

	Added weblatte to the list of registered clients.
	* doc/Protocol-A.texi (set-client-version): Added weblatte to the
	list of registered clients.

Per Cederqvist's avatar
Per Cederqvist committed
457 458 459 460 461 462
2004-04-02  Per Cederqvist  <ceder@ingate.com>

	Gentoo portability.
	* mkmi (WANT_AUTOMAKE): Set it.
	(WANT_AUTOCONF): Set it.

Per Cederqvist's avatar
Per Cederqvist committed
463 464 465 466 467
2004-01-06  Per Cederqvist  <ceder@ceder.dyndns.org>

	Fix "make check".
	* doc/constructs.expected: Updated.

468 469 470 471 472 473
2004-01-05  Per Cederqvist  <ceder@ceder.dyndns.org>

	Xenofarm: detect warnings during ckprg.
	* scripts/xenofarm.sh (ckprg): Use the ckprgwarn function, so that
	warnings during ckprg are detected.

474 475 476 477 478 479 480 481
2003-12-28  Per Cederqvist  <ceder@ceder.dyndns.org>

	Protocol A whitespace usage was only documented in the direction
	from the client to the server!
	* doc/Protocol-A.texi (Client-Server Dialog): Document how
	elements sent from the server to the client are separated and
	terminated.

482 483 484 485 486 487 488 489 490 491 492 493 494 495 496 497 498 499
2003-11-28  Per Cederqvist  <ceder@ceder.dyndns.org>

	Protocol A clarifications: %g uses the "C" locale, no-of-texts is
	misnamed, read-ranges is sorted, and the description of what is
	unread was outdated.
	* doc/Protocol-A.texi (Simple Data Types): The %g format specifier
	is locale-dependent in the C specification, so say that FLOAT is
	formatted as %g is formatted in the "C" locale.
	(Conference Status Types): Fixed the description of the
	no-of-texts field.  It is actually the difference between the
	first unused local text number and first-local-no, and that will
	differ from the number of texts in the conference if a text other
	than the oldest one is removed.
	(Membership Information): Document that read-ranges is kept in
	ascending order.
	(What do I have unread): Update the description with the version
	11 requests.

Per Cederqvist's avatar
Per Cederqvist committed
500 501 502 503 504 505
2003-10-06  Per Cederqvist  <ceder@ceder.dyndns.org>

	Port to IRIX make.
	* doc/Makefile.am (check-doc): Don't have blank lines among the
	commands, since make on IRIX chokes on them.

506 507 508 509 510 511 512 513 514 515 516 517
2003-10-04  Per Cederqvist  <ceder@ceder.dyndns.org>

	The timestamp in a database file shold be the time when the
	snapshot was taken.  It should not be updated when the file is
	changed from DIRTY to CLEAN state.
	* src/server/simple-cache.c (sync_output_header): New argument:
	include_timestamp.
	(pre_sync): The header should include a timestamp.
	(save_one_text): The header should not include a timestamp when
	the "CLEAN" marker is written.  We want the timestamp in the file
	to be the time when the snapshot was taken.

518 519
2003-10-03  Per Cederqvist  <ceder@ceder.dyndns.org>

520 521 522
	Ignore harmless warnings from fsusage.c.
	* scripts/warnings.sed: Ignore warnings produced by fsusage.c.
	
Per Cederqvist's avatar
Per Cederqvist committed
523 524 525 526
	Fixed typo.
	* m4/Makefile.am (EXTRA_DIST): Fixed typo: "intmax_t.m4" ->
	"uintmax_t.m4".

527 528 529
	Port checkkomspace to non-GNU systems.
	* src/server/Makefile.am (checkkomspace_SOURCES): Added $(GETOPT).

Per Cederqvist's avatar
Per Cederqvist committed
530 531 532 533 534 535 536 537 538
	Port fsusage to FreeBSD.
	* m4/Makefile.am (EXTRA_DIST): Added intmax_t.m4, inttype_h.m4 and
	ulonglong.m4.
	* m4/ulonglong.m4: New file, taken verbatim from GNU fileutils-4.1.
	* m4/uintmax_t.m4: New file, taken verbatim from GNU fileutils-4.1.
	* m4/inttypes_h.m4: New file, taken verbatim from GNU fileutils-4.1.
	* configure.in: Added jm_AC_TYPE_UINTMAX_T and remove the too
	simple check for inttypes.h.

539 540 541 542 543 544 545 546 547 548 549 550 551 552 553 554 555 556 557 558 559 560 561 562 563 564 565 566 567 568 569 570 571 572 573 574 575 576 577 578 579 580 581 582 583 584 585 586 587 588 589 590 591 592 593 594 595 596 597 598 599 600 601 602 603 604 605 606 607 608
	Added the checkkomspace program.
	* doc/lyskomd.texi (Parameters): Document these:
	    Free space warning level
	    Free space warning percent
	    Free inodes warning level
	    Free inodes warning percent
	    Free space critical level
	    Free space critical percent
	    Free inodes critical level
	    Free inodes critical percent
	(Invoking checkkomspace): New node.
	* src/server/stderrlog.c: New file.
	(kom_log): Moved here from dbck.c
	(restart_kom): Ditto.
	* src/server/server-config.c (parameters): Added these parameters:
	    Free space warning level
	    Free space warning percent
	    Free inodes warning level
	    Free inodes warning percent
	    Free space critical level
	    Free space critical percent
	    Free inodes critical level
	    Free inodes critical percent
	* src/server/param.h (struct kom_par): Added
	chkspc_warn_space_abs, chkspc_warn_space_percent,
	chkspc_warn_inode_abs, chkspc_warn_inode_percent,
	chkspc_crit_space_abs, chkspc_crit_space_percent,
	chkspc_crit_inode_abs and chkspc_crit_inode_percent.
	* src/server/dbck.c (kom_log): Moved to stderrlog.c.
	(restart_kom): Ditto.
	* src/server/checkkomspace.c: New file.
	* src/server/Makefile.am (sbin_PROGRAMS): Added checkkomspace, but
	only if HAVE_FSUSAGE.
	(READ_CONFIG): Removed log.c.
	(komrunning_SOURCES): Added log.c.
	(updateLysKOM_SOURCES): Added log.c.
	(checkkomspace_SOURCES): New variable.
	(dbck_SOURCES): Added strderrlog.c.
	(splitkomdb_SOURCES): Added log.c.
	* src/server/.cvsignore: Added checkkomspace.
	* src/server/testsuite/lyskomd.0/Makefile.am (EXTRA_DIST): Added
	50.exp.
	* src/server/testsuite/lyskomd.0/50.exp: New file.  Test
	checkkomspace.

	Import fsusage, from GNU fileutils-4.1.
	* AUTHORS: Document the origin of fsusage.
	* Makefile.am (SUBDIRS): Added m4.
	(ACLOCAL_AMFLAGS): Added -I m4.
	* m4/fsusage.m4: New file, taken verbatim from GNU fileutils-4.1.
	* m4/Makefile.am: New file.
	* mkmi: Added "-I m4" to top-level aclocal.
	* src/server/fsusage.h: New file, taken verbatim from GNU
	fileutils-4.1.
	* src/server/fsusage.c: New file, taken verbatim from GNU
	fileutils-4.1.
	* configure.in: Added several tests required by fsusage:
	(AC_C_VOLATILE): Required by fsusage.
	(AC_CHECK_HEADERS): Added inttypes.h, limits.h, sys/mount.h,
	sys/vfs.h, sys/fs/s5param.h, sys/filsys.h, sys/statfs.h,
	sys/statvfs.h, sys/dustat.h, fcntl.h.
	(jm_FILE_SYSTEM_USAGE): Call it.
	(HAVE_FSUSAGE): New automake conditional.
	(AC_CONFIG_FILES): Added m4/Makefile.

	Testsuite: slow machines workaround.
	* src/server/testsuite/lyskomd.0/29.exp (startup): Increase the
	sync interval to 6 hours, so that it doesn't interfere on slow
	machines.

609 610 611 612 613 614 615 616 617 618
2003-08-31  Per Cederqvist  <ceder@ceder.dyndns.org>

	Fixed errors in the generated grammars found by Peter Zander.
	* doc/checkargs.py (generate_stable_output): Don't forget to
	convert type names for functions and asynchronous messages that
	take a single argument.  Use "%Async name" instead of
	"%Async-Name".
	(generate_summary_output): Use "%Async name" instead of
	"%Async-Name".

Per Cederqvist's avatar
Per Cederqvist committed
619 620 621 622
2003-08-30  Per Cederqvist  <ceder@ceder.dyndns.org>

	* Release 2.1.2.

623 624
2003-08-29  Per Cederqvist  <ceder@ceder.dyndns.org>

Per Cederqvist's avatar
Per Cederqvist committed
625 626 627 628 629 630 631
	Spelling fixes.
	* doc/Protocol-A.texi: Spelling errors fixed.

	Makefile fix.
	* doc/Makefile.am (DISTCLEANFILES): Added protocol-a-current.txt,
	protocol-a-recommended.txt and protocol-a-full.txt.

632 633 634 635 636 637 638 639 640 641 642 643 644 645
	Release administrativa.
	* configure.in: Set version 2.1.2.
	* versions (SERVER-VERSION): Set to 2.1.2.
	(SERVER-COMPAT-VERSION): Set to 20102.
	* doc/Protocol-A.texi (PROTOEDITION): Set to 11.1.
	(VERSION): Set to 2.1.2.
	(Document Edition History): Added an entry for 11.1.  Added
	missing "Distributed with" line for 11.0.
	(Top): Say that this was "first distributed with" a certain
	lyskomd release, not that it "corresponds to" a release.  This
	way, the text is more accurate if a bug-fix release of lyskomd is
	made without updating the Protocol-A.texi file.
	* NEWS: Updated for the 2.1.2 release.
	* README: Mention that the bug 1121 fix will write stuff to the
Per Cederqvist's avatar
Per Cederqvist committed
646 647 648 649 650
	log.  Update the year in the copyright notice.
	* scripts/lyskomd-copyrights: The top-level README file has a
	copyright statement, so don't forget to update that file.  Don't
	update the generated machine-readable grammars.  Don't update the
	data files for bug-1121.exp.
651 652 653 654 655 656 657 658 659 660 661

	Don't allow the garb thread to be starved.  Tune the garb
	parameters.  (Bug 1129).
	* src/server/text-garb.c (garb_callback): Always run the garb, so
	that it cannot be completely starved.
	* src/server/server-config.c (parameters): Changed "Garb busy
	postponement" from 20 to 50 milliseconds.  Changed "Garb timeout"
	from 100 to 0 milliseconds.
	* src/include/kom-types.h (Text_stat): Remove swedish in a comment.
	* doc/lyskomd.texi (Parameters): Updated "Garb busy postponement"
	and "Garb timeout".
662 663
	* src/server/testsuite/lyskomd.0/37.exp: Updated the expected
	timeouts for the garb tuning.
664 665 666 667
	
	Document the garb.  (Bug 194).
	* doc/Protocol-A.texi (Garb): New chapter.
	
668 669 670 671 672
	Check for a potential memory leak when killing a client.
	(Bug 149).
	* src/server/internal-connections.c (kill_client): Check that no
	aux_item_list remains.

673 674 675 676 677 678 679 680 681 682 683 684 685 686 687 688 689 690 691 692
	Don't produce overlapping ranges in the read-ranges part of a
	membership.  (Bug 1121).  Make lyskomd fix the errors introduced
	by bug 1121.
	* src/server/membership.c (insert_loc_no): Handle the case where
	adjoining ranges exists properly instead of producing overlapping
	ranges.
	* src/server/ram-parse.c (fparse_read_range_0): Handle the errors
	introduced by bug 1121.  Log an error message if any damage was
	repaired.
	* src/server/testsuite/config/unix.exp (lyskomd_start): New
	argument: db_messages.
	(dbck_run): New argument: extra_lines.
	* src/server/testsuite/lyskomd.0/Makefile.am (EXTRA_DIST): Added
	bug-1121.exp, bug-1121.data and bug-1121.texts.
	* src/server/testsuite/lyskomd.0/bug-1121.exp: Test for bug 1121:
	mark-as-read mishandles read texts in some circumstances.  Also
	test that lyskomd can repair the problems introduced by this bug.
	* src/server/testsuite/lyskomd.0/bug-1121.texts: New file.
	* src/server/testsuite/lyskomd.0/bug-1121.data: New file.

693 694
2003-08-28  Per Cederqvist  <ceder@ceder.dyndns.org>

Per Cederqvist's avatar
Per Cederqvist committed
695 696 697 698
	Reduce the noise in the log.
	* src/server/connections.c (read_from_connection): Don't report
	ETIMEDOUT errors, as they seem to be quite common.

699 700 701 702 703 704 705 706 707 708 709 710 711 712 713 714 715 716 717 718 719 720 721 722
	Generate two now forms of extracted grammars.  Publish them on the
	web.  Mention them in Protocol-A.texi.
	* doc/Protocol-A.texi (Extracted grammar): New appendix that
	discusses the extracted files.
	* doc/Makefile.am (update-www): Depend on protocol-a-full.txt, so
	that checkargs.py is run first.  Copy the generated grammars to
	the web dir.
	(protocol-a-full.txt): New target.
	* doc/checkargs.py: Generate protocol-a-recommended.txt and
	protocol-a-current.txt.
	(prot_a_type.__init__): Initialize __recommended to 0.
	(prot_a_type.use): New argument: recommended.  All callers
	updated.  Set the __recommended flag once a recommended request or
	async uses this type.
	(prot_a_builtin.use_recurse): Ditto.
	(prot_a_type.recommended): New method.
	(lexer.__parse_type): Replaced the protover arguemnt with a
	request object.  All callers updated.
	(lexer.__parse_request_arg): Ditto.
	(generate_stable_output): New argument: filename,
	only_recommended.
	(generate_summary_output): New function.
	* doc/constructs.expected: Updated.

723 724 725 726 727 728 729 730
2003-08-28  Kent Engstrm  <kent@lysator.liu.se>

        Add support for the "html" alternative of mx-refuse-import.
	* run-support/aux-items.conf: Update validation regexp. Also
	set the unique flag to false.
	* doc/Protocol-A.texi (Aux-Item Types): Document "html".
	* doc/constructs.expected: Add @code{html}.

731 732 733 734 735 736 737 738 739 740 741 742 743 744 745 746 747 748 749 750 751 752 753 754 755 756 757 758 759 760 761 762 763 764 765 766 767 768 769 770 771 772 773 774 775 776 777 778 779 780 781 782 783 784 785 786 787 788 789 790 791 792 793 794 795 796 797 798 799 800 801 802 803 804 805
2003-08-28  Per Cederqvist  <ceder@ceder.dyndns.org>

	Don't crash if a person has read more than 65535 texts after the
	first unread.
	* src/server/ram-parse.c (fparse_read_range_0): Use int, not
	short, in case a person has read many texts after the first unread
	text.

	Complete the renaming of aux.h.
	* scripts/lyskomd-copyrights: src/server/aux.h has been renamed to
	src/server/aux-no.h.

	Fix several minor errors in Protocol-A.texi.
	* doc/Protocol-A.texi (Protocol Requests): The status codes for
	who-is-on-ident and get-session-info-ident were "r" but should
	have been "O".
	(login-old): Added info about when this request became obsolete in
	the section.
	(get-person-stat-old): Ditto.
	(lookup-name): Ditto.
	(get-conf-stat-older): Ditto.
	(mark-text-old): Ditto.
	(who-is-on-old): Ditto.
	(broadcast): Ditto.
	(get-session-info): Ditto.
	(re-lookup-person): Ditto.
	(re-lookup-conf): Ditto.
	(lookup-person): Ditto.
	(lookup-conf): Ditto.
	(query-read-texts-10): Ditto.
	(get-membership-10): Ditto.
	(set-connection-time-format): Fixed typo in the @section heading.
	(Asynchronous Messages): The status codes for async-new-text-old
	was "r" but should have been "O".
	(async-new-text-old): Added async number to the @section heading.
	(async-i-am-off): Ditto.
	(async-i-am-on-obsolete): Ditto.
	(async-new-name): Ditto.
	(async-i-am-on): Ditto.
	(async-sync-db): Ditto.
	(async-leave-conf): Ditto.
	(async-login): Ditto.
	(async-broadcast): Ditto.
	(async-rejected-connection): Ditto.
	(async-send-message): Ditto.
	(async-logout): Ditto.
	(async-deleted-text): Ditto.
	(async-new-text): Ditto.
	(async-new-recipient): Ditto.
	(async-sub-recipient): Ditto.
	(async-new-membership): Ditto.
	(async-new-user-area): Ditto.
	(async-new-presentation): Ditto.
	(async-new-motd): Ditto.
	(async-text-aux-changed): Ditto.

	Generate a stable machine-readable file with the syntax of
	Protocol A. 
	* doc/checkargs.py (defined_request_names): New variable.
	(defined_request_names): New variable.
	(defined_async_names): New variable.
	(set_values): New variable.
	(tt): New variable.
	(tr): New variable.
	(rt): New variable.
	(rr): New variable.
	(at): New variable.
	(ar): New variable.
	(has_suffix): New function.
	(remove_suffix): New function.
	(prot_a_type.__init__): Initialize __protover.
	(prot_a_type.line_no): Renamed from line().
	(prot_a_type.use): New argument: protover.  All callers
	and derived methods updated.
	(prot_a_builtin.use_recurse): Ditto.
806
	(prot_a_type.protover): New method.
807 808 809 810 811 812 813 814 815 816 817 818 819 820 821 822 823 824 825 826 827 828 829 830 831 832 833 834 835 836 837 838 839 840 841 842 843 844 845 846 847
	(prot_a_simple.base_type): New method.
	(prot_a_simple.array): New method.
	(prot_a_alternate.type_a): New method.
	(prot_a_alternate.type_b): New method.
	(prot_a_struct.fields): New method.
	(prot_a_bitstring.add_field): Store the bits in the order they are
	defined.
	(prot_a_bitstring.bits): New method.
	(prot_a_selection.fields): New method.
	(prot_a_enumeration_of.base_type): New method.
	(prot_a_msg): New class.
	(prot_a_request): New class.
	(prot_a_async): New class.
	(menu): New class.
	(reader.menu_re): New constant.
	(reader.getc_eofok): If a menu line for a request or async message
	is found, store the information.
	(reader.menu): New method.
	(lexer.section_re): New method.
	(lexer.__init__): __implemented_conftypes and
	__implemented_privbits both maps to None, not a random number.
	(lexer.run): If __toplevel_at() returns a non-None value, return
	it.
	(lexer.__toplevel_at): Return the result of the called "toplevel_"
	method instead of always returning None.
	(lexer.toplevel_set): New method.
	(lexer.toplevel_section): New method.
	(lexer.toplevel_findex): Store information about the request in a
	prot_a_request instance.  Check the corresponding @section heading
	and @menu item.
	(lexer.toplevel_amindex): Ditto for asyncs.
	(lexer.__parse_async): New API.
	(lexer.toplevel_bye): Generate stable names.  Return the error
	flag instead of calling sys.exit().
	(lexer.__parse_request): New API.
	(lexer.__parse_type): New argument: protover.
	(lexer.__parse_request_arg): Ditto.  Return the parsed info.
	(lexer.__bad_arg): Fixed minor typo.
	(lexer.generate_stable_names): New method.
	(generate_stable_output): New function.

848 849 850 851
2003-08-26  Per Cederqvist  <ceder@ceder.dyndns.org>

	* Release 2.1.1.

852 853
2003-08-25  Per Cederqvist  <ceder@ceder.dyndns.org>

854 855 856 857 858 859 860 861 862
	Release administrativa.
	* versions (SERVER-VERSION): Set to 2.1.1.
	(SERVER-COMPAT-VERSION): Set to 20101.
	* configure.in: Set version 2.1.1.
	* NEWS: Updated for the 2.1.1 release.

	Fix the installation instructions.
	* README: Tell the admin to run "komrunning start" if upgrading.

863 864 865 866
	Create var/lyskomd/exportdb during install.  (Bug 1119).
	* run-support/Makefile.am (install-data-local): Create
	var/lyskomd/exportdb.

867 868 869 870 871 872 873 874
	Rename aux.h to aux-no.h.  (Bug 1116).
	* src/server/Makefile.am (aux-no.h): New name for former "aux.h".
	The name "aux.h" isn't legal on the operating systems Cygwin runs
	on top of.  All references to aux.h in this file updated.
	* src/server/.cvsignore: aux.h has been renamed aux-no.h.
	* src/server/aux-items.c: Ditto.
	* src/server/text.c: Ditto.

875 876 877 878 879 880 881 882 883 884 885 886 887 888 889 890 891 892 893 894 895 896
	Added some missing auxiliary files to the distribution.
	* src/libraries/adns/Makefile.am (EXTRA_DIST): Added changelog.
	* src/libraries/adns/client/Makefile.am (EXTRA_DIST): Added
	.cvsignore.
	* src/libraries/adns/src/Makefile.am: Ditto.
	* src/libraries/liboop/Makefile.am: Ditto.

	Added test cases for write errors.
	* src/server/testsuite/lyskomd.0/49.exp: New file, that tests
	client disconnect immediately before a write().
	* src/server/testsuite/lyskomd.0/Makefile.am (EXTRA_DIST): Added
	49.exp.
	* src/include/services.h (server_sleep): New debug request.
	* src/server/fncdef.txt: Added the debug call server_sleep.
	* src/server/debug.c (server_sleep): New request.

	Testsuite cleanup.
	* src/server/testsuite/lyskomd.0/bug-52.exp: Fixed typo in
	never-executed code.
	* src/server/testsuite/lyskomd.0/47.exp: Fixed a typo in a
	comment.

897 898 899 900
2003-08-24  Per Cederqvist  <ceder@ceder.dyndns.org>

	* Release 2.1.0.

901 902 903 904 905 906 907
2003-08-23  Per Cederqvist  <ceder@lysator.liu.se>

	Fixes for the web publication of Protocol A.
	* doc/Makefile.am (protocol-a.info): Updated the rule to what
	automake-1.7.6 would have generated.  Don't generate the info file
	in $(srcdir).

908 909
2003-08-23  Per Cederqvist  <ceder@ceder.dyndns.org>

Per Cederqvist's avatar
Per Cederqvist committed
910 911 912 913 914 915
	Set the release date.
	* HACKING: Mention that the release date should be set.
	* NEWS: Set the release date to tomorrow.
	* doc/Protocol-A.texi (Document Edition History): Set the release
	date to tomorrow.

916 917 918 919 920 921 922 923 924 925 926 927 928 929 930 931 932 933 934 935 936 937 938 939 940 941 942 943
	Update copyright headers.
	* <all files>: update the year in the copyright notice.
	* scripts/lyskomd-copyrights: Ignore public domain files:
	    doc/kom-style.el 
	    scripts/warnings.sed
	Ignore tiny files:
	    scripts/definepath
	    scripts/unprefix
	    src/server/timewrap.h
	Ignore generated files:
	    src/server/paths.h
	    doc/version.texi 
	Ignore user-supplied files:
	    src/server/testsuite/config/localcfg.exp
	Ignore binaries:
	    src/server/testsuite/get-time-often
	    src/server/testsuite/test-sigjmp
	    src/server/testsuite/timeval-overflow
	Ignore temporary files:
	    src/server/testsuite/lyskomd.0/aux-items-35.conf
	    src/server/testsuite/lyskomd.0/aux-items-46.conf
	    src/server/testsuite/lyskomd.0/aux-items-bad.conf
	Ignore run-support/savecore-lyskom, not 	
	run-support/savecore.
	
	Don't claim copyright on a collection of compiler error messages.
	* scripts/warnings.sed: Placed in the public domain.

944 945 946 947 948 949 950 951 952 953 954 955 956 957 958
	Make all requests that takes a BOOL argument return bad-bool if
	something besides 0 or 1 is sent by the client.
	* doc/Protocol-A.texi (get-membership-old): Added error code
	bad-bool.
	(login): Ditto.
	(re-z-lookup): Ditto.
	(lookup-z-name): Ditto.
	(who-is-on-dynamic): Ditto.
	(get-membership-10): Ditto.
	(query-read-texts): Ditto.
	(get-membership): Ditto.
	(set-connection-time-format): Ditto.
	(Error Codes): Document error code bad-bool.
	(Protocol Version History): Mention bad-bool.
	(Document Edition History): Mention bad-bool.
Per Cederqvist's avatar
Per Cederqvist committed
959
	* NEWS: Added info about bad-bool.
960 961 962 963 964 965 966 967 968 969 970 971 972 973 974 975 976
	* src/include/kom-errno.h (enum kom_err): Added KOM_BAD_BOOL.
	* src/server/manipulate.h (CHK_BOOL): New macro.
	* src/server/session.c (login): Call CHK_BOOL for invisible.
	(who_is_on_dynamic): Call CHK_BOOL for want_visible and
	want_invisible.
	(set_connection_time_format): Call CHK_BOOL for use_utc.  Don't
	use KOM_INDEX_OUT_OF_RANGE for that error.
	* src/server/regex-match.c (re_z_lookup): Call CHK_BOOL for
	want_persons and want_confs.
	* src/server/person.c (query_read_texts): Call CHK_BOOL for
	want_read_ranges.
	* src/server/membership.c (get_membership_old): Call CHK_BOOL for
	want_read_texts.
	(get_membership_10): Ditto.
	(get_membership): Call CHK_BOOL for want_read_ranges.
	* src/server/conference.c (lookup_z_name): Call CHK_BOOL for
	want_persons and want_confs.
Per Cederqvist's avatar
Per Cederqvist committed
977 978 979
	* src/server/testsuite/lyskomd.0/01.exp: Expect bad-bool, not
	index-out-of-range, from set-connection-time-format(2).
	* src/server/testsuite/lyskomd.0/03.exp: Ditto.
980 981
	* src/server/testsuite/lyskomd.0/48.exp: New file: Test the
	bad-bool error code.
982 983
	* src/server/testsuite/lyskomd.0/Makefile.am (EXTRA_DIST): Added
	48.exp.
984 985 986 987 988 989 990 991 992 993 994 995 996 997

	Make statement-like macros more robust.
	* src/server/manipulate.h (CHK_CONNECTION): Use the "do ... while
	(0)" construct to make this macro expand to exactly one statement.
	(CHK_LOGIN): Ditto.
	(CONF_ZERO): Ditto.
	(CHK_EXIST): Ditto.
	(GET_P_STAT): Ditto.
	(VOID_GET_P_STAT): Ditto.
	(GET_C_STAT): Ditto.
	(VOID_GET_C_STAT): Ditto.
	(GET_T_STAT): Ditto.
	(VOID_GET_T_STAT): Ditto.

998 999 1000 1001 1002 1003 1004 1005 1006 1007 1008 1009 1010 1011 1012 1013 1014 1015 1016 1017 1018 1019 1020 1021 1022 1023 1024 1025 1026 1027
	"make distcheck" lint.
	* src/server/testsuite/lyskomd.0/Makefile.am (MOSTLYCLEANFILES):
	Added aux-items-35.conf and aux-items-46.conf.
	* doc/Makefile.am (MOSTLYCLEANFILES): Added Protocol-A.dvi.
	* src/server/testsuite/lyskomd.0/35.exp: Store the temporary
	aux-item file in lyskomd.0/aux-items-35.conf, instead of
	35-aux.conf, just like other test files already do.
	* src/server/testsuite/.cvsignore: Don't ignore 35-aux.conf.
	* src/server/testsuite/lyskomd.0/.cvsignore: Ignore aux-items-35.conf.
	* src/server/testsuite/lyskomd.0/46.exp: Remove the temporary file
	before copying files to it, to avoid permission problems if the
	files in the source tree has read-only permissions.

	Updated the copyright-updating scripts.
	* scripts/update-copyright: Updated copyright heading.
	* scripts/lyskomd-copyrights: Updated copyright heading.  Don't
	update copyright headers of adns or liboop.

	Update version numbers.
	* HACKING: Updated where version numbers are found.
	* configure.in: Set version 2.1.0.
	* versions (PROTOCOL-A-LEVEL): 11.
	(SERVER-VERSION): 2.1.0.
	(SERVER-COMPAT-VERSION): 20100.
	* doc/Protocol-A.texi (PROTOEDITION): Set to 11.0.
	(PROTOVER): Set to 11.
	(VERSION): Set to 2.1.0.
	(Client-Server Dialog): Talk about "version 10 or newer", not
	only "version 10".

1028
	Release administrativa.
Per Cederqvist's avatar
Per Cederqvist committed
1029
	* NEWS: Updated for the 2.1.0 release.
1030 1031 1032 1033 1034 1035 1036 1037 1038 1039 1040 1041 1042 1043 1044 1045 1046
	* HACKING: Mention that our automake patch is obsolete.
	* README: Some minor errors fixed.  Ask the reader to vote for the
	bugs he considers important.

	Update the protocol and document history of Protocol A.
	* doc/Protocol-A.texi (Future changes): Don't mention bugs that
	are closed.  Mention that the list of included bugs is
	incomplete.
	(Protocol Version History): Document version 11.
	(Document Edition History): Document edition 11.0.
	* doc/constructs.expected: Updated.

	Revert the Xenofarm automount workaround.
	* src/server/connections.c (dump_connections): Revert the
	automount workaround attempt.  It didn't work, and it also
	introduced an endless loop.

1047 1048
2003-08-22  Per Cederqvist  <ceder@ceder.dyndns.org>

1049 1050 1051 1052 1053 1054 1055 1056 1057 1058 1059 1060 1061 1062 1063 1064 1065 1066 1067 1068 1069 1070 1071 1072 1073 1074 1075 1076
	Keep track of clients that are beind killed, and don't re-add them
	to the kill list.  (Bug 1100).
	* src/server/internal-connections.h (get_conn_by_number): Document
	that it returns NULL if the session doesn't exist.
	* src/server/internal-connections.c (init_connection): Set the new
	kill_status field of Connection instead of the old kill_pending
	field.
	(new_client): Ditto.
	(check_conn_exists): Dead code removed.
	(get_conn_by_number): Don't crash if no connections exist.  It
	isn't abnormal if the supplied connection number doesn't exist, so
	remove code inside NDEFENSIVE_CHECKS that assumed it was.
	Whitespace fixes.
	* src/server/connections.h (enum kill_state): New enum.
	(Connection): Replaced the "Bool kill_pending" field with a "enum
	kill_stat kill_status" field.
	* src/server/connections.c (check_kill_flg): Use the new
	kill_status field of Connection instead of the old kill_pending
	field.
	(read_from_connection): Ditto.
	(add_to_kill_list): Ditto.  Don't add the client if it is dying.
	
	Update var/lyskomd.clients when the handshake completes by a DNS
	response.
	* src/server/connections.c (dns_resolution): If the DNS resolution
	completed the handshake, dump out the connections again to reflect
	that fact.

Per Cederqvist's avatar
Per Cederqvist committed
1077 1078 1079 1080 1081 1082
	Test for bug 1100.
	* src/server/testsuite/lyskomd.0/47.exp: New file: Test client
	disconnect while logged in and receiving async-logout.
	* src/server/testsuite/lyskomd.0/Makefile.am (EXTRA_DIST): Added
	47.exp.

1083 1084 1085 1086 1087 1088
	Xenofarm automount workaround.
	* src/server/connections.c (dump_connections): If the rename of
	the file fails with ENOENT, retry it a few times before giving
	up.  This is an attempt to work around an automount problem on the
	Xenofarm at Lysator.

1089 1090 1091 1092 1093 1094 1095 1096
	Fixed an error in the new function l2gi_prev().
	* src/server/local-to-global.c (l2gi_prev): Don't read memory
	before the start of an array.

	Make dump_connections() static.
	* src/server/internal-connections.h (dump_connections): Removed.
	* src/server/connections.c (dump_connections): Now static.

1097 1098 1099 1100 1101 1102 1103 1104 1105 1106 1107 1108 1109 1110 1111 1112 1113 1114 1115 1116 1117 1118 1119 1120 1121 1122 1123 1124 1125 1126 1127 1128 1129 1130 1131 1132 1133 1134 1135 1136 1137 1138 1139 1140 1141 1142 1143 1144 1145 1146 1147
	New requests: local-to-global-reverse and
	map-created-texts-reverse.  (Bug 1092).
	* doc/Protocol-A.texi (Mapping Local to Global Text Numbers):
	Renamed the field "later-texts-exists" in "Text-Mapping" to
	"more-texts-exists", so that it is useful for the new requests
	defined below.  Updated the descriptions of some fields for the
	same purpose.
	(local-to-global): Added a link to local-to-global-reverse.
	(map-created-texts): Added a link to map-created-texts-reverse.
	(set-connection-time-format): Added a missing protocol number.
	(local-to-global-reverse): New request.
	(map-created-texts-reverse): New request.
	* src/server/fncdef.txt: Added local_to_global_reverse and
	map_created_texts_reverse.
	* src/include/services.h (map_created_texts_reverse): New
	request.
	(local_to_global_reverse): New request.
	* src/server/text.c (local_to_global_reverse): New function.
	* src/server/person.c (map_created_texts_reverse): New request.
	* src/include/kom-types.h (Text_mapping_reverse): New struct.
	* src/server/prot-a.c (prot_a_reply): Handle
	rt_text_mapping_reverse.
	* src/server/prot-a-output.h, src/server/prot-a-output.c
	(prot_a_output_text_mapping_reverse): New function.
	* src/server/connections.h (enum res_type): Added
	rt_text_mapping_reverse.
	(union result_holder): Added text_mapping_reverse.
	* src/server/testsuite/renumber.el
	(renumber-lyskom-send-simple-expect-indented): New function.
	* src/server/testsuite/lyskomd.0/00.exp: Added test cases for
	local-to-global-reverse and map-created-texts-reverse.
	* src/server/testsuite/lyskomd.0/01.exp: Ditto.
	* src/server/testsuite/lyskomd.0/03.exp: Ditto.

	Added support for reverse local-to-global iterators.
	* src/include/kom-types.h (L2g_reverse_iterator): New struct.
	* src/server/local-to-global.h (l2gi_searchsome_reverse): New
	function.
	(l2gi_prev): New function.
	* src/server/Makefile.am (check-l2g): New target.
	* src/server/local-to-global.c (find_block_index_key_reverse): New
	static function.
	(l2gi_searchsome_reverse): New function.
	(l2gi_prev): New function.
	* src/server/testsuite/test-l2g.c (main): Handle command "b", for
	testing backwards iterators.  ("r", "R", and "I" were all taken).
	* src/server/testsuite/l2g.0/07.exp: Added test cases for reverse
	iterators.
	* src/server/testsuite/l2g.0/10.exp: Ditto.
	* src/server/testsuite/l2g.0/11.exp: Ditto.

1148 1149
2003-08-20  Per Cederqvist  <ceder@ceder.dyndns.org>

Per Cederqvist's avatar
Per Cederqvist committed
1150 1151 1152 1153 1154 1155 1156 1157
	Code cleanup.  (Bug 1097).
	* src/server/aux-items.c (find_linked_aux_item_list): Removed all
	traces of PERS_OBJECT_TYPE.
	(mark_linked_object_as_changed): Ditto.
	* src/include/kom-types.h (enum object_type): Removed all traces
	of PERS_OBJECT_TYPE.  Don't explicitly specify the values of the
	constants, since they don't matter.

1158 1159 1160 1161 1162 1163 1164 1165 1166 1167 1168 1169
	Fix output of linefeed in Protocol A examples.  (Bug 376).
	* doc/Protocol-A.texi (Connecting to the Server): Say "linefeed",
	not "newline" or "line feed".
	(create-text): Ditto.
	(Reformattable Text (text/x-kom-basic)): Ditto.
	(The User Area): Ditto.
	(Text formatting): Ditto.
	(Protocol Version History): Ditto.
	(get-text): Use "@bullet{}" to represent linefeed characters
	inside hollerith strings.
	(create-text-old): Ditto.
	(create-anonymous-text-old): Ditto.
Per Cederqvist's avatar
Per Cederqvist committed
1170
	* doc/constructs.expected: Updated.
1171

1172 1173 1174 1175 1176 1177 1178 1179 1180 1181 1182 1183 1184 1185 1186 1187 1188 1189 1190 1191 1192 1193 1194 1195 1196 1197 1198 1199 1200 1201 1202
	Return the error code long-array instead of a protocol error if
	a too long array is sent to mark-as-read.  (Bug 836).  Get rid of
	c_local_text_no_p -- use Number_list instead.
	* src/server/fncdef.txt (mark_as_read): Changed the argument from
	"num num c_local_text_no_p" to the more modern "num num_list".
	* src/include/services.h (mark_as_read): Changed the arguments
	"int no_of_text; const Local_text_no *text_arr" into a single more
	modern-style Number_list argument.
	* src/server/membership.c (mark_as_read): Changed the arguments
	"int no_of_text; const Local_text_no *text_arr" into a single more
	modern-style Number_list argument.  Return KOM_LONG_ARRAY if the
	array is too long.  Simplified the code.  
	(set_read_ranges): Call CHK_CONNECTION before checking the
	supplied read_ranges.
	* src/server/prot-a.c (prot_a_init): Removed
	handling of c_local_text_no_p.
	(prot_a_destruct): Ditto.
	* src/server/prot-a-parse-arg-c.awk: Removed
	handling of c_local_text_no_p.
	* src/server/internal-connections.c (init_connection): Removed
	handling of c_local_text_no_p.
	(kill_client): Ditto.  But do check for left-over data in
	num_list.
	* src/server/connections.h (Connection): Removed
	c_local_text_no_p.
	* src/server/connections.c (free_parsed): Removed handling of
	c_local_text_no_p.
	* src/server/call-switch.awk: Removed handling of
	c_local_text_no_p.
	* src/server/testsuite/lyskomd.0/29.exp: Don't expect bug 836.

1203 1204 1205 1206 1207 1208 1209 1210 1211 1212 1213 1214 1215 1216
	Send mail using sendmail, to avoid malformed mails.  (Bug 185).
	Make the path to sendmail configurable.  Make it possible to turn
	off mail delivery completely.
	* configure.in: Search for a sendmail binary.
	* doc/lyskomd.texi (Parameters): Document "sendmail path".
	* src/server/param.h (struct kom_par): Added sendmail_path.
	* src/server/server-config.c (parameters): Added "sendmail path".
	(read_configuration): Handle it.
	* src/server/updateLysKOM.c (checkstatus): Send mail by opening a
	pipe to "sendmail -t".  That is more portable than pipeing a
	message into "mail".  Turn off mail delivery completely if
	"sendmail path" is set to ":".
	* src/server/Makefile.am (paths.h): Write SENDMAIL_PATH.

1217 1218 1219 1220
	Test suite: Don't forget to distribute aux-items-bad.conf.
	* src/server/testsuite/lyskomd.0/Makefile.am (EXTRA_DIST): Added
	aux-items-bad.conf.

1221 1222 1223 1224 1225 1226 1227 1228 1229 1230 1231 1232 1233 1234 1235 1236 1237 1238 1239 1240 1241
	Fix text_read_access() so that it uses the privileges of the
	supplied connection, and never uses active_connection. (Bug 178).
	Fixing this also required me to remove the ability to specify an
	faq-text for a freshly created person on a session where nobody is
	logged on; this could be considered a security fix.
	* src/server/text.c (person_text_read_access): Removed.  Integrate
	the functionality inteo text_read_access.
	(text_read_access): Use the supplied Connection for all checks.
	Don't use active_connection, ENA, ACTPERS or ACT_P.  Fail and log
	an error message if the Connection is NULL.  Use is_supervisor()
	instead of is_supervisor_2().
	* src/server/manipulate.h (is_supervisor_2): Removed.
	(person_text_read_access): Removed.
	* src/server/conference.c (is_supervisor_2): Removed.
	* src/server/aux-items.c (aux_item_validate_existing_text): Don't
	allow access to texts if you are not logged in.  This change means
	that create-person cannot accept aux-items with the
	existing-readable-text validator if nobody is logged in.  I think
	this is more reasonable than the number of special cases in the
	code that were needed to support the old functionality.
	* src/server/testsuite/lyskomd.0/40.exp: Don't expect bug 178.
1242
	* src/server/testsuite/lyskomd.0/41.exp: Don't expect bug 178.
1243 1244 1245 1246 1247
	* src/server/testsuite/lyskomd.0/03.exp: Don't expect to be able
	to create a person with an faq-text.  Add the faq-text using a
	modify-conf-info request instead.  Test that creation of a new
	person with a faq-text item really fails.

1248 1249 1250 1251 1252 1253 1254 1255 1256 1257 1258 1259 1260 1261 1262 1263 1264 1265 1266 1267 1268 1269 1270 1271 1272 1273 1274 1275 1276 1277 1278 1279 1280 1281 1282 1283 1284 1285 1286 1287 1288 1289 1290 1291 1292 1293 1294 1295 1296 1297 1298 1299 1300 1301 1302 1303 1304 1305 1306 1307 1308 1309 1310 1311 1312 1313 1314 1315 1316 1317 1318 1319 1320 1321 1322 1323 1324 1325 1326 1327 1328 1329 1330 1331 1332 1333 1334 1335 1336 1337 1338 1339 1340 1341 1342 1343 1344 1345 1346 1347 1348 1349 1350 1351 1352 1353 1354 1355 1356 1357 1358 1359 1360 1361 1362 1363 1364 1365 1366 1367 1368
	Keep track of the connection that is creating an aux-item, so that
	ENA_C() and text_read_access() can be used.  Get rid of broken
	concept "owner", and introduced "subordinate" instead.  (Bug 334).
	Some other aux-item-related code cleanup.
	* src/server/aux-items.h (Aux_item_validation_data): Removed
	item_creator, object_creator and owner_check.  Added
	creating_conn and subordinate.  Placed add_to_list,
	start_looking_at and object_type inside #if 0..#endif, since they
	are not currently used.
	(aux_inherit_items):Removed arguments target_creator, creating
	and object_type.  Added argument subordinate.  Changed type of
	arguments object_no and object, since we know that they are a
	Text_no and Text_stat, respectively.
	(check_delete_aux_item_list): Renamed the "owner" argument
	"subordinate".
	(text_stat_check_add_aux_item): Removed the item_creator and
	creating arguments.  Added the creating_conn argument.
	(conf_stat_check_add_aux_item_list): Ditto.
	(system_check_add_aux_item_list): Ditto.
	(conf_stat_check_add_aux_item): Declaration for undefined and
	unused function removed.
	(conf_stat_add_aux_item): Ditto.
	* src/server/aux-items.c (aux_item_validate): Return Success, not
	Bool.  All callers updated.
	(aux_item_add_perm): Removed the arguments item_creator,
	object_creator and owner_check.  Added the arguments creating_conn
	and subordinate.  All callers updated.  Simplified the code.  Use
	is_supervisor() instead of is_strictly_supervisor() where
	appropriate.  Use the supplied connection instead of checking that
	ACTPERS is the item_creator.  Adjusted for the new contents of
	Aux_item_validation_data.
	(aux_inherit_items): Removed arguments target_creator, creating
	and object_type.  Added argument subordinate.  Changed type of
	arguments object_no and object, since we know that they are a
	Text_no and Text_stat, respectively.
	(filter_aux_item_list): Assert that the viewer_conn argument is
	non-NULL.
	(check_delete_aux_item_list): Renamed the "owner" argument
	"subordinate".
	(text_stat_check_add_aux_item_list): Removed the item_creator and
	creating arguments.  Added the creating_conn argument, and assert
	it is non-NULL.
	(conf_stat_check_add_aux_item_list): Replaced the "creator"
	argument with "creating_conn".  Argument assertions added.
	(system_check_add_aux_item_list): Ditto.
	(aux_item_validate_existing_text): Changed return type from Bool
	to Success.  Use person_text_read_access() only when necessary,
	and add a comment explaining why it might happen.
	* src/server/text.c (create_text_add_aux): Adjusted to new API of
	aux_inherit_items().
	(do_create_text): Adjusted to new API of
	text_stat_check_add_aux_item_list().
	(modify_text_info): Ditto.
	* src/server/person.c (create_person_generic): Adjusted to new API
	of conf_stat_check_add_aux_item_list().
	* src/server/manipulate.h (text_read_access): Docstring updated.
	(person_text_read_access): Docstring added.
	* src/server/conference.c (do_create_conf): New argument:
	creating_connection.  All callers updated.
	(modify_conf_info): Adjusted to new API of
	check_delete_aux_item_list() and
	conf_stat_check_add_aux_item_list().
	* src/server/admin.c (modify_system_info): Adjusted to new API of
	system_check_add_aux_item_list().
	* src/server/testsuite/lyskomd.0/24.exp: Use good_bad_expect
	instead of a "if {0}" construct.  Don't expect it to fail.  Fixed
	a broken test outcome.

	Added code to diagnose Success vs. Bool mixups.
	* HACKING (Release generation): Use TYPE_CHECK_COMPILATION for
	increased quality assurance.
	* src/include/misc-types.h (TYPE_CHECK_COMPILATION): New
	compilation switch.
	(SUCCESS_AS_PTR): Ditto.
	(Bool): Provide a pointer-based non-working implementation, that
	can be used for compile-time checks, but not for running code.
	(Success): Provide a pointer-based implementation.  Switch to
	using it, at least for now; this might give us good warnings from
	the compilers.
	* src/server/misc-types.c: New file.  Implementation of the types
	in misc-type.h.
	* src/server/Makefile.am (libcheck_a_SOURCES): Added
	misc-types.c.
	(lyskomd_SOURCES): Ditto.
	(komrunning_SOURCES): Ditto.
	(updateLysKOM_SOURCES): Ditto.
	(dbck_SOURCES): Ditto.
	(splitkomdb_SOURCES): Ditto.
	* src/server/testsuite/Makefile.am (test_l2g_LDADD): libcheck.a
	must be linked twice.
	(get_time_often_LDADD): Ditto.

	Make etags find our documentation.
	* Makefile.am (AM_ETAGSFLAGS): Use --declarations, so that the
	tags command finds the documentation.

	Include an Emacs indentation style for our code.
	* doc/kom-style.el ("KOM"): New C style for c-add-style.  This
	sets up proper indentation for Emacs.
	* doc/Makefile.am (EXTRA_DIST): Added kom-style.el.

	Added more hacking documentation.
	* doc/lyskomd.texi (The Database): Mention that untranslated
	Swedish text exists.
	(local-to-global): Ditto.
	(Modifying Stored Types): Mention that dbck needs to be updated.
	(Coding conventions): New section.

	Document that send async-deleted-text, async-new-text,
	async-new-text-old, async-new-recipient and async-sub-recipient
	are sent to recipients of text linked to the relevant text.
	The functionality was added 2003-01-13.  (Bug 59).
	* doc/Protocol-A.texi (Aux-Item Inheritance): Added a missing
	parenthesis.
	(async-new-text-old): Document that recipients of linked texts
	also gets this message.
	(async-deleted-text): Ditto.
	(async-new-text): Ditto.
	(async-new-recipient): Ditto.
	(async-sub-recipient): Ditto.

1369 1370
2003-08-19  Per Cederqvist  <ceder@ceder.dyndns.org>

1371 1372 1373 1374
	Test permissons for setting the canonical-name aux-item.
	* src/server/testsuite/lyskomd.0/45.exp: Test that only an enabled
	admin can set a canonical-name aux-item on the system.

1375 1376 1377 1378 1379 1380 1381 1382
	Improve aux-item documentation.
	* doc/Protocol-A.texi (About Aux-Items): Reduce the amount of text
	talking about aux-items on persons.
	(Aux-Item Inheritance): New node.
	* doc/lyskomd.texi (Aux-Item Definition File): Document that
	author-only and supervisor-only has no effect on items placed on
	the system.

1383 1384 1385 1386 1387 1388 1389 1390 1391 1392 1393 1394 1395
	Speed up the test suite slightly.
	* src/server/testsuite/lyskomd.0/gen-19.py (DEBUG): Turn it off.

	Improve documentation on set-connection-time-format, and make it
	recommended.
	* doc/Protocol-A.texi (Common Types): Mention that times can be
	expressed in UTC if the set-connection-time-format request has
	been used.
	(get-last-text): Ditto.
	(set-connection-time-format): Change the status from experimental
	to recommended.
	* doc/constructs.expected: Updated.

1396 1397 1398 1399 1400 1401 1402 1403 1404 1405 1406 1407 1408 1409 1410 1411 1412 1413 1414 1415
	Fixed a crash in setup_timer().
	* src/libraries/libmisc/timeval-util.c (setup_timer): Fixed a
	fence error that triggered an assertion with a probability of one
	in a million.

	Warn about using SIGWINCH to read a new aux-item file.  If there
	is a syntax error, the server will abort.
	* doc/lyskomd.texi (Parameters): Warn about Bug 1095.
	(Signals): Ditto.

	Test parsing of broken aux-items at runtime.
	* src/server/testsuite/lyskomd.0/46.exp: New file: Test parsing a
	broken aux-item file as a result of SIGWINCH.
	* src/server/testsuite/lyskomd.0/Makefile.am (EXTRA_DIST): Added
	46.exp.
	* src/server/testsuite/config/unix.exp (lyskomd_death): New
	reason: "restart_kom".
	* src/server/testsuite/lyskomd.0/aux-items-bad.conf: New file with
	syntax errors.

1416 1417 1418 1419 1420
2003-08-18  Per Cederqvist  <ceder@ceder.dyndns.org>

	Write test case for aux-item inheritance.
	* src/server/testsuite/lyskomd.0/Makefile.am (EXTRA_DIST): Added
	45.exp.
1421 1422
	* src/server/testsuite/lyskomd.0/45.exp: New file: Test aux-item
	inheritance.
1423

1424 1425
2003-08-17  Per Cederqvist  <ceder@ceder.dyndns.org>

1426 1427 1428 1429 1430 1431 1432 1433 1434 1435 1436 1437 1438 1439 1440 1441
	Simplified the statistics code.  Don't crash if the time moves
	backwards, but log a nice error message including how much the
	time moved.
	* src/server/connections.c (set_time): Log how much the time moves
	backwards when it moves backwards.
	* src/server/stats.h (check_all_stats): Removed. 
	* src/server/stats.c: Don't attempt to keep a running average.
	Rounding errors are far too common.  Recompute it whenever it is
	requested instead.  This results in simpler code as well.
	(dump_stats): Removed.
	(update_one_stat): Check if the time moves backward, to avoid
	triggering an assertion.
	(is_nonzero): Removed.
	(check_all_stats): Removed.
	* src/server/ramkomd.c (main): Removed calls to check_all_stats().

1442 1443 1444 1445 1446 1447 1448 1449 1450 1451 1452 1453 1454 1455 1456 1457 1458 1459 1460 1461 1462 1463 1464 1465 1466 1467 1468 1469 1470 1471 1472 1473 1474 1475 1476 1477 1478 1479 1480 1481 1482 1483 1484 1485 1486 1487 1488 1489 1490 1491 1492 1493 1494 1495 1496 1497 1498 1499 1500 1501 1502 1503 1504 1505 1506 1507 1508 1509 1510 1511 1512 1513 1514 1515 1516 1517
	Simplify handling of the sent_by misc-info.  Allow the supervisor
	of the author and sender to modify or remove a recipient (Bug 966,
	Bug 181 and Bug 1089).  Don't allow a text to be both a footnote
	and a comment of the same text at the same time (Bug 1090).  Code
	cleanup.
	* doc/Protocol-A.texi (add-comment): Added error codes
	already-comment and already-footnote.
	(add-footnote): Added error codes already-comment.
	* src/server/text.c (find_textlink): New static function.
	(is_supervisor_of_sender): New static function.
	(is_sender): Removed.
	(is_comm_sender): Removed.
	(is_comment_to): Removed.
	(recp_sent_by): Removed.
	(is_footnote_to): Removed.
	(sender): Removed.
	(do_add_footnote): New arguments: the Text_stat objects.  All
	callers updated.
	(do_add_comment): Ditto.
	(do_add_recipient): The text_s argument may no longer be NULL.
	All callers updated.
	(skip_recp): New simplified API.
	(filter_secret_info): Use is_supervisor_of_sender() instead of
	recp_sent_by().  Use the new API of is_supervisor().
	(person_text_read_access): Use is_supervisor_2() instead of
	is_supervisor() until bug 178 is fixed.
	(create_text_add_miscs): New argument: new_stat.  Pass it to
	do_add_footnote(), do_add_comment() and do_add_recipient() for a
	slight performance gain.  All callers updated.
	(delete_text): Use new API of is_supervisor().
	(add_recipient): Allow the supervisor of the author to modify the
	recipient (fixes bug 966).  Allow the supervisor of the sender to
	modify the recipient (fixes bug 181).
	(sub_recipient): Use the new API of is_supervisor().  Allow the
	supervisor of the sender to remove the recipient (fixes bug
	1089).
	(check_add_textlink): New helper function with the common parts of
	add_comment() and add_footnote().  Don't allow a text to be both a
	footnote and a comment of the same text at the same time (fixes
	bug 1090).
	(add_comment): Move code to check_add_textlink().
	(sub_comment): Use find_textlink() instead of is_comment_to().
	Use new API of is_supervisor().  Allow the supervisor of the
	sender to remove the comment link.
	(add_footnote): Use check_add_textlink() to simplify the code.
	(sub_footnote): Use find_textlink() instead of is_footnote_to().
	Use new API of is_supervisor().
	* src/server/session.c (login_old): Use new API of
	is_supervisor().  Simplify and reindent code.
	(login): Ditto.
	(disconnect): Ditto.
	* src/server/person.c (set_passwd): Use new API of
	is_supervisor().  Simplify and reindent code.
	* src/server/membership.c (access_perm_helper): Use new API of
	is_supervisor().  Simplify and reindent code.
	(sub_member): Ditto.
	(add_member_common): Ditto.
	(do_get_members): Ditto.
	* src/server/conference.c (is_supervisor): Replaced the Pers_no
	and Person arguments with a Connection argument.
	(is_supervisor_2): New function, with the old API of
	is_supervisor().
	(set_permitted_submitters): Use new API of is_supervisor().
	(set_super_conf): Ditto.
	* src/server/aux-items.c (filter_aux_item_list): Use new API of
	is_supervisor().
	(check_delete_aux_item_list): Ditto.
	* src/server/manipulate.h (is_supervisor): Replaced the Pers_no
	and Person arguments with a Connection argument.
	(is_supervisor_2): New function, with the old API of
	is_supervisor().  This is only used in one place, and will be
	removed when bug 178 is fixed.
	* src/server/testsuite/lyskomd.0/44.exp: Bugs 966, 181, 1089 and
	1090 are now fixed.
	* src/server/testsuite/lyskomd.0/14.exp: Bug 181 is now fixed.

1518 1519 1520 1521 1522 1523 1524
	Added test cases for bug 1090.
	* src/server/testsuite/lyskomd.0/44.exp: Added test cases for bug
	1090.
	* src/server/testsuite/config/unix.exp (good_bad_expect): Handle
	the case where the bad result contains only a success indicator
	and a refno (such as "=1032").

1525 1526 1527 1528 1529 1530 1531 1532 1533 1534
	Allow recipients to be converted in a few more cases.
	* doc/Protocol-A.texi (add-recipient): The permission-denied error
	code is given if you are not supervisor of the author, recipient
	or sender.  The condition used to be more restrictive.

	Test suite fixes.
	* src/server/testsuite/lyskomd.0/14.exp: Simplified the code by
	using good_bad_expect.
	* src/server/testsuite/lyskomd.0/41.exp: Fixed typo in a comment.

Per Cederqvist's avatar
Per Cederqvist committed
1535
	Added test cases for bug 181, bug 966 and bug 1089.
1536 1537
	* src/server/testsuite/lyskomd.0/44.exp: New file, that tests
	conversion of recipients from one type to another.
Per Cederqvist's avatar
Per Cederqvist committed
1538 1539
	* src/server/testsuite/lyskomd.0/Makefile.am (EXTRA_DIST): Added
	44.exp.
1540 1541 1542 1543 1544

	Fixed dist error introduced yesterday.
	* scripts/Makefile.am (EXTRA_DIST): Dont forget to distribute the
	$(noinst_SCRIPTS).

1545 1546
2003-08-16  Per Cederqvist  <ceder@ceder.dyndns.org>

1547 1548 1549 1550 1551 1552 1553 1554 1555 1556 1557 1558 1559 1560 1561 1562 1563 1564 1565 1566 1567 1568 1569 1570 1571 1572 1573 1574
	Send and receive times in UTC, if the clients requests it.  
	(Bug 957).
	* doc/Protocol-A.texi (set-connection-time-format): New request.
	* src/include/services.h (set_connection_time_format): New
	request.
	* src/server/connections.h (Connection): Added use_utc.
	* src/server/fncdef.txt: Added set_connection_time_format.
	* src/server/text.c (get_last_text): Handle use_utc.
	* src/server/session.c (set_connection_time_format): New
	function.
	* src/server/prot-a-output.c (prot_a_output_time): Handle use_utc.
	* src/server/internal-connections.c (init_connection): Initialize
	use_utc.
	(new_client): Ditto.
	* src/server/prot-a-parse.c (prot_a_parse_misc_info): This code
	used to convert the broken-down time to a time_t using mktime().
	The value was never used.  Instead of introducing a portability
	problem using by calling timelocal(), we now simply store a 0 in
	the time field of rec_time and sent_at misc-infos.
	* src/server/testsuite/lyskomd.0/01.exp: Test
	set_connection_time_format.
	* src/server/testsuite/lyskomd.0/03.exp: Ditto.
	
	Installation fixes.
	* README: Fixed minor error in upgrade instructions.
	* run-support/Makefile.am (install-data-local): Create var/run,
	which will hold the pid file.

1575 1576 1577 1578 1579 1580 1581 1582 1583 1584 1585 1586 1587 1588 1589 1590 1591 1592 1593
	New async: async-text-aux-changed.  (Bug 910).
	* doc/Protocol-A.texi (Client-Specific Aux-Item Types): Reserve
	10200-10299 for private test use.  Refer to Bugzilla.
	(async-text-aux-changed): New async message.
	* src/server/async.h (enum async): Added ay_text_aux_changed.
	* src/server/prot-a-send-async.h, src/server/prot-a-send-async.c
	(prot_a_async_text_aux_changed): New function.
	* src/server/send-async.c, src/server/send-async.h
	(async_text_aux_changed): New function.
	* src/server/session.c (accept_async): Handle ay_text_aux_changed.
	* src/server/text.c (send_async_text_aux_changed): New function.
	(modify_text_info): Call it.
	* src/server/testsuite/lyskomd.0/03.exp: Handle
	async-text-aux-changed.
	* src/server/testsuite/lyskomd.0/Makefile.am (EXTRA_DIST): Added
	43.exp.
	* src/server/testsuite/lyskomd.0/43.exp: New file.  Test
	async-text-aux-changed.

Per Cederqvist's avatar
Per Cederqvist committed
1594
	Code cleanup.
1595 1596 1597 1598 1599 1600 1601 1602 1603
	* src/server/aux-items.c: Fixed indentation of bool operators.
	(commit_aux_item_list_internal): Removed a redundant "continue".
	(delete_aux_item_list): Fixed cut-n-paste errors in comments.
	(aux_item_list_add_items): Simplified the code.
	* src/server/prot-a-parse.c (prot_a_parse_misc_info_list): Fixed
	a cut-n-paste error: wrong function name in log message.
	* src/server/testsuite/lyskomd.0/00.exp: Removed the "test"
	argument to one of the simple_expect, to help diagnose problems
	where 127.0.0.1 isn't properly resolved.
Per Cederqvist's avatar
Per Cederqvist committed
1604

Per Cederqvist's avatar
Per Cederqvist committed
1605 1606 1607 1608
	The man-pages will stay.
	* HACKING: It is good to have man-pages that refer to the real
	documentation, so don't remove them.  (Bug 77).

1609 1610 1611 1612 1613
	Refer to Bugzilla, not email.  (Bug 724).
	* All files: Change the phrase "Please mail bug reports to
	bug-lyskom@lysator.liu.se." to "Please report bugs at
	http://bugzilla.lysator.liu.se/."

1614 1615 1616 1617 1618 1619 1620 1621 1622 1623 1624 1625 1626 1627 1628 1629 1630 1631 1632 1633 1634 1635 1636 1637 1638 1639 1640 1641 1642 1643 1644 1645 1646 1647 1648 1649 1650 1651 1652 1653 1654 1655 1656 1657 1658 1659 1660 1661 1662 1663 1664 1665 1666 1667 1668 1669 1670 1671 1672 1673 1674 1675 1676 1677 1678 1679 1680 1681 1682
	Make config files refer to their documentation.
	* run-support/aux-items.conf: Added a reference to the
	documentation.
	* run-support/config: Added a reference to the documentation.

	Use better file names, so that lyskomd can be installed in /usr or
	/usr/local.  (Bug 66 and Bug 132).
	* NEWS: Documented all changed path names.
	* README: Removed the warning not to use /usr/local as the
	prefix.  Document how an upgrade should be made.  Document what
	needs to be done when upgrading from pre-2.1.0 to 2.1.0.  The
	binaries are now in "sbin", not "bin".
	* Makefile.am (SUBDIRS): Moved scripts before doc.
	* src/server/server-config.c (assign_path): New define for
	assign_string.
	(unassign_path): New define for unassign_string.
	(parameters): Use assign_path and unassign_path for all path
	parameters, to make them easier to find.  Use default values from
	paths.h instead of hardcoding the defaults in the file.  A lot of
	values have changed; see the summary in the NEWS file.
	(CONFIG_FILE): Removed.  This is now a define in paths.h.
	(compiled_config_file): New constant.
	(get_default_config_file_name): Use compiled_config_file instead
	of CONFIG_FILE.
	(free_default_config_file_name): Ditto.
	* src/server/connections.h: Updated a comment.
	* src/server/Makefile.am (MOSTLYCLEANFILES): Added paths.h.
	(AM_CPPFLAGS): Removed the DEFAULT_PREFIX define.  Use paths.h
	instead.
	(sbin_PROGRAMS): Was: bin_PROGRAMS.
	(BUILT_SOURCES): Added paths.h.
	(.gdbinit): Removed redundant "-f" option to $(RM).
	(DEFP): New constant.
	(paths.h): New target.
	(server-config.o): Added explicit dependency on paths.h.
	* scripts/unprefix: New script.
	* scripts/definepath: New script.
	* scripts/common.make (dbdir): Moved here from
	db-crypt/db/Makefile.am.
	(exportdir): New constant.
	* scripts/Makefile.am (noinst_SCRIPTS): Added unprefix and
	definepath.  Moved lyskomd-copyrights and update-copyright here
	from EXTRA_DIST.
	* run-support/config: Mention that this is installed as
	lyskomd.conf.
	* run-support/Makefile.am (sbin_SCRIPTS): Renamed savecore to
	savecore-lyskom, to avoid confusion with the /usr/bin/savecore
	program on Solaris.
	(MOSTLYCLEANFILES): Track the change.
	(savecore-lyskom): Track the change.
	(install-data-local): Install the config file as lyskomd.conf, not
	config.  Create the lyskomd.cores directory.
	(uninstall-local): Track the name change of lyskomd.conf.
	* src/server/testsuite/config/unix.exp (lyskomd_start): Use the
	old path names, so that we don't have to update the entire test
	suite.
	* doc/lyskomd.texi: Updated the default paths to the new world
	order.
	* db-crypt/db/Makefile.am (dbdir): Moved to common.make.

	Added man pages that refer to the Texinfo documentation for all
	installed programs.  Made the man pages static.
	* doc/man/splitkomdb.8: New file.
	* doc/man/savecore-lyskom.8: New file.
	* doc/man/lyskomd.8: Removed the version number.
	* doc/man/updateLysKOM.8: Removed the version number.
	* doc/man/Makefile.am (man_MANS): Added savecore-lyskom.8 and
	splitkomdb.8.

1683 1684
2003-08-15  Per Cederqvist  <ceder@ceder.dyndns.org>

1685 1686 1687 1688 1689 1690 1691 1692 1693 1694 1695 1696 1697 1698 1699 1700 1701 1702 1703 1704 1705 1706 1707 1708 1709 1710 1711 1712
	Distribution fix.
	* mkmi: Always regenerate scripts/depcomp.

	Xenofarm fix.
	* src/server/testsuite/lyskomd.0/person-cov.exp: Added "DNS log
	threshold: 3600" to support a certain slow Xenofarm computer.

	Calculate the file name of the configuration file in a single
	place.  Related code cleanup.
	* src/server/updateLysKOM.c (main): Use
	get_default_config_file_name() to find the config file.
	* src/server/splitkomdb.c (main): Ditto.
	* src/server/komrunning.c (main): Ditto.
	* src/server/dbck.c (main): Ditto.
	* src/server/ramkomd.c (main): Ditto.  Don't set read_config_file
	-- nothing used the value.
	* src/server/server-config.c (read_config_file): Variable removed.
	(CONFIG_FILE): Now static.
	(default_config): New static variable.
	(get_default_config_file_name): New function.
	(free_default_config_file_name): New function.
	(DEFAULT_DBASE_DIR): Removed.
	* src/server/param.h (read_config_file): Variable removed.
	* src/include/kom-config.h (DEFAULT_DBASE_DIR): Removed.
	(CONFIG_FILE): Removed.
	(get_default_config_file_name): New function.
	(free_default_config_file_name): New function.

1713 1714 1715 1716 1717 1718 1719 1720 1721 1722 1723 1724 1725 1726 1727 1728 1729 1730 1731 1732 1733 1734 1735 1736 1737 1738 1739 1740 1741 1742 1743 1744 1745
	Handle scheduling of different weights.  The scheduling priority
	must still be 0.
	* doc/lyskomd.texi (Parameters): Document "Default priority", "Max
	priority", "Default weight" and "Max weight".
	* src/server/connections.h (Connection): New field: schedule.
	* src/server/server-config.c (parameters): Added "Default
	priority", "Max priority", "Default weight" and "Max weight".
	(require_less_eq): New static function.
	(read_configuration): Check the new parameters.  For now,
	max_priority must be set to 0, since connections.c can only handle
	a single priority.
	* src/server/param.h (struct kom_par): New fields:
	default_priority, max_priority, default_weight, max_weight.
	* src/server/internal-connections.c (init_connection): Initialize
	the schedule.
	* src/server/connections.c (login_request): Set the priority and
	weight from param.default_priority and param.default_weight.
	(adjust_penalty): Use the weight to reduce the penalty.
	(get_scheduling): Return the actual scheduling information from
	the connection.
	(set_scheduling): Check the priority and weight against
	param.max_priority and param.max_weight.  Allow the user to change
	them, as long as it is done within those limits.  Store the new
	values in the connection.  Note: the current implementation cannot
	handle more than one priority.  The check for that is made in
	server-config.c.
	* src/server/testsuite/get-time-often.c (longopts): Added
	--priority and --weight.
	(main): Handle the new options.
	* src/server/testsuite/lyskomd.0/01.exp: The new default weight is
	20.  The new maximum weight is 100.
	* src/server/testsuite/lyskomd.0/03.exp: Ditto.

1746 1747 1748 1749 1750 1751 1752 1753 1754 1755 1756 1757 1758 1759 1760 1761 1762 1763 1764 1765 1766 1767 1768 1769 1770 1771 1772 1773 1774 1775 1776 1777 1778 1779 1780 1781 1782 1783 1784 1785 1786 1787 1788 1789 1790 1791 1792 1793 1794 1795 1796 1797 1798
2003-08-14  Per Cederqvist  <ceder@ceder.dyndns.org>

	Added the set-scheduling and get-scheduling requests.  The current
	implementation only supports priority==0 and weight==1.
	* doc/Protocol-A.texi (Session Information): Added Scheduling-Info.
	(get-scheduling): New request.
	(set-scheduling): New request.
	(Error Codes): Added priority-denied, weight-denied and
	weight-zero.
	* src/server/prot-a.c (prot_a_reply): Handle rt_scheduling_info.
	* src/server/prot-a-output.h, src/server/prot-a-output.c
	(prot_a_output_scheduling_info): New function.
	* src/server/fncdef.txt: Added get_scheduling and set_scheduling.
	* src/server/connections.h (enum res_type): Added
	rt_scheduling_info.
	(union result_holder): Added scheduling_info.
	* src/server/connections.c (get_scheduling): New request.
	(may_change_scheduling): New static function.
	(set_scheduling): New request.
	* src/include/services.h (set_scheduling): New request.
	(get_scheduling): New request.
	* src/include/kom-types.h (Scheduling_info): New struct.
	* src/include/kom-errno.h (enum kom_err): Added
	KOM_PRIORITY_DENIED, KOM_WEIGHT_DENIED and KOM_WEIGHT_ZERO.
	* src/server/testsuite/lyskomd.0/01.exp: Test get-scheduling and
	set-scheduling.
	* src/server/testsuite/lyskomd.0/03.exp: Ditto.

	Test suite improvement.
	* src/server/testsuite/Makefile.am
	(../../libraries/adns/client/adnshost): New target.
	(site.exp): Depend on adnshost, so that "make check" works even if
	"make check" has not been run in the adns directory.

	Minor doc fixes.
	* doc/lyskomd.texi (Parameters): Document the default value of
	"Low penalty".  Removed some text that was left behind a
	cut-n-paste operation.
	(Adding a New Protocol Request): Added a CHK_CONNECTION call to
	the example.

	Changed status of set-keep-commented from experimental to
	recommended.
	* doc/Protocol-A.texi (Protocol Requests): Changed the status of
	set-keep-commented from experimental to recommended.
	(set-keep-commented): Ditto.

	Clarify the description of get-stats-description and get-stats.
	* doc/Protocol-A.texi (get-stats-description): State that this
	always returns the same during a session, so clients can cache
	this value aggressively.
	(get-stats): Elaborate on the access-denied error code.

1799 1800
2003-08-13  Per Cederqvist  <ceder@ceder.dyndns.org>

1801 1802 1803 1804 1805 1806 1807 1808 1809 1810 1811 1812 1813 1814 1815 1816 1817 1818 1819 1820 1821 1822 1823 1824 1825 1826
	Use AM_PATH_PYTHON to find the Python interpreter.  (Bug 329).
	* configure.in: Use AM_PATH_PYTHON.
	(HAVE_PYTHON): New automake conditional.
	* HACKING: Use Automake 1.7.6 with a patch for better Python
	handling.
	* doc/Makefile.am (check-doc): Only perform the tests if
	HAVE_PYTHON is set.  Use $(PYTHON) instead of python.  Print a
	warning if HAVE_PYTHON isn't set.
	* src/server/testsuite/Makefile.am (check-lyskomd): Only perform
	the tests if HAVE_PYTHON is set.  Use $(PYTHON) instead of python.
	Print a warning if HAVE_PYTHON isn't set.
	(check-leaks): Ditto.
	(site.exp): Set python to the Python interpreter found by
	configure.
	* src/server/testsuite/lyskomd.0/Makefile.am ($(srcdir)/15.exp):
	Use $(PYTHON) instead of python.  Use the HAVE_PYTHON Automake
	conditional.
	($(srcdir)/19.exp): Ditto.
	* src/server/testsuite/config/unix.exp (obtain_lock): Use $python
	from site.exp instead of hardcoding "python".
	(client_start): Ditto.
	(client_start_fail): Ditto.
	* scripts/xenofarm.sh: Removed tests for python.  We should get a
	warning message from "make check" if python is missing, but it
	should still succeed.

1827 1828 1829 1830 1831 1832 1833 1834 1835 1836
	Testsuite improvement.
	* src/server/testsuite/lyskomd.0/37.exp: Added "DNS log threshold"
	entries so that a slow DNS server won't interfere with this test.

	Protocol-A.texi was incompatible with the texinfo.tex supplied
	with Automake 1.7.6.
	* doc/Protocol-A.texi (\tensltt): Define this command using TeX
	syntax, and add a \global.  I have only a vague idea what this
	does, but it seems to work with texinfo.tex 2003-05-04.08.

1837 1838 1839 1840 1841 1842 1843 1844 1845 1846 1847 1848 1849 1850 1851 1852 1853 1854 1855 1856 1857 1858 1859 1860 1861 1862 1863 1864 1865
	Create scripts/common.make, and include it from all LysKOM
	Makefile.am files.  Move the RM setting to it.  (Bug 851).
	* scripts/common.make: New file, which contains makefile fragments
	that all makefiles should include.
	(RM): Moved this variable to this file.
	* db-crypt/db/Makefile.am: Include common.make instead of setting
	RM.
	* doc/Makefile.am: Ditto.
	* run-support/Makefile.am: Ditto.
	* src/libraries/libeintr/Makefile.am: Ditto.
	* src/server/testsuite/Makefile.am: Ditto.
	* src/server/testsuite/lyskomd.0/Makefile.am: Ditto.
	* Makefile.am: Include common.make.
	* db-crypt/Makefile.am: Ditto.
	* scripts/Makefile.am: Ditto.
	* src/Makefile.am: Ditto.
	* src/include/Makefile.am: Ditto.
	* src/include/server/Makefile.am: Ditto.
	* src/libraries/Makefile.am: Ditto.
	* src/libraries/libansi/Makefile.am: Ditto.
	* src/libraries/libcommon/Makefile.am: Ditto.
	* src/libraries/libmisc/Makefile.am: Ditto.
	* src/server/Makefile.am: Ditto.
	* src/server/testsuite/config/Makefile.am: Ditto.
	* doc/man/Makefile.am: Ditto.
	
	Automake lint.
	* configure.in: Use new-style call to AM_INIT_AUTOMAKE.

1866 1867 1868 1869 1870 1871 1872 1873 1874 1875 1876 1877 1878 1879 1880 1881
	Implement find-next-conf-no and find-previous-conf-no.  
	(Bug 123).
	* doc/Protocol-A.texi (Server Information): Refer to
	find-previous-conf-no.
	(find-next-conf-no): New request.
	(find-previous-conf-no): New request.
	* src/server/fncdef.txt: Added find_next_conf_no and
	find_previous_conf_no.
	* src/include/services.h, src/server/conference.c
	(find_next_conf_no): New function.
	(find_previous_conf_no): New function.
	* src/server/testsuite/lyskomd.0/01.exp: Test find_next_conf_no
	and find_previous_conf_no.
	* src/server/testsuite/lyskomd.0/03.exp: Ditto.
	* src/server/testsuite/lyskomd.0/41.exp: Ditto.

1882 1883 1884 1885 1886 1887 1888 1889 1890 1891 1892 1893
	The test suite failed on really really slow computers.
	* src/server/testsuite/config/unix.exp (lyskomd_start): Increase
	the "Connect timeout" and "Login timeout" to 1 day, unless the
	test specifies them.  I suspect that the "Connect timeout"
	interfered with a test on one of the slower members of the
	Xenofarm.
	* src/server/testsuite/lyskomd.0/38.exp: Fixed a race condition:
	if the client was killed at the wrong time, the test would crash.
	* src/server/testsuite/lyskomd.0/37.exp: Pass a $base_config to
	lyskomd_start in the other place as well.  The fix made on
	2003-08-07 was incomplete.

1894 1895 1896 1897 1898 1899 1900 1901 1902 1903 1904 1905 1906 1907 1908 1909 1910 1911 1912 1913 1914 1915 1916 1917 1918 1919 1920 1921 1922 1923 1924
	Implement first-unused-conf-no and first-unused-text-no.
	(Bug 121).
	* doc/Protocol-A.texi (first-unused-conf-no): New request.
	(first-unused-text-no): New request.
	(Server Information): Refer to the new requests instead of
	inferior ways to get the same information.
	* src/include/services.h (first_unused_conf_no): New function.
	(first_unused_text_no): New function.
	* src/server/text.c (first_unused_text_no): New function.
	* src/server/simple-cache.c (query_next_conf_no): New function.
	* src/server/prot-a.c (prot_a_reply): Handle rt_conf_no.
	* src/server/prot-a-output.h, src/server/prot-a-output.c
	(prot_a_output_conf_no): New function.
	* src/server/fncdef.txt: Added first_unused_conf_no and
	first_unused_text_no.
	* src/server/connections.h (enum res_type): Aded rt_conf_no.
	(union result_holder): Added conf_no.
	* src/server/conference.c (first_unused_conf_no): New function.
	* src/server/cache.h (query_next_conf_no): New function.
	* src/server/testsuite/lyskomd.0/01.exp: Test
	114:first-unused-conf-no and 115:first-unused-text-no.
	* src/server/testsuite/lyskomd.0/03.exp: Ditto.
	* src/server/testsuite/lyskomd.0/41.exp: Ditto.

	Makefile improvements that might help portability to
	CYGWIN_NT-5.0.
	* src/server/testsuite/Makefile.am (AM_CPPFLAGS): Added ../../..,
	so that config.h is properly found.
	* src/server/Makefile.am (aux.h): Don't append to a non-existing
	file; use ">" instead of ">>" to create it.

1925 1926
2003-08-12  Per Cederqvist  <ceder@ceder.dyndns.org>

1927 1928 1929 1930 1931 1932 1933 1934 1935 1936 1937 1938 1939 1940 1941 1942 1943 1944 1945 1946 1947 1948 1949 1950 1951 1952
	Implement and use has_access() to simplify code.  (Bug 723).
	* src/server/manipulate.h, src/server/membership.c (has_access):
	New function.  This is a simple wrapper around access_perm that
	simplifies the code.
	* src/server/membership.c (filter_conf_no): Use has_access()
	instead of access_perm() to simplify code.
	(sub_member): Ditto.
	(add_member_common): Ditto.
	* src/server/text.c (filter_secret_info): Ditto.
	(add_recipient): Ditto.
	(sub_recipient): Ditto.
	* src/server/send-async.c (async_new_name): Ditto.
	(async_new_user_area): Ditto.
	* src/server/regex-match.c (lookup_regexp): Ditto.
	* src/server/person.c (do_query_read_texts): Ditto.
	* src/server/conference.c (set_conf_errno): Ditto.
	(lookup_name): Ditto.
	(lookup_z_name): Ditto.
	(do_lookup): Ditto.
	(send_async_new_presentation): Ditto.
	(send_async_new_motd): Ditto.
	(modify_conf_info): Ditto.
	* src/server/admin.c (send_message): Ditto.
	* doc/lyskomd.texi (Adding a New Protocol Request): Fixed broken
	code example.  Use has_access() instead of access_perm().

1953 1954 1955 1956 1957 1958 1959 1960 1961 1962 1963 1964 1965 1966 1967 1968 1969 1970 1971 1972 1973 1974 1975 1976 1977 1978 1979
	lyskomd is now officially shut down via SIGTERM.  (SIGHUP still
	works, and SIGINT also shuts it down.)  (Bug 808).
	* doc/lyskomd.texi (Parameters): lyskomd is now officially shut
	down by SIGTERM, not SIGHUP.
	(Signals): Ditto.  However, SIGHUP and SIGINT also works.
	(Administration): Recommend SIGTERM, not SIGHUP.
	(Bugs): Removed entry about terminating on SIGINT and SIGTERM.
	* src/server/testsuite/config/unix.exp (lyskomd_death): The reason
	"signal" now means SIGTERM.  Also handle "sighup" and "sigint".
	* src/server/updateLysKOM.c (checkstatus): Use SIGTERM, not
	SIGHUP.
	* src/server/sigflags.h: Updated a comment: SIGTERM is now the
	official way to shut down lyskomd.
	* src/server/ramkomd.c (server_init): Handle SIGTERM and SIGINT
	the same way as SIGHUP: shut down.
	(main): Ditto.
	(sighandler_term): New name for former sighandler_hup.  Log proper
	message depending on which signal was used to shut down the
	server.
	* src/server/komrunning.c (shutdown_lyskom): Use SIGTERM, not
	SIGHUP, to shut down lyskomd.
	* src/server/connections.c: Updated a comment: SIGTERM is now the
	official way to shut down lyskomd.
	* src/server/testsuite/lyskomd.0/31.exp: Test to shut down the
	server by both SIGTERM, SIGHUP and SIGINT.
	* src/server/testsuite/lyskomd.0/07.exp: Use TERM instead of HUP
	to shut down lyskomd.
1980 1981 1982 1983 1984 1985 1986 1987
	* src/server/testsuite/lyskomd.0/11.exp: Ditto.
	* src/server/testsuite/lyskomd.0/30.exp: Ditto.
	* src/server/testsuite/lyskomd.0/37.exp: Ditto.
	* src/server/testsuite/lyskomd.0/38.exp: Ditto.
	* src/server/testsuite/lyskomd.0/39.exp: Ditto.
	* src/server/testsuite/lyskomd.0/bug-48.exp: Ditto.
	* src/server/testsuite/lyskomd.0/bug-52.exp: Ditto.
	* src/server/testsuite/lyskomd.0/bug-810.exp: Ditto.
1988