* [Bluez-devel] a2dpd: cmdplay fails: "No such application: 'amarok player play'"
@ 2007-01-15 2:05 Robert Huitl
2007-01-15 10:16 ` Frédéric DALLEAU
2007-01-15 15:23 ` Brad Midgley
0 siblings, 2 replies; 8+ messages in thread
From: Robert Huitl @ 2007-01-15 2:05 UTC (permalink / raw)
To: bluez-devel
Hi again,
the cmd* functions do not work for me:
cmdplay=dcop amarok player play
When I press the play button, the following line is print (from somewhere
inside the execlp() call):
No such application: 'amarok player play'
I hacked around this problem by using system() instead of execlp():
Index: a2dpd_ipc.c
===================================================================
RCS file: /cvsroot/bluetooth-alsa/plugz/alsa-plugins/a2dpd/a2dpd_ipc.c,v
retrieving revision 1.3
diff -u -r1.3 a2dpd_ipc.c
--- a2dpd_ipc.c 12 Jan 2007 16:34:01 -0000 1.3
+++ a2dpd_ipc.c 15 Jan 2007 02:02:15 -0000
@@ -384,8 +384,10 @@
close_server_socket_on_fork();
// Replace children with new process
- i = execlp(command, command, argv[0], NULL);
- DBG("execlp failed %s=%d (errno=%d:%s)", cmd,
i, errno, strerror(errno));
+ system(cmd);
+ exit(0);
break;
case -1:
// failed
Regards,
Robert
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Bluez-devel mailing list
Bluez-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bluez-devel
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [Bluez-devel] a2dpd: cmdplay fails: "No such application: 'amarok player play'"
2007-01-15 2:05 [Bluez-devel] a2dpd: cmdplay fails: "No such application: 'amarok player play'" Robert Huitl
@ 2007-01-15 10:16 ` Frédéric DALLEAU
2007-01-23 19:20 ` Michel Brabants
2007-01-15 15:23 ` Brad Midgley
1 sibling, 1 reply; 8+ messages in thread
From: Frédéric DALLEAU @ 2007-01-15 10:16 UTC (permalink / raw)
To: BlueZ development
Hi,
The command line is restricted to one program (dcop) and one parameter =
(amarok player play). You can overcome this issue by doing a shell =
script taking one parameter which will it turn execute your program =
using whatever parameters you want.
Fr=E9d=E9ric
Robert Huitl a =E9crit :
> Hi again,
>
> the cmd* functions do not work for me:
> cmdplay=3Ddcop amarok player play
>
> When I press the play button, the following line is print (from somewhere =
> inside the execlp() call):
> No such application: 'amarok player play'
>
> I hacked around this problem by using system() instead of execlp():
>
> Index: a2dpd_ipc.c
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
> RCS file: /cvsroot/bluetooth-alsa/plugz/alsa-plugins/a2dpd/a2dpd_ipc.c,v
> retrieving revision 1.3
> diff -u -r1.3 a2dpd_ipc.c
> --- a2dpd_ipc.c 12 Jan 2007 16:34:01 -0000 1.3
> +++ a2dpd_ipc.c 15 Jan 2007 02:02:15 -0000
> @@ -384,8 +384,10 @@
> close_server_socket_on_fork();
>
> // Replace children with new process
> - i =3D execlp(command, command, argv[0], =
NULL);
> - DBG("execlp failed %s=3D%d (errno=3D%d:%=
s)", cmd, =
> i, errno, strerror(errno));
> + system(cmd);
> + exit(0);
> break;
> case -1:
> // failed
>
>
> Regards,
> Robert
>
> -------------------------------------------------------------------------
> Take Surveys. Earn Cash. Influence the Future of IT
> Join SourceForge.net's Techsay panel and you'll get the chance to share y=
our
> opinions on IT & business topics through brief surveys - and earn cash
> http://www.techsay.com/default.php?page=3Djoin.php&p=3Dsourceforge&CID=3D=
DEVDEV
> _______________________________________________
> Bluez-devel mailing list
> Bluez-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/bluez-devel
> =
-- =
Frederic
Without the wind, the grass does not move.
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=3Djoin.php&p=3Dsourceforge&CID=3DDE=
VDEV
_______________________________________________
Bluez-devel mailing list
Bluez-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bluez-devel
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [Bluez-devel] a2dpd: cmdplay fails: "No such application: 'amarok player play'"
2007-01-15 10:16 ` Frédéric DALLEAU
@ 2007-01-23 19:20 ` Michel Brabants
2007-01-24 10:48 ` Frédéric DALLEAU
0 siblings, 1 reply; 8+ messages in thread
From: Michel Brabants @ 2007-01-23 19:20 UTC (permalink / raw)
To: bluez-devel
SGV5LAoKSSBjb21waWxlZCB0aGUgYWxzYS1ibHVldG9vdGggc29mdHdhcmUgZnJvbSBjdnMgYXQg
MjIvMS0yMDA3ICh5ZXN0ZXJkYXkpLkknbQp0cnlpbmcgdG8gdXNlIGF2cmNwLCBob3dldmVyIGl0
IGlzbid0IHdvcmtpbmcgZm9yIG1lLiBhMmRwZCBpcyByZWNlaXZpbmcKdGhlIGNvbW1hbmRzIGFu
ZCBmb3JraW5nIHRoZSBjb21tYW5kcyBJIHdhbnQgaXQgdG8gZm9yaywgYnV0IGl0IGRvZXNuJ3Qg
ZG8Kc2VlbWluZ2x5IC4uLiBJIHRyaWVkIHRoZSAiZGNvcCBhbWFyb2sgLi4uIi1jb21tYW5kcyBh
bmQKdGhlICJhbWFyb2sgLS1uZXh0Ii1jb21tYW5kIGFuZCBib3RoIGRvbid0IHdvcmsuIEhvd2V2
ZXIsIEkgZG9uJ3Qgc2VlIGFueQplcnJvcnMgYWxzbyAuLi4gSXMgdGhlcmUgYW55IG11ciBlZGVi
dWctb3V0cHV0IHRoYXQgSSBjYW4gZW5hYmxlPwpIZXJlIGlzIHdoYXQgSSBzZWUgYXQgdGhlIG1v
bWVudDoKCkEyRFBEWzAxOC45MTddOiBhMmRwX2hhbmRsZV9hdnJjcF9tZXNzYWdlOiBhdnJjcF9m
b3J3YXJkIERPV04KQTJEUERbMDE4LjkxN106IGEyZHBkX3NpZ25hbF9jb21tYW5kOiBhdnJjcF9m
b3J3YXJkCkEyRFBEWzAxOC45MTddOiBhMmRwX2hhbmRsZV9hdnJjcF9tZXNzYWdlOiBbbmV4dF0g
YW1hcm9rIC0tbmV4dApBMkRQRFswMTguOTE4XTogY2xvc2Vfc2VydmVyX3NvY2tldF9vbl9mb3Jr
OiBDbG9zaW5nIHNlcnZlciBzb2NrZXQgOApBMkRQRFswMTguOTE4XTogYXN5bmNfcnVuX3Byb2Nl
c3M6IEZvcmtlZCBhbWFyb2sgLS1uZXh0CkEyRFBEWzAxOS4wMDRdOiBhMmRwX2hhbmRsZV9hdnJj
cF9tZXNzYWdlOiBhdnJjcF9mb3J3YXJkIFVQCkEyRFBEWzAxOS4wMDRdOiBhMmRwZF9zaWduYWxf
Y29tbWFuZDogYXZyY3BfZm9yd2FyZAoKR3JlZXRpbmdzIGFuZCB0aGFuayB5b3UsCgpNaWNoZWwK
CkZyw6lkw6lyaWMgREFMTEVBVSB3cm90ZToKCj4gSGksCj4gVGhlIGNvbW1hbmQgbGluZSBpcyBy
ZXN0cmljdGVkIHRvIG9uZSBwcm9ncmFtIChkY29wKSBhbmQgb25lIHBhcmFtZXRlcgo+IChhbWFy
b2sgcGxheWVyIHBsYXkpLiBZb3UgY2FuIG92ZXJjb21lIHRoaXMgaXNzdWUgYnkgZG9pbmcgYSBz
aGVsbAo+IHNjcmlwdCB0YWtpbmcgb25lIHBhcmFtZXRlciB3aGljaCB3aWxsIGl0IHR1cm4gZXhl
Y3V0ZSB5b3VyIHByb2dyYW0KPiB1c2luZyB3aGF0ZXZlciBwYXJhbWV0ZXJzIHlvdSB3YW50Lgo+
IAo+IEZyw6lkw6lyaWMKPiAKPiBSb2JlcnQgSHVpdGwgYSDDqWNyaXQgOgo+PiBIaSBhZ2FpbiwK
Pj4KPj4gdGhlIGNtZCogZnVuY3Rpb25zIGRvIG5vdCB3b3JrIGZvciBtZToKPj4gY21kcGxheT1k
Y29wIGFtYXJvayBwbGF5ZXIgcGxheQo+Pgo+PiBXaGVuIEkgcHJlc3MgdGhlIHBsYXkgYnV0dG9u
LCB0aGUgZm9sbG93aW5nIGxpbmUgaXMgcHJpbnQgKGZyb20gc29tZXdoZXJlCj4+IGluc2lkZSB0
aGUgZXhlY2xwKCkgY2FsbCk6Cj4+IE5vIHN1Y2ggYXBwbGljYXRpb246ICdhbWFyb2sgcGxheWVy
IHBsYXknCj4+Cj4+IEkgaGFja2VkIGFyb3VuZCB0aGlzIHByb2JsZW0gYnkgdXNpbmcgc3lzdGVt
KCkgaW5zdGVhZCBvZiBleGVjbHAoKToKPj4KPj4gSW5kZXg6IGEyZHBkX2lwYy5jCj4+ID09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT0KPj4gUkNTIGZpbGU6IC9jdnNyb290L2JsdWV0b290aC1hbHNhL3BsdWd6L2Fsc2EtcGx1
Z2lucy9hMmRwZC9hMmRwZF9pcGMuYyx2Cj4+IHJldHJpZXZpbmcgcmV2aXNpb24gMS4zCj4+IGRp
ZmYgLXUgLXIxLjMgYTJkcGRfaXBjLmMKPj4gLS0tIGEyZHBkX2lwYy5jIDEyIEphbiAyMDA3IDE2
OjM0OjAxIC0wMDAwICAgICAgMS4zCj4+ICsrKyBhMmRwZF9pcGMuYyAxNSBKYW4gMjAwNyAwMjow
MjoxNSAtMDAwMAo+PiBAQCAtMzg0LDggKzM4NCwxMCBAQAo+PiAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICBjbG9zZV9zZXJ2ZXJfc29ja2V0X29uX2ZvcmsoKTsKPj4KPj4gICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgLy8gUmVwbGFjZSBjaGlsZHJlbiB3aXRoIG5ldyBw
cm9jZXNzCj4+IC0gICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIGkgPSBleGVjbHAoY29t
bWFuZCwgY29tbWFuZCwgYXJndlswXSwKPj4gTlVMTCk7Cj4+IC0gICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgIERCRygiZXhlY2xwIGZhaWxlZCAlcz0lZCAoZXJybm89JWQ6JXMpIiwKPj4g
Y21kLCBpLCBlcnJubywgc3RyZXJyb3IoZXJybm8pKTsKPj4gKyAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgc3lzdGVtKGNtZCk7Cj4+ICsgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgIGV4aXQoMCk7Cj4+ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIGJyZWFrOwo+
PiAgICAgICAgICAgICAgICAgICAgICAgICAgY2FzZSAtMToKPj4gICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgLy8gZmFpbGVkCj4+Cj4+Cj4+IFJlZ2FyZHMsCj4+ICAgUm9iZXJ0Cj4+
Cj4+IC0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t
LS0tLS0tLS0tLS0tLS0tLS0tLS0KPj4gVGFrZSBTdXJ2ZXlzLiBFYXJuIENhc2guIEluZmx1ZW5j
ZSB0aGUgRnV0dXJlIG9mIElUCj4+IEpvaW4gU291cmNlRm9yZ2UubmV0J3MgVGVjaHNheSBwYW5l
bCBhbmQgeW91J2xsIGdldCB0aGUgY2hhbmNlIHRvIHNoYXJlCj4+IHlvdXIgb3BpbmlvbnMgb24g
SVQgJiBidXNpbmVzcyB0b3BpY3MgdGhyb3VnaCBicmllZiBzdXJ2ZXlzIC0gYW5kIGVhcm4KPj4g
Y2FzaAo+PiBodHRwOi8vd3d3LnRlY2hzYXkuY29tL2RlZmF1bHQucGhwP3BhZ2U9am9pbi5waHAm
cD1zb3VyY2Vmb3JnZSZDSUQ9REVWREVWCj4+IF9fX19fX19fX19fX19fX19fX19fX19fX19fX19f
X19fX19fX19fX19fX19fX19fIEJsdWV6LWRldmVsIG1haWxpbmcgbGlzdAo+PiBCbHVlei1kZXZl
bEBsaXN0cy5zb3VyY2Vmb3JnZS5uZXQKPj4gaHR0cHM6Ly9saXN0cy5zb3VyY2Vmb3JnZS5uZXQv
bGlzdHMvbGlzdGluZm8vYmx1ZXotZGV2ZWwKPj4gICAKPiAKPiAKCgoKLS0tLS0tLS0tLS0tLS0t
LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t
LQpUYWtlIFN1cnZleXMuIEVhcm4gQ2FzaC4gSW5mbHVlbmNlIHRoZSBGdXR1cmUgb2YgSVQKSm9p
biBTb3VyY2VGb3JnZS5uZXQncyBUZWNoc2F5IHBhbmVsIGFuZCB5b3UnbGwgZ2V0IHRoZSBjaGFu
Y2UgdG8gc2hhcmUgeW91cgpvcGluaW9ucyBvbiBJVCAmIGJ1c2luZXNzIHRvcGljcyB0aHJvdWdo
IGJyaWVmIHN1cnZleXMgLSBhbmQgZWFybiBjYXNoCmh0dHA6Ly93d3cudGVjaHNheS5jb20vZGVm
YXVsdC5waHA/cGFnZT1qb2luLnBocCZwPXNvdXJjZWZvcmdlJkNJRD1ERVZERVYKX19fX19fX19f
X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KQmx1ZXotZGV2ZWwgbWFpbGlu
ZyBsaXN0CkJsdWV6LWRldmVsQGxpc3RzLnNvdXJjZWZvcmdlLm5ldApodHRwczovL2xpc3RzLnNv
dXJjZWZvcmdlLm5ldC9saXN0cy9saXN0aW5mby9ibHVlei1kZXZlbAo=
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Bluez-devel] a2dpd: cmdplay fails: "No such application: 'amarok player play'"
2007-01-23 19:20 ` Michel Brabants
@ 2007-01-24 10:48 ` Frédéric DALLEAU
2007-01-25 19:02 ` Michel Brabants
0 siblings, 1 reply; 8+ messages in thread
From: Frédéric DALLEAU @ 2007-01-24 10:48 UTC (permalink / raw)
To: michel.brabants, BlueZ development
SGVsbG8sCgpJdCBzZWVtcyB0byBmb3JrIGNvcnJlY3RseSBhbmQgYW1hcm9rIGRvIG5vdCBkaXNw
bGF5IGFueSBlcnJvcnMuIEkgCnN1Z2dlc3QgeW91IHRyeSB0aGUgJ2FtYXJvayAtLW5leHQnIGZy
b20gdGhlIHNhbWUgc2hlbGwgeW91IHN0YXJ0ZWQgYTJkcCAKZnJvbSB0byBtYWtlIHN1cmUgaWYg
aXQgZWZmZWN0aXZlbHkgd29yay4KTm93IGlmIHlvdSByZXRyaWV2ZSB0aGUgbGF0ZXN0IGN2cywg
dGhlIGRjb3Agc3R5bGUgY29tbWFuZCB3aWxsIHdvcmsgCihzZWUgc2FtcGxlLmEyZHByYykuCgpN
aWNoZWwgQnJhYmFudHMgYSDDqWNyaXQgOgo+IEhleSwKPgo+IEkgY29tcGlsZWQgdGhlIGFsc2Et
Ymx1ZXRvb3RoIHNvZnR3YXJlIGZyb20gY3ZzIGF0IDIyLzEtMjAwNyAoeWVzdGVyZGF5KS5JJ20K
PiB0cnlpbmcgdG8gdXNlIGF2cmNwLCBob3dldmVyIGl0IGlzbid0IHdvcmtpbmcgZm9yIG1lLiBh
MmRwZCBpcyByZWNlaXZpbmcKPiB0aGUgY29tbWFuZHMgYW5kIGZvcmtpbmcgdGhlIGNvbW1hbmRz
IEkgd2FudCBpdCB0byBmb3JrLCBidXQgaXQgZG9lc24ndCBkbwo+IHNlZW1pbmdseSAuLi4gSSB0
cmllZCB0aGUgImRjb3AgYW1hcm9rIC4uLiItY29tbWFuZHMgYW5kCj4gdGhlICJhbWFyb2sgLS1u
ZXh0Ii1jb21tYW5kIGFuZCBib3RoIGRvbid0IHdvcmsuIEhvd2V2ZXIsIEkgZG9uJ3Qgc2VlIGFu
eQo+IGVycm9ycyBhbHNvIC4uLiBJcyB0aGVyZSBhbnkgbXVyIGVkZWJ1Zy1vdXRwdXQgdGhhdCBJ
IGNhbiBlbmFibGU/Cj4gSGVyZSBpcyB3aGF0IEkgc2VlIGF0IHRoZSBtb21lbnQ6Cj4KPiBBMkRQ
RFswMTguOTE3XTogYTJkcF9oYW5kbGVfYXZyY3BfbWVzc2FnZTogYXZyY3BfZm9yd2FyZCBET1dO
Cj4gQTJEUERbMDE4LjkxN106IGEyZHBkX3NpZ25hbF9jb21tYW5kOiBhdnJjcF9mb3J3YXJkCj4g
QTJEUERbMDE4LjkxN106IGEyZHBfaGFuZGxlX2F2cmNwX21lc3NhZ2U6IFtuZXh0XSBhbWFyb2sg
LS1uZXh0Cj4gQTJEUERbMDE4LjkxOF06IGNsb3NlX3NlcnZlcl9zb2NrZXRfb25fZm9yazogQ2xv
c2luZyBzZXJ2ZXIgc29ja2V0IDgKPiBBMkRQRFswMTguOTE4XTogYXN5bmNfcnVuX3Byb2Nlc3M6
IEZvcmtlZCBhbWFyb2sgLS1uZXh0Cj4gQTJEUERbMDE5LjAwNF06IGEyZHBfaGFuZGxlX2F2cmNw
X21lc3NhZ2U6IGF2cmNwX2ZvcndhcmQgVVAKPiBBMkRQRFswMTkuMDA0XTogYTJkcGRfc2lnbmFs
X2NvbW1hbmQ6IGF2cmNwX2ZvcndhcmQKPgo+ICAgCgpGcsOpZMOpcmljCgoKLS0tLS0tLS0tLS0t
LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t
LS0tLQpUYWtlIFN1cnZleXMuIEVhcm4gQ2FzaC4gSW5mbHVlbmNlIHRoZSBGdXR1cmUgb2YgSVQK
Sm9pbiBTb3VyY2VGb3JnZS5uZXQncyBUZWNoc2F5IHBhbmVsIGFuZCB5b3UnbGwgZ2V0IHRoZSBj
aGFuY2UgdG8gc2hhcmUgeW91cgpvcGluaW9ucyBvbiBJVCAmIGJ1c2luZXNzIHRvcGljcyB0aHJv
dWdoIGJyaWVmIHN1cnZleXMgLSBhbmQgZWFybiBjYXNoCmh0dHA6Ly93d3cudGVjaHNheS5jb20v
ZGVmYXVsdC5waHA/cGFnZT1qb2luLnBocCZwPXNvdXJjZWZvcmdlJkNJRD1ERVZERVYKX19fX19f
X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KQmx1ZXotZGV2ZWwgbWFp
bGluZyBsaXN0CkJsdWV6LWRldmVsQGxpc3RzLnNvdXJjZWZvcmdlLm5ldApodHRwczovL2xpc3Rz
LnNvdXJjZWZvcmdlLm5ldC9saXN0cy9saXN0aW5mby9ibHVlei1kZXZlbAo=
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Bluez-devel] a2dpd: cmdplay fails: "No such application: 'amarok player play'"
2007-01-24 10:48 ` Frédéric DALLEAU
@ 2007-01-25 19:02 ` Michel Brabants
0 siblings, 0 replies; 8+ messages in thread
From: Michel Brabants @ 2007-01-25 19:02 UTC (permalink / raw)
To: bluez-devel
SGVsbG8sCgpJIGNvbXBpbGVkIGEyZHBkIHllc3RlcmRheSBJIHRoaW5rLiBBbnl3YXksIHRoZSB2
ZXJzaW9uIHdhc24ndCB0aGUgcHJvYmxlbQp3aXRoICJhbWFyb2sgLS1uZXh0Ii4gSSBoYWQgdG8g
ZmlsbCBpbiB0aGUgY29tcGxldGUgcGF0aCB0byB0aGUgZXhhY3V0YWJsZS4KU28gaXQgc2hvdWxk
IGhhdmUgYmVlbiAiL29wdC9rZGUvYmluL2FtYXJvayAtLW5leHQiIGZvciBtZS4gSSdtIHVzaW5n
IHRoZQpkY29wLWNvbW1hbmRzIG5vdyBhbnl3YXksIGJ1dCBhbHNvIHdpdGggdGhlIGZ1bGwgcGF0
aCBmcm9tIHRoZQpyb290LWRpcmVjdG9yeS4gVGh1cywgIi9vcHQva2RlL2Jpbi9kY29wIC4uLiIu
IFRoaXMgd29ya3MuIFRoZXJlIGlzIG5vClBBVEgtdmFyaWFibGUgZGVmaW5lZCBmb3IgdGhlIG5l
dyBwcm9jZXNzL3RocmVhZC8uLi4KClRoYW5rIHlvdSBhbmQgZ3JlZXRpbmdzLAoKTWljaGVsCgpG
csOpZMOpcmljIERBTExFQVUgd3JvdGU6Cgo+IEhlbGxvLAo+IAo+IEl0IHNlZW1zIHRvIGZvcmsg
Y29ycmVjdGx5IGFuZCBhbWFyb2sgZG8gbm90IGRpc3BsYXkgYW55IGVycm9ycy4gSQo+IHN1Z2dl
c3QgeW91IHRyeSB0aGUgJ2FtYXJvayAtLW5leHQnIGZyb20gdGhlIHNhbWUgc2hlbGwgeW91IHN0
YXJ0ZWQgYTJkcAo+IGZyb20gdG8gbWFrZSBzdXJlIGlmIGl0IGVmZmVjdGl2ZWx5IHdvcmsuCj4g
Tm93IGlmIHlvdSByZXRyaWV2ZSB0aGUgbGF0ZXN0IGN2cywgdGhlIGRjb3Agc3R5bGUgY29tbWFu
ZCB3aWxsIHdvcmsKPiAoc2VlIHNhbXBsZS5hMmRwcmMpLgo+IAo+IE1pY2hlbCBCcmFiYW50cyBh
IMOpY3JpdCA6Cj4+IEhleSwKPj4KPj4gSSBjb21waWxlZCB0aGUgYWxzYS1ibHVldG9vdGggc29m
dHdhcmUgZnJvbSBjdnMgYXQgMjIvMS0yMDA3Cj4+ICh5ZXN0ZXJkYXkpLkknbSB0cnlpbmcgdG8g
dXNlIGF2cmNwLCBob3dldmVyIGl0IGlzbid0IHdvcmtpbmcgZm9yIG1lLgo+PiBhMmRwZCBpcyBy
ZWNlaXZpbmcgdGhlIGNvbW1hbmRzIGFuZCBmb3JraW5nIHRoZSBjb21tYW5kcyBJIHdhbnQgaXQg
dG8KPj4gZm9yaywgYnV0IGl0IGRvZXNuJ3QgZG8gc2VlbWluZ2x5IC4uLiBJIHRyaWVkIHRoZSAi
ZGNvcCBhbWFyb2sKPj4gLi4uIi1jb21tYW5kcyBhbmQgdGhlICJhbWFyb2sgLS1uZXh0Ii1jb21t
YW5kIGFuZCBib3RoIGRvbid0IHdvcmsuCj4+IEhvd2V2ZXIsIEkgZG9uJ3Qgc2VlIGFueSBlcnJv
cnMgYWxzbyAuLi4gSXMgdGhlcmUgYW55IG11ciBlZGVidWctb3V0cHV0Cj4+IHRoYXQgSSBjYW4g
ZW5hYmxlPyBIZXJlIGlzIHdoYXQgSSBzZWUgYXQgdGhlIG1vbWVudDoKPj4KPj4gQTJEUERbMDE4
LjkxN106IGEyZHBfaGFuZGxlX2F2cmNwX21lc3NhZ2U6IGF2cmNwX2ZvcndhcmQgRE9XTgo+PiBB
MkRQRFswMTguOTE3XTogYTJkcGRfc2lnbmFsX2NvbW1hbmQ6IGF2cmNwX2ZvcndhcmQKPj4gQTJE
UERbMDE4LjkxN106IGEyZHBfaGFuZGxlX2F2cmNwX21lc3NhZ2U6IFtuZXh0XSBhbWFyb2sgLS1u
ZXh0Cj4+IEEyRFBEWzAxOC45MThdOiBjbG9zZV9zZXJ2ZXJfc29ja2V0X29uX2Zvcms6IENsb3Np
bmcgc2VydmVyIHNvY2tldCA4Cj4+IEEyRFBEWzAxOC45MThdOiBhc3luY19ydW5fcHJvY2Vzczog
Rm9ya2VkIGFtYXJvayAtLW5leHQKPj4gQTJEUERbMDE5LjAwNF06IGEyZHBfaGFuZGxlX2F2cmNw
X21lc3NhZ2U6IGF2cmNwX2ZvcndhcmQgVVAKPj4gQTJEUERbMDE5LjAwNF06IGEyZHBkX3NpZ25h
bF9jb21tYW5kOiBhdnJjcF9mb3J3YXJkCj4+Cj4+ICAgCj4gCj4gRnLDqWTDqXJpYwo+IAo+IAo+
IC0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t
LS0tLS0tLS0tLS0tLS0tLS0KPiBUYWtlIFN1cnZleXMuIEVhcm4gQ2FzaC4gSW5mbHVlbmNlIHRo
ZSBGdXR1cmUgb2YgSVQKPiBKb2luIFNvdXJjZUZvcmdlLm5ldCdzIFRlY2hzYXkgcGFuZWwgYW5k
IHlvdSdsbCBnZXQgdGhlIGNoYW5jZSB0byBzaGFyZQo+IHlvdXIgb3BpbmlvbnMgb24gSVQgJiBi
dXNpbmVzcyB0b3BpY3MgdGhyb3VnaCBicmllZiBzdXJ2ZXlzIC0gYW5kIGVhcm4KPiBjYXNoCj4g
aHR0cDovL3d3dy50ZWNoc2F5LmNvbS9kZWZhdWx0LnBocD9wYWdlPWpvaW4ucGhwJnA9c291cmNl
Zm9yZ2UmQ0lEPURFVkRFVgo+IF9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f
X19fX19fX19fIEJsdWV6LWRldmVsIG1haWxpbmcgbGlzdAo+IEJsdWV6LWRldmVsQGxpc3RzLnNv
dXJjZWZvcmdlLm5ldAo+IGh0dHBzOi8vbGlzdHMuc291cmNlZm9yZ2UubmV0L2xpc3RzL2xpc3Rp
bmZvL2JsdWV6LWRldmVsCgoKCi0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t
LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0KVGFrZSBTdXJ2ZXlzLiBFYXJuIENh
c2guIEluZmx1ZW5jZSB0aGUgRnV0dXJlIG9mIElUCkpvaW4gU291cmNlRm9yZ2UubmV0J3MgVGVj
aHNheSBwYW5lbCBhbmQgeW91J2xsIGdldCB0aGUgY2hhbmNlIHRvIHNoYXJlIHlvdXIKb3Bpbmlv
bnMgb24gSVQgJiBidXNpbmVzcyB0b3BpY3MgdGhyb3VnaCBicmllZiBzdXJ2ZXlzIC0gYW5kIGVh
cm4gY2FzaApodHRwOi8vd3d3LnRlY2hzYXkuY29tL2RlZmF1bHQucGhwP3BhZ2U9am9pbi5waHAm
cD1zb3VyY2Vmb3JnZSZDSUQ9REVWREVWCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f
X19fX19fX19fX19fX19fCkJsdWV6LWRldmVsIG1haWxpbmcgbGlzdApCbHVlei1kZXZlbEBsaXN0
cy5zb3VyY2Vmb3JnZS5uZXQKaHR0cHM6Ly9saXN0cy5zb3VyY2Vmb3JnZS5uZXQvbGlzdHMvbGlz
dGluZm8vYmx1ZXotZGV2ZWwK
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Bluez-devel] a2dpd: cmdplay fails: "No such application: 'amarok player play'"
2007-01-15 2:05 [Bluez-devel] a2dpd: cmdplay fails: "No such application: 'amarok player play'" Robert Huitl
2007-01-15 10:16 ` Frédéric DALLEAU
@ 2007-01-15 15:23 ` Brad Midgley
2007-01-15 16:43 ` Frédéric DALLEAU
1 sibling, 1 reply; 8+ messages in thread
From: Brad Midgley @ 2007-01-15 15:23 UTC (permalink / raw)
To: BlueZ development
Robert
this idea looks ok. The extra fork() could be removed.
Brad
> Index: a2dpd_ipc.c
> ===================================================================
> RCS file: /cvsroot/bluetooth-alsa/plugz/alsa-plugins/a2dpd/a2dpd_ipc.c,v
> retrieving revision 1.3
> diff -u -r1.3 a2dpd_ipc.c
> --- a2dpd_ipc.c 12 Jan 2007 16:34:01 -0000 1.3
> +++ a2dpd_ipc.c 15 Jan 2007 02:02:15 -0000
> @@ -384,8 +384,10 @@
> close_server_socket_on_fork();
>
> // Replace children with new process
> - i = execlp(command, command, argv[0], NULL);
> - DBG("execlp failed %s=%d (errno=%d:%s)", cmd,
> i, errno, strerror(errno));
> + system(cmd);
> + exit(0);
> break;
> case -1:
> // failed
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Bluez-devel mailing list
Bluez-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bluez-devel
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [Bluez-devel] a2dpd: cmdplay fails: "No such application: 'amarok player play'"
2007-01-15 15:23 ` Brad Midgley
@ 2007-01-15 16:43 ` Frédéric DALLEAU
2007-01-16 3:43 ` Brad Midgley
0 siblings, 1 reply; 8+ messages in thread
From: Frédéric DALLEAU @ 2007-01-15 16:43 UTC (permalink / raw)
To: BlueZ development
Brad,
I already tried system here and this led to strange interactions between =
processes. I just tried it again (in case it would be better) and it is =
still the same.
Here is the procedure to reproduce :
- xmms must not be started.
- start a2dpd.
- then from an avrcp headset press play to connect : xmms will start.
- kill a2dpd : xmms will deadlock.
Any player should behave the same. I found that the a2dpd main unix =
socket must be closed immediatly after any fork(). I suspect the socket =
remain open in the xmms process space which will try to write to an open =
socket that is no longer polled for data, thus deadlock (btw, can any =
linux guru confirm?). Any process started by system() will be forked but =
this socket won't be closed.
More over, the execution must be asynchronous, system("xmms") will wait =
until you terminate xmms before returning, system("xmms &") should work.
It is still possible to fork() then close_server_socket() then system() =
as did Robert, but a shell is spawned for that (+ 2 additionnal fork =
calls!) which is a few megabytes that could be used better...
It should be enough to rewrite the parser code just before so it handles =
more parameters.
char* argv[2];
strncpy(command, cmd, sizeof(command));
command[sizeof(command)-1]=3D '\0';
argv[0] =3D strchr(command, ' ');
if(argv[0]) { *argv[0]=3D'\0'; argv[0]++; }
I stay open to any other solutions if this limitation is a real problem, =
but as stated in amarok man page :
Options:
-r, --previous
Skip backwards in playlist
-p, --play
Start playing current playlist
-s, --stop
Stop playback
--pause
Pause playback
-f, --next
Skip forwards in playlist
Fr=E9d=E9ric
Brad Midgley a =E9crit :
> Robert
>
> this idea looks ok. The extra fork() could be removed.
>
> Brad
>
> =
>> Index: a2dpd_ipc.c
>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>> RCS file: /cvsroot/bluetooth-alsa/plugz/alsa-plugins/a2dpd/a2dpd_ipc.c,v
>> retrieving revision 1.3
>> diff -u -r1.3 a2dpd_ipc.c
>> --- a2dpd_ipc.c 12 Jan 2007 16:34:01 -0000 1.3
>> +++ a2dpd_ipc.c 15 Jan 2007 02:02:15 -0000
>> @@ -384,8 +384,10 @@
>> close_server_socket_on_fork();
>>
>> // Replace children with new process
>> - i =3D execlp(command, command, argv[0],=
NULL);
>> - DBG("execlp failed %s=3D%d (errno=3D%d:=
%s)", cmd, =
>> i, errno, strerror(errno));
>> + system(cmd);
>> + exit(0);
>> break;
>> case -1:
>> // failed
>> =
>
> -------------------------------------------------------------------------
> Take Surveys. Earn Cash. Influence the Future of IT
> Join SourceForge.net's Techsay panel and you'll get the chance to share y=
our
> opinions on IT & business topics through brief surveys - and earn cash
> http://www.techsay.com/default.php?page=3Djoin.php&p=3Dsourceforge&CID=3D=
DEVDEV
> _______________________________________________
> Bluez-devel mailing list
> Bluez-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/bluez-devel
> =
-- =
Frederic
Without the wind, the grass does not move.
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=3Djoin.php&p=3Dsourceforge&CID=3DDE=
VDEV
_______________________________________________
Bluez-devel mailing list
Bluez-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bluez-devel
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2007-01-25 19:02 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-01-15 2:05 [Bluez-devel] a2dpd: cmdplay fails: "No such application: 'amarok player play'" Robert Huitl
2007-01-15 10:16 ` Frédéric DALLEAU
2007-01-23 19:20 ` Michel Brabants
2007-01-24 10:48 ` Frédéric DALLEAU
2007-01-25 19:02 ` Michel Brabants
2007-01-15 15:23 ` Brad Midgley
2007-01-15 16:43 ` Frédéric DALLEAU
2007-01-16 3:43 ` Brad Midgley
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox