| Summary: | ASTERISK-10469: Delete from addons app_saycountpl | ||
| Reporter: | Igor Goncharovsky (igorg) | Labels: | |
| Date Opened: | 2007-10-08 05:34:11 | Date Closed: | 2007-10-15 02:01:02 |
| Priority: | Trivial | Regression? | No |
| Status: | Closed/Complete | Components: | Addons/General |
| Versions: | Frequency of Occurrence | ||
| Related Issues: | |||
| Environment: | Attachments: | ( 0) saycountpl-455-1.patch | |
| Description: | This patch delete from addons trivial app_saycountpl. Also documentate how to make same with simple macros. http://lists.digium.com/pipermail/asterisk-dev/2007-October/029789.html | ||
| Comments: | By: Tilghman Lesher (tilghman) 2007-10-09 11:13:04 While you are certainly welcome to add some documentation, you may NOT remove an application without deprecating its use for a release cycle, first. By: Igor Goncharovsky (igorg) 2007-10-09 20:29:01 Ok, I get it. Then I offer to deprecate this application and delete in next release for following reasons: * This is trivial pice of code * This code have no dependecies from GPL libraries, this can be rewrited and placed into asterisk * If problem of saying counting words is actual, this also actual for other the Polish language and solved by simple macros. I update proposed patch next day. By: Tilghman Lesher (tilghman) 2007-10-11 00:10:08 Are you a native Polish speaker? By: Igor Goncharovsky (igorg) 2007-10-11 00:17:20 No, but I am native Russian speaker. The principle of saying counting words is the similar. By: Igor Goncharovsky (igorg) 2007-10-11 00:24:35 Anyway, if it can not be deleted, this application must be corrected: it now use argument separation by "|". I think this bug can be closed and question about deprecating of app_saycountpl raised in maillist. By: Tilghman Lesher (tilghman) 2007-10-15 02:01:02 closed on request of reporter | ||