[Home]

Summary:ASTERISK-07631: [patch][post 1.4] Patch to make Background() return a status code in the same way Playback() does.
Reporter:Nick Barnes (bcnit)Labels:
Date Opened:2006-08-30 06:26:40Date Closed:2006-09-27 15:03:57
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) patch.txt.txt
Description:Firstly, I'm not a programmer, so please excuse sloppy coding or misunderstandings, but if somebody could run with this I'd be a happy bunny!

The Playback() application returns ${PLAYBACKSTATUS} (either "FAILED" or "SUCCESS"), but the Background() core feature does not.

Attached is a diff against pbx.c which implements the return of ${BACKGROUNDSTATUS} (either "FAILED" or "SUCCESS") in the same way Playback() does.

I've tested it insofar as I can on a production machine and it works for me. Is there any way this can be implemented in Asterisk going forwards?
Comments:By: Serge Vecher (serge-v) 2006-08-30 10:21:54

1) looks like you've done your patch backwards
2) since this is a new feature, it needs to be against trunk, not 1.2.x release.

Thanks.

By: Nick Barnes (bcnit) 2006-08-31 09:47:51

1) Doh. Told you I wasn't a programmer. Diff the right way round uploaded.
2) OK. I'll see what I can do.

By: Serge Vecher (serge-v) 2006-08-31 10:15:04

3) also, please fax a disclaimer in to Digium. See bottom of this page http://bugs.digium.com/main_page.php

By: Nick Barnes (bcnit) 2006-08-31 11:14:26

3) Done.

By: Jason Parker (jparker) 2006-09-27 15:03:56

Similar patch committed to svn trunk in revision 43814.