Summary: | ASTERISK-03566: [patch] app_meetme admin pin patch breaks parsing of meetme.conf | ||
Reporter: | kuj (kuj) | Labels: | |
Date Opened: | 2005-02-22 15:50:00.000-0600 | Date Closed: | 2008-01-15 15:25:55.000-0600 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Applications/app_meetme |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) meetme.txt | |
Description: | Today's patch to app_meetme.c (1.81) breaks the parsing of meetme.conf. While mochouinard's original patch for ASTERISK-2355 still "breaks up" the "conf => ..." entries at both "|" and "," seperators, looks like Mark's released CVS version does not use "," as field seperator in meetme.conf. ****** ADDITIONAL INFORMATION ****** Meetme.conf.sample documents "," as the only field seperator for the room, pin and admin pin. However, with latest patch a correct meetme.conf is ignoring "," as a seperator, resulting in "invalid conference number" messages. Too trivial to need a disclaimer. | ||
Comments: | By: Mark Spencer (markster) 2005-02-22 23:10:55.000-0600 Sorry! My mistake. By: Digium Subversion (svnbot) 2008-01-15 15:25:55.000-0600 Repository: asterisk Revision: 5067 U trunk/apps/app_meetme.c ------------------------------------------------------------------------ r5067 | markster | 2008-01-15 15:25:55 -0600 (Tue, 15 Jan 2008) | 2 lines Fix silly typo (bug ASTERISK-3566) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=5067 |