ChangeLog 561 KB
Newer Older
1
2
2009-05-31  Per Cederqvist  <ceder@lysator.liu.se>

3
4
5
	Make it possible to select output format 3.
	* src/server/ram-io.c (set_output_format): Handle format 3.

Per Cederqvist's avatar
Per Cederqvist committed
6
7
8
9
10
11
	Fixed typos in comments.
	* src/server/simple-cache.c (cached_create_text),
	* src/server/text.c (do_create_text),
	* src/server/testsuite/get-time-often.c (read_server): Fixed typos
	in comments.

Per Cederqvist's avatar
Per Cederqvist committed
12
13
14
	Minor source code cleanup.
	* src/server/ram-smalloc.c: Added a missing newline.

15
16
17
18
	Fix a race condition in the test suite.
	* src/server/testsuite/lyskomd.0/03.exp: Fix a race condition by
	sleeping one second.

19
20
21
	Avoid calling ungetc() on EOF.
	* src/server/ram-parse.c (fskipwhite): Don't unget EOF.

22
23
24
25
26
27
28
29
30
31
2008-03-31  Per Cederqvist  <ceder@lysator.liu.se>

	Turn off the garb by default, due to popular request.
	* doc/lyskomd.texi (Parameters): Changed "Garb" from "on" to
	"off".
	(Administration): Suggest that the administrator may want to turn
	on the garb.
	* src/server/server-config.c (parameters): Changed "Garb" from
	"on" to "off".

32
33
34
35
36
37
2007-07-14  Per Cederqvist  <ceder@lysator.liu.se>

	Fixed a broken example in the documentation.
	* doc/lyskomd.texi (Files): The lyskomd.clients example line was
	missing the third field.  Fix.

38
39
2007-07-03  Per Cederqvist  <ceder@lysator.liu.se>

40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
	Implement dbfile wrappers for putc, fputs, feof and ferror, and
	use them.
	* src/server/ram-io.c, src/server/ram-io.h
	(dbfile_putc): New function.
	(dbfile_fputs): New function.
	(dbfile_feof): New function.
	(dbfile_ferror): New function.
	* src/server/simple-cache.c (save_one_text): Use dbfile_ferror.
	(sync_part): Ditto.
	* src/server/ram-output.c (foutput_string): Use dbfile_putc.
	(foutput_ulong): Ditto.
	(foutput_atsign): Ditto.
	(foutput_newline): Ditto.
	(foutput_space): Ditto.
	(foutput_bool): Ditto.
	(foutput_array_start): Use dbfile_fputs.
	(foutput_array_end): Ditto.
	(foutput_array_nodata): Ditto.
	* src/server/ram-io.c (dbfile_change_magic): Use dbfile_ferror.
	* src/server/local-to-global.c (l2g_write): Use dbfile_putc.
	* src/server/dbck-cache.c (init_cache): Use dbfile_feof().

62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
	Let dbfile_open_write() write the file header.  Extract the code
	that changes the state to dbfile_change_magic().  Reduce code
	duplication by teaching foutput_header() to handle old file
	formats.
	* src/server/simple-cache.c (pre_sync): Adjusted to new API of
	dbfile_open_write(), including removing calls to foutput_header().
	(save_one_text): Extracted parts of the functionality to
	dbfile_change_magic().
	* src/server/ram-output.c (foutput_header): Moved support for file
	formats 0, 1 and 2 here from sync_output_header().
	* src/server/ram-io.c, src/server/ram-io.h
	(dbfile_open_write): New argument: magic.  Write a file header
	with the specified state.
	(dbfile_change_magic): New function, extracted from
	cache_sync_all() in dbck-cache.c and save_one_text().
	* src/server/dbck-cache.c (sync_output_header): Removed.
	The ability to emit old-style headers moved to foutput_header().
	(cache_sync_all): Adjusted to new API of dbfile_open_write,
	including removing calls to sync_output_header().  Use
	dbfile_change_magic() instead of rewind() and
	sync_output_header().
	* src/server/Makefile.am (libcheck_a_SOURCES): Added
	server-time.c.

Per Cederqvist's avatar
Per Cederqvist committed
86
87
88
89
90
2007-07-01  Per Cederqvist  <ceder@lysator.liu.se>

	Use a modern autoconf.
	* mkmi (WANT_AUTOCONF): Set to 2.5, not 1.5.

91
92
93
94
95
96
97
98
2007-03-01  Per Cederqvist  <ceder@lysator.liu.se>

	Fix a minor inconsistency in the Protocol A documentation.
	* doc/Protocol-A.texi (The Misc-Info List): Document that it is up
	to the server to convert bcc-recpt to cc-recpt or to ignore them,
	when clients use an old-style request.  This agrees better with
	the wordings in get-text-stat-old.

99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
2006-11-06  Per Cederqvist  <ceder@lysator.liu.se>

	Introduce datafile format version 3.  A textstat contains
	"generation" and "reclamation" fields.
	* src/include/kom-types.h (Text_stat): Added fields generation
	and reclamation.
	* src/server/simple-cache.c (cached_get_text): Handle text file
	generation 0 in the old-fashioned way.  Added a #warning about
	other generations.  Check return value of fseek.
	(cached_create_text): Added a #warning about nonzero generations
	of the text file.
	(cached_create_text): Initialize generation and reclamation.
	(save_one_conf): Do a file copy only if the formats of file A and
	B are the same.  Otherwise, parse the object into a temporary
	object and re-serialize it.
	(save_one_pers): Ditto.
	(save_one_text): Ditto.
	(init_cache): Handle data file format 3.
	* src/server/ram-parse.c (fparse_text_stat_3): New static function.
	(fparse_text_stat_2): Set generation and reclamation to 0.
	(fparse_text_stat_0): Ditto.
	(fparse_info): Handle data file format 3.
	(fparse_conference): Ditto.
	(fparse_person): Ditto.
	(fparse_membership): Ditto.
	(fparse_membership_list): Ditto.
	(fparse_text_stat): Ditto.
	(fparse_member): Ditto.
	* src/server/ram-output.c (foutput_info): Handle data file format 3.
	(foutput_text_stat_3): New static function.
	(foutput_person): Ditto.
	(foutput_conference): Ditto.
	(foutput_text_stat): Ditto.
	(foutput_membership): Ditto.
	(foutput_membership_list): Ditto.
	(foutput_member): Ditto.
	(foutput_conf_type): Ditto.
	* src/server/ram-io.c (output_format): Initialize it to 3.
	(dbfile_open_read): Handle data file format 3.
	* src/server/memory.c (copy_text_stat): Handle generation and
	reclamation.
	(init_text_stat): Initialize generation and reclamation.
	* src/server/membership.c (read_ranges_postcondition): Use data
	file format 3 when dumping a membership to stderr.
	* src/server/dbck.c (main): Handle data file format 3.
	* src/server/dbck-cache.c (cached_get_text): Handle text file
	generation 0 in the old-fashioned way.  Added a #warning about
	other generations.
	(cached_flush_text): Flush a generation 0 text file.  Added a
	#warning about other generations.
	(sync_output_header): Handle data file format 3.
	(cache_sync_all): Ditto.
	(init_cache): Ditto.
	* src/server/testsuite/test-l2g.c (main): Use data
	file format 3 when reading an Local2Global structure from stdin,
	or writing it to stdout.

156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
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
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
2006-10-20  Per Cederqvist  <ceder@lysator.liu.se>

	Wrap all file pointers that refer to the database files inside a
	"struct dbfile".  Add a database version field to that struct, and
	remove the global database format variables.  This allows lyskomd
	to have files in different versions open at the same time, so that
	it can convert format X to format X+1 on the fly.  (This is more
	preparation for bug 1616.)
	* src/server/ram-io.h (struct dbfile): New struct.
	(dbfile_delete): New function.
	(set_output_format): New function.
	(dbfile_open_write): New function.
	(dbfile_open_read): New function.
	(dump_dbfile_stats): New function.
	(dbfile_ftell): The argument is now a struct dbfile*, not FILE*.
	(dbfile_getc): Ditto.
	(dbfile_ungetc): Ditto.
	* src/server/ram-io.c (output_format): Moved from ram-output.c and
	made static variable.  This defines the output format of all
	struct dbfile objects created after a change to the variable.
	(nr_dbfile_objects): New static statistics variable.
	(nr_dbfile_files): New static statistics variable.
	(dbfile_new): New static function.
	(dbfile_delete): New function.
	(set_output_format): Moved from ram-output.c.
	(get_version): Moved from simple-cache.c.  Changed return type to
	int.  The argument is an open dbfile, not a file name.  Use
	dbfile_getc() instead of getc().
	(dbfile_open_read): New function.
	(dbfile_open_write): New function.
	(dump_dbfile_stats): New function.
	(dbfile_ftell): The argument is now a struct dbfile*, not FILE*.
	(dbfile_getc): Ditto.
	(dbfile_ungetc): Ditto.
	* src/server/conference.c: Include ram-io.h, since
	local-to-global.h needs it.
	* src/server/memory.c: Ditto.
	* src/server/person.c: Ditto.
	* src/server/prot-a-output.c: Ditto.
	* src/server/text.c: Ditto.
	* src/server/membership.c: Include ram-io.h.
	(read_ranges_postcondition): Set up a dbfile wrapper around stderr
	for the report created if check_membership() fails.
	* src/server/local-to-global.h, src/server/local-to-global.c
	(l2g_read): The fp argument is now a struct dbfile*, not a FILE*.
	* src/server/dbck.c (oformat): Removed.  The format is stored
	inside the dbfile struct instead.
	(main): Report an error message if an attempt to use an undefined
	output format is detected.
	* src/server/dbck-cache.c: Change all FILE* that refer to the
	status files to struct dbfile* in all functions, and made
	appropriate changes.  Use fp->format instead of the global
	variable oformat or data_file_version.
	(get_version): Moved to ram-io.c.
	(sync_output_header): Removed the oformat argument.  Use
	fp->format instead.  All callers updated.
	(cache_sync_all): Use dbfile_open_write() instead of i_fopen(),
	and dbfile_delete() instead of i_fclose().
	(init_cache): Removed the data_file_version variable, as the
	format is now stored inside the dbfile struct.  Use
	dbfile_open_read() instead of i_fopen(), and dbfile_delete()
	instead of i_fclose().
	* doc/lyskomd.texi (Modifying Stored Types): The default database
	format is now defined in ram-io.c, and nowhere else.  It is
	preferrable if lyskomd is able to convert an old format to a new
	one after an upgrade.
	* src/server/testsuite/test-l2g.c (main): Added a dbfile wrapper
	around stdin in call to l2g_read(), and another around stderr in
	call to l2g_write().
	* src/server/testsuite/Makefile.am (test_l2g_LDADD): Include
	libeintr.a.
	* src/server/simple-cache.c: Change all FILE* that refer to the
	status files to struct dbfile* in all functions, and made
	appropriate changes.
	(pre_sync): Use dbfile_delete() instead of i_fclose(), and
	dbfile_open_write() instead of i_fopen().
	(post_sync): Use dbfile_delete() instead of i_fclose(), and
	dbfile_open_read() instead of i_fopen().
	(save_one_text): Use dbfile_delete() instead of i_fclose(), and
	dbfile_open_read() instead of i_fopen().
	(sync_part): Use dbfile_delete() instead of i_fclose().
	(init_cache): Removed local variable datafile_version; use
	fp->format instead.  Use dbfile_open_read() instead of i_fopen().
	Removed call to set_input_format(), as dbfile_open_read() does
	the same thing.
	(free_all_cache): Use dbfile_delete() instead of i_fclose().
	* src/server/ramkomd.c (dump_exit_statistics): Dump dbfile stats.
	* src/server/ram-parse.h, src/server/ram-parse.c: Change all
	FILE* that refer to the status files to struct dbfile* in all
	functions, and made appropriate changes.
	(set_input_format): Removed.
	* src/server/ram-parse.c (input_format): Removed.  All users
	changed to use fp->format instead, so that lyskomd can read files
	with different versions at the same time.
	(fparse_text_stat_2): Code cleanup.
	(fparse_text_stat_0): Use dbfile_ungetc() instead of ungetc().
	* src/server/ram-output.h, src/server/ram-output.c: Change all
	FILE* that refer to the status files to struct dbfile* in all
	functions, and made appropriate changes.
	(output_format): Moved to ram-io.c and made static.
	(set_output_format): Moved to ram-io.c.
257

258
259
260
261
262
263
264
265
266
267
268
269
270
	Fixed more missed opportunities to use fparse_set_pos() and
	foutput_newline().
	* src/server/dbck-cache.c (get_version): Use fparse_set_pos()
	intead of fseek() where appropriate.
	(init_cache): Ditto.
	* src/server/simple-cache.c (get_version): Ditto.
	(pre_sync): Removed dead code.  Use foutput_newline() and
	fparse_set_pos() where appropriate.
	(save_one_conf): Use foutput_newline() where appropriate.
	(save_one_pers): Ditto.
	(save_one_text): Ditto.
	(init_cache): Use fparse_set_pos() where appropriate.

271
272
273
274
275
276
277
278
279
	Improve logging and error messages.
	* src/server/simple-cache.c (read_person): Use restart_kom instead
	of kom_log when there is a size mismatch.
	(read_conference): Ditto.
	(pre_sync): Log the file name and error message when the file to
	save in cannot be opened.
	(init_cache): Improve the error message when the database is in
	the wrong format.

280
281
282
283
	Fixed a missing opportunity to use foutput_newline.
	* src/server/dbck-cache.c (cache_sync_all): Replaced another
	fprintf call with a call to foutput_newline.

284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
2006-10-18  Per Cederqvist  <ceder@lysator.liu.se>

	Renamed sync_output_header to foutput_header, and clean up the
	time handling.  This is more preparation for bug 1616.
	* src/server/simple-cache.c (sync_output_header): Renamed to
	foutput_header and moved to ram-output.c.  All callers updated.
	* src/server/ramkomd.c (current_time): Moved to server-time.c.
	(main): Use set_initial_time instead of calling gettimeofday and
	accessing current_time directly.
	* src/server/ram-output.c, src/server/ram-output.h
	(foutput_header): Renamed from sync_output_header.  Use
	output_format instead of a hardcoding it to 2.
	* src/server/dbck.c (main): Call set_initial_time().
	* src/server/dbck-cache.c (sync_output_header): Use foutput_header
	to output a format 2 header.
	* src/server/server-time.c: New file.
	(current_time): Moved here from ramkomd.c.
	* src/server/server-time.c, src/server/server-time.h
	(set_time): Moved here from connections.c.
	(set_initial_time): New function.
	* src/server/connections.c (set_ time): Moved to server-time.c.
	* src/server/Makefile.am (GENOBJS): Added server-time.c.
	(DBCK): Ditto.

308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
2006-10-17  Per Cederqvist  <ceder@lysator.liu.se>

	Added functions wrapper functions around a few stdio functions,
	and use them when accessing database files.  This is a preparatory
	step for introducing "struct dbfile", which will enable lyskomd to
	manipulate more than one database file format at the same time.
	That ability is needed for solving bug 1616.
	* src/server/ram-io.c, src/server/ram-io.h: New files.
	(dbfile_ftell): New function.
	(dbfile_getc): New function.
	(dbfile_ungetc): New function.
	* src/server/Makefile.am (libcheck_a_SOURCES): Added ram-io.c.
	(lyskomd_SOURCES): Added ram-io.h and ram-io.c.
	(DBCK): Added ram-io.h and ram-io.c.
	* src/server/simple-cache.c (read_person): Check the return value
	from fparse_set_pos().
	(read_conference): Ditto.
	(read_text_stat): Ditto.
	(write_conf): New static function.
	(save_one_conf): Use it to simplify code.
	(write_text): New static function.
	(save_one_text): Use it to simplify code.
	* src/server/ram-parse.h, src/server/ram-parse.c
	(fparse_set_pos): New function.
	* src/server/ram-output.c: Include ram-io.h.
	(foutput_space): New static function.  Use it where appropriate.
	(foutput_bool): Ditto.
	(foutput_array_start): Ditto.
	(foutput_array_end): Ditto.
	(foutput_array_nodata): Ditto.
	* src/server/ram-output.h, src/server/ram-output.c
	(foutput_atsign): New function.
	(foutput_newline): New function.
	* src/server/dbck-cache.c: Include ram-io.h.  Use dbfile_getc()
	instead of getc(), dbfile_ungetc() instead of ungetc(), and
	dbfile_ftell() instead of ftell(), when accessing the database
	file.  Use foutput_atsign and foutput_newline when appropriate.
	* src/server/local-to-global.c: Ditto.
	* src/server/ram-parse.c: Ditto.
	* src/server/simple-cache.c: Ditto.

349
350
351
352
353
354
355
2006-10-01  Per Cederqvist  <ceder@lysator.liu.se>

	Check that conferences read from the database are the expected
	size.  (Bug 168).
	* src/server/simple-cache.c (read_person): Sanitycheck the
	supplied size.

356
357
358
359
360
361
362
2006-09-29  Per Cederqvist  <ceder@lysator.liu.se>

	Check that conferences read from the database are the expected
	size.  (Bug 169).
	* src/server/simple-cache.c (read_conference): Sanitycheck the
	supplied size.

363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
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
428
429
430
431
432
433
434
435
436
437
438
439
440
2006-09-28  Per Cederqvist  <ceder@lysator.liu.se>

	Renamed the error code illegal-aux-item to illegal-aux-item-tag.
	Added error code 62=aux-exists, 63=aux-bad-object,
	64=illegal-aux-data, 65=aux-create-only, 66=aux-modify-only,
	67=aux-server-only, 68=aux-data-exists and 69=aux-no-such-text.
	Return proper error codes.  (Bug 192).
	* doc/Protocol-A.texi (Predefined Aux-Item Types): Renamed the
	error code illegal-aux-item to illegal-aux-item-tag.
	(create-text): Updated the error code section to mention the
	aux-related error codes illegal-aux-item-tag, aux-item-permission,
	aux-exists, aux-bad-object, illegal-aux-data, aux-modify-only,
	aux-server-only, aux-data-exists and aux-no-such-text instead of
	only illegal-aux-item and aux-item-permission.  Refer to the
	"Error Codes" node instead of duplicating the reasons here.
	(create-anonymous-text): Ditto.
	(create-conf): Ditto.
	(create-person): Ditto.
	(modify-text-info): Ditto, but with aux-create-only instead of
	aux-modify-only.
	(modify-conf-info): Ditto.
	(modify-system-info): Ditto.
	(Error Codes): Renamed error code 48 from illegal-aux-item to
	illegal-aux-item-tag, and redefine it to not include errors in the
	data.  Added error code 62=aux-exists, 63=aux-bad-object,
	64=illegal-aux-data, 65=aux-create-only, 66=aux-modify-only,
	67=aux-server-only, 68=aux-data-exists and 69=aux-no-such-text.
	* doc/Makefile.am (check-doc): Replaced the error code
	illegal-aux-item with illegal-aux-item-tag and
	illegal-aux-item-data.
	* src/include/kom-errno.h (enum kom_err): Renamed KOM_ILL_AUX to
	KOM_ILL_AUX_TAG.  Added KOM_AUX_EXISTS, KOM_AUX_BAD_OBJECT,
	KOM_ILL_AUX_DATA, KOM_AUX_CREATE_ONLY, KOM_AUX_MODIFY_ONLY,
	KOM_AUX_SERVER_ONLY, KOM_AUX_DATA_EXISTS and KOM_AUX_NO_SUCH_TEXT.
	* src/server/aux-items.c (aux_item_validate): Doc added.  Added
	missing punctuation in comments.  Set kom_errno to
	KOM_ILL_AUX_DATA instead of KOM_ILL_AUX when a regexp cannot be
	compiled or doesn't match an entry, or when a validator fails.
	(aux_item_add_perm): Document that kom_errno is set on failure.
	Do not modify kom_errno on success.  Track name change of
	KOM_ILL_AUX_TAG.  Set KOM_AUX_MODIFY_ONLY, KOM_AUX_CREATE_ONLY,
	KOM_AUX_SERVER_ONLY when appropriate (used to be KOM_AUX_PERM).
	Don't override the error code set by aux_item_check_unique() or
	aux_item_validate() when any of those check fails.
	(aux_item_check_unique): Set kom_errno to KOM_AUX_EXISTS or
	KOM_AUX_DATA_EXISTS on failure (used to sometimes be
	KOM_ILL_AUX).
	(text_stat_check_add_aux_item_list): Track name change of
	KOM_ILL_AUX_TAG.  Set kom_errno to KOM_AUX_BAD_OBJECT if the
	aux-item is not allowed on the active object (used to be
	KOM_AUX_PERM).
	(conf_stat_check_add_aux_item_list): Ditto.
	(system_check_add_aux_item_list): Ditto.
	(aux_item_validate_existing_text): Set kom_errno to
	KOM_AUX_NO_SUCH_TEXT instead of NO_SUCH_TEXT.  Set it to
	KOM_ILL_AUX_DATA if the text number is non-numeric.
	* src/server/testsuite/lyskomd.0/00.exp: Updated the expected
	error codes for aux-item-related errors.
	* src/server/testsuite/lyskomd.0/03.exp: Ditto.
	* src/server/testsuite/lyskomd.0/04.exp: Ditto.
	* src/server/testsuite/lyskomd.0/14.exp: Ditto.
	* src/server/testsuite/lyskomd.0/17.exp: Ditto.
	* src/server/testsuite/lyskomd.0/55.exp: Ditto.
	* src/server/testsuite/lyskomd.0/aux-items-cov.exp: Ditto.
	* src/server/testsuite/lyskomd.0/bug-572.exp: Ditto.
	* src/server/testsuite/lyskomd.0/20.exp: Ditto.  Remove the
	want_illegal_aux_item proc and simplify the code that checks the
	error codes.
	* src/server/testsuite/lyskomd.0/23.exp: Ditto.
	* src/server/testsuite/lyskomd.0/26.exp: Ditto.
	* src/server/testsuite/lyskomd.0/28.exp: Ditto.

	Added test cases for handling duplicate faq-text aux-items.
	* src/server/testsuite/lyskomd.0/21.exp: Added new tests that
	attempt to set duplicate faq-text aux-items on a conference in
	various ways, and removing it and re-adding it.  Also test adding
	two different faq-text aux-items at the same time.

441
442
443
444
445
446
447
448
449
450
2006-09-09  Per Cederqvist  <ceder@lysator.liu.se>

	Get rid of gcc-4.1.1 warnings by initializing more fields.
	* src/libraries/libcommon/misc-parser.c (parse_next_misc):
	Initialize bcc_recipient, received_at and is_sent.  This gets rid
	of a gcc-4.1.1 warning.
	* src/libraries/libcommon/parser.c (parse): Always  initialize
	answer.arguments and answer.indexes, to get rid of a gcc-4.1.1
	warning.

451
452
453
454
455
456
457
458
459
460
461
462
463
464
2006-09-02  Per Cederqvist  <ceder@lysator.liu.se>

	Rename the aux-item field "unique" to "one-per-person".
	(Preparation for solving bug 327).
	* doc/lyskomd.texi (Aux-Item Definition File): Renamed the
	"unique" field to "one-per-person", and clarify what it means.
	* src/server/aux-item-def-parse.y: Track the name change of the
	"one-per-person" aux-item field.
	(parse_aux_item_definitions): Ditto.
	* run-support/aux-items.conf: Ditto.
	* src/server/testsuite/leaks.0/lots-aux-items.conf: Ditto.
	* src/server/testsuite/lyskomd.0/aux-items.conf: Ditto.
	* src/server/testsuite/lyskomd.0/bug-64.aux: Ditto.

465
466
467
468
469
470
471
2006-08-04  Per Cederqvist  <ceder@lysator.liu.se>

	Add more test cases for Local_to_global. (Bug 217).
	* src/server/testsuite/l2g.0/09.exp: Added test cases for copying
	a Local_to_global where the last entry (or entries) has been
	deleted.

472
473
2006-08-01  Per Cederqvist  <ceder@lysator.liu.se>

474
475
476
477
	More aux-item-related test cases.
	* src/server/testsuite/lyskomd.0/55.exp: Added test cases for
	adding an aux-item to the wrong type of of object.
	
478
479
480
481
482
483
484
485
	Added more test cases for aux-items with restrictions on when they
	can be set.
	* src/server/testsuite/lyskomd.0/55.exp: New file.  Test creation
	of aux-items that are only allowed to be set when modifying or
	creating an object.
	* src/server/testsuite/lyskomd.0/aux-items-55.conf: New file, used
	by 55.exp.

Per Cederqvist's avatar
Per Cederqvist committed
486
487
488
489
	Comment fix.
	* src/server/aux-items.c (aux_item_validate): Fixed a typo in a
	comment.

Per Cederqvist's avatar
Per Cederqvist committed
490
491
492
493
494
	Test suite fix.
	* src/server/testsuite/lyskomd.0/person-cov.exp: Handle the new
	message when a person is deleted, even when debug calls are
	enabled.

Per Cederqvist's avatar
Per Cederqvist committed
495
496
497
498
499
500
	Code cleanup.
	* src/server/aux-items.c (aux_item_check_unique): Removed bogus
	code that find_aux_item_definitions to be needlessly called.
	Force the caller to supply the definition, as all callers already
	do.

501
502
503
504
	Removed code with no effect.  (Bug 142).
	* src/server/aux-items.c (aux_item_link_items): Removed assignment
	to a local variable that is not used after the assignment.

Per Cederqvist's avatar
Per Cederqvist committed
505
506
507
508
	Fixed typo in comment.
	* src/server/testsuite/lyskomd.0/aux-items-cov.exp: Fixed typo in
	comment.

509
510
511
512
513
514
	Removed bogus FIXME comments.  (Bug 141).
	* src/server/aux-items.c (aux_item_link_items): Removed FIXME
	comment about bug 141.  Bug 141 is invalid.  The call to
	mark_text_as_changed is not redundant, since we must mark both the
	source and destination item as changed.

515
516
517
518
519
	Document local-to-global.  (Bug 144).
	* doc/lyskomd.texi (local-to-global): Translated old Swedish text
	that describes the reasoning behind the local-to-global structure,
	and update it to match the current implementation.

520
521
522
523
524
525
526
2006-07-31  Per Cederqvist  <ceder@lysator.liu.se>

	Document the database process.  (Bug 144, partially)
	* doc/lyskomd.texi (The Database): Translated old Swedish text
	that describes how the database is implemented to English.
	Updated the description to match the current implementation.

Per Cederqvist's avatar
Per Cederqvist committed
527
528
529
530
531
532
533
534
535
536
537
538
539
2006-07-27  Per Cederqvist  <ceder@lysator.liu.se>

	Log changed names.
	* src/server/conference.c: Don't include s-string.h twice, but do
	include string-malloc.h.
	(change_name): Log the old and new name, and the user that changes
	it.
	* src/server/testsuite/lyskomd.0/03.exp: Be prepared for the new
	log message.
	* src/server/testsuite/lyskomd.0/06.exp: Ditto.
	* src/server/testsuite/lyskomd.0/13.exp: Ditto.
	* src/server/testsuite/lyskomd.0/send-async-cov.exp: Ditto.

540
541
2006-07-26  Per Cederqvist  <ceder@lysator.liu.se>

542
543
544
545
	Improve doc.  (Bug 144, partially).
	* doc/lyskomd.texi (Parameters): Document "Max mark_as_read
	chunks" and "Max super_conf loop".

Per Cederqvist's avatar
Per Cederqvist committed
546
547
548
549
550
	Code cleanup.
	* src/server/conf-file.c (check_trailer): New function.
	(convert_to_long): Use it to simplify the code.
	(convert_to_ulong): Ditto.
	
551
552
553
554
555
556
557
558
	Use strtol instead of atol in more places, and avoid a compiler
	warning.  (Bug 143).
	* src/server/conf-file.c (convert_to_ulong): New function.
	(assign_text_no): Use it instead of convert_to_long to get rid of
	a compiler warning.
	(assign_ulong): Use convert_to_ulong instead of atol.
	(assign_uint): Ditto.  Check for overflow.

559
560
561
562
563
564
565
	Use strtol instead of atol.  (Bug 143).
	* src/server/conf-file.c (convert_to_long): New function.
	(assign_text_no): Use it instead of using atol.  Check for
	overflow.
	(assign_conf_no): Ditto.
	(assign_int): Ditto.

566
567
568
569
570
571
572
573
574
575
576
577
	Log a message when a conference (or person) is deleted.
	* src/server/conference.c (delete_conf): Log a message when a
	conference is deleted, which includes the name of the conference
	and the user ID of the person that performs the deletion.
	* src/server/testsuite/config/prot-a.exp (kom_delete_conf): Expect
	the above message.
	* src/server/testsuite/lyskomd.0/03.exp: Ditto.
	* src/server/testsuite/lyskomd.0/13.exp: Ditto.
	* src/server/testsuite/lyskomd.0/32.exp: Ditto.
	* src/server/testsuite/lyskomd.0/conference-cov.exp: Ditto.
	* src/server/testsuite/lyskomd.0/person-cov.exp: Ditto.

578
579
580
581
582
583
584
2006-04-24  Per Cederqvist  <ceder@lysator.liu.se>

	Tell svn to ignore core files.
	* src/server/testsuite: Ignore core and vgcore.*.

	Replaced the "Client host" and "Client port" configuration file
	parameters with "Listen", and allow lyskomd to listen to more than
585
	one port.  (Bug 1112).
586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
614
615
616
617
618
619
620
621
622
623
624
625
626
627
628
629
630
631
632
633
634
635
636
637
638
639
640
641
642
643
644
645
646
647
	* doc/lyskomd.texi (Parameters): Replaced "Client host" and
	"Client port" with "Listen".
	* src/server/ramkomd.c (start_listen_sockets): New static
	function.  Listen to all the ports specified by the "Listen:"
	parameter.
	(server_init): Use start_listen_sockets() instead of calling
	isc_listentcp().  Removed the host and client_port arguments.  All
	callers updated.
	* src/server/testsuite/lyskomd.0/54.exp: New test suite.
	* src/server/conf-file.c (ctor_ipport_list): New static function.
	(dtor_ipport_list): New static function.
	(assign_ipport_list): New static function.
	(cf_ipport_list): New constant.
	* src/server/conf-file.h (struct ipport_entry): New struct.
	(struct ipport_list): New struct.
	(cf_ipport_list): New constant.
	* src/server/param.h (struct kom_par): Replaced "ip_client_host"
	and "ip_client_port" with "listen".
	* src/server/server-config.c (parameters): Ditto.
	* src/server/prot-a-parse-arg-c.awk: Include "conf-file.h" from
	the generated files.
	* src/server/admin.c: Include "conf-file.h".
	* src/server/aux-items.c: Ditto.
	* src/server/checkkomspace.c: Ditto.
	* src/server/conference.c: Ditto.
	* src/server/connections.c: Ditto.
	* src/server/dbck-cache.c: Ditto.
	* src/server/dbck.c: Ditto.
	* src/server/disk-end-of-atomic.c: Ditto.
	* src/server/komrunning.c: Ditto.
	* src/server/lockdb.c: Ditto.
	* src/server/membership.c: Ditto.
	* src/server/person.c: Ditto.
	* src/server/prot-a-output.c: Ditto.
	* src/server/prot-a-parse.c: Ditto.
	* src/server/prot-a.c: Ditto.
	* src/server/regex-match.c: Ditto.
	* src/server/rfc931.c: Ditto.
	* src/server/send-async.c: Ditto.
	* src/server/session.c: Ditto.
	* src/server/simple-cache.c: Ditto.
	* src/server/splitkomdb.c: Ditto.
	* src/server/text-garb.c: Ditto.
	* src/server/text.c: Ditto.
	* src/server/updateLysKOM.c: Ditto.
	* run-support/config: Removed the "Client port" setting.

	Adjust the test suite for the above change.
	* src/server/testsuite/lyskomd.0/bug-64.exp: Expect memory leaks
	due to bug 1599.
	* src/server/testsuite/lyskomd.0/53.exp: Expect a more informative
	error message when isc_listentcp fails.
	* src/server/testsuite/lyskomd.0/11.exp: Expect an IP address in
	the "Listening for clients" log message.
	* src/server/testsuite/config/unix.exp (clientport_2): New
	constant.
	(clientport_3): New constant.
	(lyskomd_start): New argument: listen_messages.  Use "Listen:"
	instead of "Client port:".
	(lyskomd_fail_start): Use "Listen:" instead of "Client port:".
	(client_start): New optional argument: port.

648
649
650
651
652
653
654
655
656
657
658
659
660
661
662
663
664
665
666
667
668
669
670
671
672
2006-02-09  Per Cederqvist  <ceder@lysator.liu.se>

	Added valgrind suppressions for optimised code.
	* src/server/testsuite/lyskomd.supp (aid_lexer_opt): New
	suppression.
	(aid_lexer_2_opt): Ditto.

	New test case: start lyskomd while the socket is bound.
	* src/server/testsuite/lyskomd.0/53.exp: New test case.  Test
	startup when the socket is already bound by another process.  This
	exposed a memory leak in libisc which is also fixed in this
	commit.
	* src/server/testsuite/tcpbind.py: New helper script.  Bind a tcp
	port, so that it is occupied.  Used by 53.exp.
	* src/server/testsuite/config/unix.exp (lyskomd_fail_start): New
	optional argument. expected_leaks.  Fix database setup.  Remove
	the server from expect_after and expect_always once it has failed.
	(utility_start): New function.
	(utility_death): New function.
	(talk_to): Handle talking to utilities.

	Use strerror() instead of logging a plain errno integer.
	* src/server/lockdb.c (lock_db): Use strerror() to get a
	human-readable error message.

Per Cederqvist's avatar
Per Cederqvist committed
673
674
2006-02-02  Per Cederqvist  <ceder@lysator.liu.se>

675
676
677
678
679
680
681
682
683
684
685
686
	Log when the test suite lock is obtained and released.
	* src/server/testsuite/locksuite.py (now): New function.
	(main): Include a timestamp in the "locked" and "bye" messages.
	* src/server/testsuite/config/unix.exp (obtain_lock): Log when the
	lock is obtained.  Log a warning if the expect statement should be
	exited for unexpected reasons.
	(release_lock): Log when the lock is released.

	Fix a minor memory leak.
	* src/server/conf-file.c (read_config): Fix a memory leak when the
	config file was not found.

Per Cederqvist's avatar
Per Cederqvist committed
687
688
689
690
	Compilation error fix.
	* src/server/server-config.c (dump_cfg_timevals): Updated to the
	changes in struct datatype introduced yesterday.

691
692
2006-02-01  Per Cederqvist  <ceder@lysator.liu.se>

693
694
695
696
697
698
699
700
701
702
703
704
705
706
707
708
709
710
711
712
713
	Configuration cleanup: use constructor/destructor semantics.
	* src/server/conf-file.h (struct datatype): Replaced the freer
	attribute with ctor and dtor attributes.  All users updated.
	Added some documentation.
	(free_config): New function.
	* src/server/conf-file.c (init_init): Call the constructors for
	all parameters.
	(free_config): New function, that calls the destructors for all
	parameters.
	(ctor_string): New static function. 
	(dtor_string): New static function, which replaces
	unassign_string.
	(assign_string): Expect the object to already be constructed.
	(unassign_string): Removed.
	(cf_string): Use ctor_string and dtor_string.
	* src/server/server-config.c (cf_log_param): Updated for new
	struct datatype.
	(cf_jubel): Ditto.
	(cf_ident_param): Ditto.
	(free_configuration): Use free_config to do the work.

Per Cederqvist's avatar
Per Cederqvist committed
714
715
716
717
718
719
720
721
722
723
724
725
726
727
728
729
730
	Namespace cleanup.
	* src/server/conf-file.c: Moved all struct datatype objects to the
	end of the file, to avoid having to forward-declare all of the
	static functions mentioned below.
	(assign_text_no): Now static.
	(assign_conf_no): Ditto.
	(assign_int): Ditto.
	(assign_ulong): Ditto.
	(assign_uint): Ditto.
	(assign_string): Ditto.
	(assign_bool): Ditto.
	(assign_double): Ditto.
	(assign_timeval): Ditto.
	(unassign_string): Ditto.
	* src/server/conf-file.h: Don't declare the above functions, which
	are now static.

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
	Configuration file definitions: use a datatype struct, instead of
	having separate "assigner" and "freer" fields in the parameter
	definition table.
	* src/server/conf-file.h (struct datatype): New struct.
	(struct parameter): Move the assigner and freer fields to struct
	datatype, and include a pointer to a const struct datatype.
	* src/server/conf-file.c, src/server/conf-file.h
	(cf_text_no): New constant struct.
	(cf_conf_no): Ditto.
	(cf_int): Ditto.
	(cf_ulong): Ditto.
	(cf_uint): Ditto.
	(cf_string): Ditto.
	(cf_bool): Ditto.
	(cf_double): Ditto.
	(cf_timeval): Ditto.
	* src/server/conf-file.c (assign_defaults): Handle configuration
	datatypes.
	(configure_line): Ditto.
	* src/server/server-config.c (assign_path): Removed.
	(unassign_path): Removed.
	(cf_path): New. Replaces assign_path and unassign_path.
	(cf_log_param): New const struct.
	(cf_jubel): New const struct.
	(cf_ident_param): New const struct.
	(parameters): Replaced all assigners and freers with the pointers
	to the proper datatype structs.
	(free_configuration): Handle configuration datatypes.

760
761
2006-01-04  Per Cederqvist  <ceder@lysator.liu.se>

762
763
764
765
	Made HACKING more future-proof.
	* HACKING: Refer to kom.lysator.liu.se instead of u137 (which has
	been replaced by compulsion for some time).

766
767
768
769
770
	Removed a redundant xenofarm result file.
	* scripts/xenofarm.sh: Don't create the makeinfo.txt file.  The
	information in it was already present in the id_txlog.txt file
	created by id_tx.

771
772
2006-01-03  Per Cederqvist  <ceder@lysator.liu.se>

773
774
775
776
777
	Fixed a use-after-free error found by valgrind.  This could only
	be triggered by a broken aux-item definition file, so it was
	mostly harmless.
	* src/server/aux-item-def-parse.y (aux_item_def_check_trigger):
	Don't free a string before using it.
778
	(aux_item_def_check_validate): Ditto.
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
806
807
808
809
810
811
812
813
814
815
	Fixed various harmless memory and file descriptor leaks found by
	valgrind.
	* src/server/testsuite/lyskomd.supp: Added aid_lexer_2.
	* src/server/testsuite/lyskomd.0/regexp-match-cov.exp (shutdown):
	Don't expect any suppressed leaked blocks.
	* src/server/testsuite/lyskomd.0/aux-items-cov.exp: Expect less
	suppressed leaked blocks.
	* src/server/testsuite/lyskomd.0/37.exp: Adjusted to new report
	format when configuration errors are found.
	* src/server/testsuite/lyskomd.0/conf-file-cov.exp: Ditto.
	* src/server/simple-cache.c (free_all_cache): Close text_file and
	file_a.
	* src/server/testsuite/lyskomd.supp: Removed i_fopen-2.3.1 and
	i_fopen-2.3.5, which were actually caused by the above problem.
	* src/server/server-config.c (read_configuration): Check the
	return value of read_config().  Free the configuration before
	calling restart_kom if there are any problems with the
	configuration.  Report the name of the configuation file if
	problems are found.
	* src/server/ramkomd.c (dump_exit_statistics): Call clear_info()
	instead of free_kom_info.
	(free_kom_info): Removed.  The new clear_info() function contains
	a better implementation, with less code duplication.
	* src/server/memory.c, src/server/kom-memory.h (clear_info): New
	function.
	* src/server/dbck.c (free_person_scratchpad): New static function.
	(free_person_scratch): Ditto.
	(main): Call free_person_scratch(), clear_info(),
	free_configuration() and free_all_dbck_cache() when terminating to
	make it possible to check for leaks with valgrind.
	* src/server/dbck-cache.c, src/server/dbck-cache.h
	(free_all_dbck_cache): New function.
	* src/server/conf-file.c, src/server/conf-file.h (read_config):
	Changed return type from void to Success.  Return FAILURE instead
	of calling restart_kom when configuration errors are found.

Per Cederqvist's avatar
Per Cederqvist committed
816
817
2006-01-02  Per Cederqvist  <ceder@lysator.liu.se>

818
819
820
821
822
823
824
825
826
827
	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.

828
829
830
831
	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
832
833
834
835
836
837
838
	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.

839
840
841
842
843
844
845
846
847
848
849
850
851
852
853
854
855
856
857
858
859
860
861
862
863
864
865
866
867
868
869
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
870
871
2005-12-28  Per Cederqvist  <ceder@lysator.liu.se>

872
873
874
875
876
877
878
879
880
881
882
883
884
885
886
	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
887
888
889
890
891
	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.

892
2005-12-27  Per Cederqvist  <ceder@lysator.liu.se>
893
894
895
896

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

897
898
899
900
901
902
903
904
905
906
907
908
909
910
911
912
913
	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.
914
915
916
917
918
919
920
921
922
923
924
925

	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
926
927
928
929
930
931
932
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.

933
934
935
936
937
938
939
940
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
941
942
2005-12-23  Per Cederqvist  <ceder@lysator.liu.se>

943
944
945
946
947
948
949
950
951
	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
952
953
954
955
956
957
958
959
960
961
962
963
964
965
966
967
968
969
970
971
	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.

972
973
974
975
976
977
978
979
980
981
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.

982
983
984
985
986
987
988
989
990
991
992
993
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.)

994
995
2005-12-19  Per Cederqvist  <ceder@lysator.liu.se>

Per Cederqvist's avatar
Per Cederqvist committed
996
997
998
999
1000
	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.