Re: [PD] Error: invalid command name pd
Hi Jonathan, OK, here is a rundown of what I did to get tot working. Remember that I added elements FROM the svn version TO the downloadable verision from Krzystof's site. in the miXed/shared/hammer directory: gui.c needed all pd [concat etc] statements changed to pdsend [concat etc] as was advised. file.c and file.h: I changed the function void hammereditor_open(t_hammerfile *f, char *title); to add the third argument: void hammereditor_open(t_hammerfile *f, char *title, char *owner); and copied the relevant function from the svn version into file.c, overwriting the version that was there. Also, in the downloaded tarball from Krzystof's site (I hope I got his name right) there was an extra function I added to file.h and file.c: void hammereditor_setdirty(t_hammerfile *f, int flag); from the svn version. NOTE: I am only pasting the file.h declarations, so you'll have to check the files out for yourself. But I am including a tarball of only the files I changed. I'm pretty sure that's all I did, but the annoying thing about the download was that cyclone had its own separate copy of the shared/ directory, so each file had to be copied into it as well. That's why you'll find a duplicated of the files in the cyclone folder, where none exists in the svn folder (again - I've removed everything I didn't have to change). Also, I made sure I unzipped both the downloaded cyclone source file and the downloaded toxy source file into a folder I called miXed somewhere else from the svn copy on my hard drive. I don't know if it made any difference, but they were in the same folder I guess. That's what I did. I didn't have to edit the Makefile.common in the root miXed directory, as I should expect since my m_pd.h, m_imp.h and g_canvas.h files I always copy to /usr/include when I install a new version of Pd. But I also notice that in the download versions of cyclone and toxy there are copies of m_pd.h and m_imp.h floating around! Cheers, Ed Ninja Jamm - a revolutionary new music remix app from Ninja Tune and Seeper, for iPhone and iPad http://www.ninjajamm.com/ Gemnotes-0.2: Live music notation for Pure Data, now with dynamics! http://sharktracks.co.uk/ From: Jonathan Wilkes jancs...@yahoo.com To: Ed Kelly morph_2...@yahoo.co.uk Cc: pd-list@iem.at pd-list@iem.at Sent: Saturday, 17 August 2013, 16:17 Subject: Re: [PD] Error: invalid command name pd On 08/16/2013 09:44 PM, Ed Kelly wrote: Hi Jonathan, I did that, but the SVN code has been hacked in a way where [tot] doesn't work. My hacks on the other hand (in the email miXed remix earlier - tarball enclosed) make [tot] work and other things are broken. The files gui.c, file.c and file.h in miXed/shared/hammer and the file scriptlet.c in miXed/shared/toxy are the files that needed pd [concat etc etc] changing to pdsend [concat etc etc]. My tot works with Pd 0.44 and that means I can do some gigs soon with my tracker patches. That's good enough for me, for now, but it would be great if we could get the whole of miXed fixed up to the latest Pd standards. What are the difference between your tarball and svn for the code that is shared among the various libraries in miXed? -Jonathan I don't want to go near widget though - I got it to work once only, in 2005 on a G5 mac! Cheers, Ed miXed_changeFiles.tar.gz Description: GNU Zip compressed data ___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list
Re: [PD] Error: invalid command name pd
On 08/16/2013 09:44 PM, Ed Kelly wrote: Hi Jonathan, I did that, but the SVN code has been hacked in a way where [tot] doesn't work. My hacks on the other hand (in the email miXed remix earlier - tarball enclosed) make [tot] work and other things are broken. The files gui.c, file.c and file.h in miXed/shared/hammer and the file scriptlet.c in miXed/shared/toxy are the files that needed pd [concat etc etc] changing to pdsend [concat etc etc]. My tot works with Pd 0.44 and that means I can do some gigs soon with my tracker patches. That's good enough for me, for now, but it would be great if we could get the whole of miXed fixed up to the latest Pd standards. What are the difference between your tarball and svn for the code that is shared among the various libraries in miXed? -Jonathan I don't want to go near widget though - I got it to work once only, in 2005 on a G5 mac! Cheers, Ed ___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list
Re: [PD] Error: invalid command name pd
I think my problem with this, and when I tried it last time it was the same, is that I get these compile errors even after I've pointed the PD_DIR setting to either: /root/software/pd/src/pd-0.44-0 or /root/software/pd/src/pd-0.44-0/src When using miXed from the subversion repository. I notice if I download cyclone from http://suita.chopin.edu.pl/~czaja/miXed/externs/cyclone.html and toxy from http://suita.chopin.edu.pl/~czaja/miXed/externs/toxy.html ...they compile OK. So there is something wrong with the svn version of miXed perhaps? Now I still get my tcl errors from [tot] but I can have a go at fixing the tcl code now. Cheers, Ed I think the problem is with [tot .] from the miXed library. Does anyone know how to compile it? It's causing me a headache. Typing make from within the miXed library from the svn repository comes up with errors where it can't seem to find basic functions, e.g. some lines tot.c:(.text+0xbcf): undefined reference to `gensym' tot.c:(.text+0xbff): undefined reference to `canvas_getrootfor' tot.c:(.text+0xc17): undefined reference to `canvas_getrootfor' /tmp/ccctv252.o: In function `tot_getcanvas': tot.c:(.text+0xc5b): undefined reference to `canvas_class' tot.c:(.text+0xc69): undefined reference to `pd_findbyclass' tot.c:(.text+0xc79): undefined reference to `glist_getcanvas' tot.c:(.text+0xcb7): undefined reference to `loud_warning' tot.c:(.text+0xd48): undefined reference to `loud_error' some more lines Look inside Makefile.common and change the line near (or at) the top that sets PD_DIR. It looks like it's supposed to point to the src directory, but I'm not absolutely sure about that. then try make clean and make from inside the miXed directory. If that doesn't work, add to the line in Makefile.common that starts with INCLUDES = . You'll want to add a something like this: I/path/to/the/place/where/m_pd.h/lives Then make clean and make. Now it should create everything without error. All your binaries will be located inside miXed/bin, including [tot] which I guess is what you're after. At this point, you have to do this: 1) Run your patch 2) Note the errors and place in the tcl code where they happen 3) From the miXed directory, grep -r part of the erroneous code that calls pd instead of pdsend 4) Note the filename that grep returns 5) Go to the corresponding line of the offending file and change pd to pdsend. Also, I think the old pd tcl proc had an escaped colon after it, whereas the pdsend proc doesn't require one. I'm not sure whether that makes a difference or not. 6) make clean and make 7) repeat until you don't get any errors. If you're feeling generous, do this for all the toxy c files as well as the test files, until the entire library actually works with pd-0.44, then diff against the old toxy and post so we can update the code. Let me know if you have any problems with getting your patch to work. -Jonathan It gives similar outpuit for tot, and for pddplink ? Ed -Jonathan ___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list
Re: [PD] Error: invalid command name pd
On 08/16/2013 03:12 AM, Ed Kelly wrote: I think my problem with this, and when I tried it last time it was the same, is that I get these compile errors even after I've pointed the PD_DIR setting to either: Did you also add to the line that begins INCLUDES = ? That did the trick for me. -Jonathan /root/software/pd/src/pd-0.44-0 or /root/software/pd/src/pd-0.44-0/src When using miXed from the subversion repository. I notice if I download cyclone from http://suita.chopin.edu.pl/~czaja/miXed/externs/cyclone.html and toxy from http://suita.chopin.edu.pl/~czaja/miXed/externs/toxy.html ...they compile OK. So there is something wrong with the svn version of miXed perhaps? Now I still get my tcl errors from [tot] but I can have a go at fixing the tcl code now. Cheers, Ed I think the problem is with [tot .] from the miXed library. Does anyone know how to compile it? It's causing me a headache. Typing make from within the miXed library from the svn repository comes up with errors where it can't seem to find basic functions, e.g. some lines tot.c:(.text+0xbcf): undefined reference to `gensym' tot.c:(.text+0xbff): undefined reference to `canvas_getrootfor' tot.c:(.text+0xc17): undefined reference to `canvas_getrootfor' /tmp/ccctv252.o: In function `tot_getcanvas': tot.c:(.text+0xc5b): undefined reference to `canvas_class' tot.c:(.text+0xc69): undefined reference to `pd_findbyclass' tot.c:(.text+0xc79): undefined reference to `glist_getcanvas' tot.c:(.text+0xcb7): undefined reference to `loud_warning' tot.c:(.text+0xd48): undefined reference to `loud_error' some more lines Look inside Makefile.common and change the line near (or at) the top that sets PD_DIR. It looks like it's supposed to point to the src directory, but I'm not absolutely sure about that. then try make clean and make from inside the miXed directory. If that doesn't work, add to the line in Makefile.common that starts with INCLUDES = . You'll want to add a something like this: I/path/to/the/place/where/m_pd.h/lives Then make clean and make. Now it should create everything without error. All your binaries will be located inside miXed/bin, including [tot] which I guess is what you're after. At this point, you have to do this: 1) Run your patch 2) Note the errors and place in the tcl code where they happen 3) From the miXed directory, grep -r part of the erroneous code that calls pd instead of pdsend 4) Note the filename that grep returns 5) Go to the corresponding line of the offending file and change pd to pdsend. Also, I think the old pd tcl proc had an escaped colon after it, whereas the pdsend proc doesn't require one. I'm not sure whether that makes a difference or not. 6) make clean and make 7) repeat until you don't get any errors. If you're feeling generous, do this for all the toxy c files as well as the test files, until the entire library actually works with pd-0.44, then diff against the old toxy and post so we can update the code. Let me know if you have any problems with getting your patch to work. -Jonathan It gives similar outpuit for tot, and for pddplink ? Ed -Jonathan ___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list
Re: [PD] Error: invalid command name pd
On 07/30/13 21:03, Ed Kelly wrote: I think the problem is with [tot .] from the miXed library. afaik tot is broken with pd=0.44. tot heavily uses the olde gui infrastructure, which simply isn't there any no more :-( Oh damn! I spent ages building my tracker-style sequencers and getting them to work with tot ;( your best bet would be to move to a gui-plugin (at least that's what i have done for my projects using [tot]) What do you mean a gui-plugin? Is this something new, or some other technique for getting the extended keyboard and mouse data from a window? Please tell all... Cheers, Ed gfmadsr IOhannes ___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list ___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list
Re: [PD] Error: invalid command name pd
On 07/30/2013 03:03 PM, Ed Kelly wrote: First click Details on your error message and post the output. Doh! Of course. Sorry, I naturally distrust error reports having been brought up on Window$. Error messages are all the same in my patch: invalid command name pd invalid command name pd while executing pd [concat #hammergui _vised .x988db98.c 1 \;] invoked from within if {[hammergui_ispatcher .x988db98.c]} {pd [concat #hammergui _vised .x988db98.c 1 \;]} (command bound to event) I think the problem is with [tot .] from the miXed library. Does anyone know how to compile it? It's causing me a headache. Typing make from within the miXed library from the svn repository comes up with errors where it can't seem to find basic functions, e.g. some lines tot.c:(.text+0xbcf): undefined reference to `gensym' tot.c:(.text+0xbff): undefined reference to `canvas_getrootfor' tot.c:(.text+0xc17): undefined reference to `canvas_getrootfor' /tmp/ccctv252.o: In function `tot_getcanvas': tot.c:(.text+0xc5b): undefined reference to `canvas_class' tot.c:(.text+0xc69): undefined reference to `pd_findbyclass' tot.c:(.text+0xc79): undefined reference to `glist_getcanvas' tot.c:(.text+0xcb7): undefined reference to `loud_warning' tot.c:(.text+0xd48): undefined reference to `loud_error' some more lines Look inside Makefile.common and change the line near (or at) the top that sets PD_DIR. It looks like it's supposed to point to the src directory, but I'm not absolutely sure about that. then try make clean and make from inside the miXed directory. If that doesn't work, add to the line in Makefile.common that starts with INCLUDES = . You'll want to add a something like this: I/path/to/the/place/where/m_pd.h/lives Then make clean and make. Now it should create everything without error. All your binaries will be located inside miXed/bin, including [tot] which I guess is what you're after. At this point, you have to do this: 1) Run your patch 2) Note the errors and place in the tcl code where they happen 3) From the miXed directory, grep -r part of the erroneous code that calls pd instead of pdsend 4) Note the filename that grep returns 5) Go to the corresponding line of the offending file and change pd to pdsend. Also, I think the old pd tcl proc had an escaped colon after it, whereas the pdsend proc doesn't require one. I'm not sure whether that makes a difference or not. 6) make clean and make 7) repeat until you don't get any errors. If you're feeling generous, do this for all the toxy c files as well as the test files, until the entire library actually works with pd-0.44, then diff against the old toxy and post so we can update the code. Let me know if you have any problems with getting your patch to work. -Jonathan It gives similar outpuit for tot, and for pddplink ? Ed -Jonathan ___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list
Re: [PD] Error: invalid command name pd
Oh, and if you do try this, make sure you only change the pd tcl commands and not pd message selectors: pd [concat pd init blah blah blah Above, the first pd is a tcl command and should be changed to pdsend. However, the second pd is part of a string that gets interpreted by the core as the selector of a message (i.e., the receive symbol pd). The second part should be left as is. But if you happen to accidentally change a selector you'll probably get an error to the Pd console, pdsend: no such object. -Jonathan ___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list
Re: [PD] Error: invalid command name pd
On 07/30/13 21:03, Ed Kelly wrote: I think the problem is with [tot .] from the miXed library. afaik tot is broken with pd=0.44. tot heavily uses the olde gui infrastructure, which simply isn't there any no more :-( your best bet would be to move to a gui-plugin (at least that's what i have done for my projects using [tot]) gfmadsr IOhannes signature.asc Description: OpenPGP digital signature ___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list
Re: [PD] Error: invalid command name pd
First click Details on your error message and post the output. Doh! Of course. Sorry, I naturally distrust error reports having been brought up on Window$. Error messages are all the same in my patch: invalid command name pd invalid command name pd while executing pd [concat #hammergui _vised .x988db98.c 1 \;] invoked from within if {[hammergui_ispatcher .x988db98.c]} {pd [concat #hammergui _vised .x988db98.c 1 \;]} (command bound to event) I think the problem is with [tot .] from the miXed library. Does anyone know how to compile it? It's causing me a headache. Typing make from within the miXed library from the svn repository comes up with errors where it can't seem to find basic functions, e.g. some lines tot.c:(.text+0xbcf): undefined reference to `gensym' tot.c:(.text+0xbff): undefined reference to `canvas_getrootfor' tot.c:(.text+0xc17): undefined reference to `canvas_getrootfor' /tmp/ccctv252.o: In function `tot_getcanvas': tot.c:(.text+0xc5b): undefined reference to `canvas_class' tot.c:(.text+0xc69): undefined reference to `pd_findbyclass' tot.c:(.text+0xc79): undefined reference to `glist_getcanvas' tot.c:(.text+0xcb7): undefined reference to `loud_warning' tot.c:(.text+0xd48): undefined reference to `loud_error' some more lines It gives similar outpuit for tot, and for pddplink ? Ed -Jonathan ___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list
Re: [PD] Error: invalid command name pd
Then, it's just missing m_pd.h. You'll have to modify the makefile to set the include path or copy your m_pd.h into the same directory as tot.c (or maybe just set an environment variable... I don't know for sure). Chuck On Tue, Jul 30, 2013 at 2:03 PM, Ed Kelly morph_2...@yahoo.co.uk wrote: First click Details on your error message and post the output. Doh! Of course. Sorry, I naturally distrust error reports having been brought up on Window$. Error messages are all the same in my patch: invalid command name pd invalid command name pd while executing pd [concat #hammergui _vised .x988db98.c 1 \;] invoked from within if {[hammergui_ispatcher .x988db98.c]} {pd [concat #hammergui _vised .x988db98.c 1 \;]} (command bound to event) I think the problem is with [tot .] from the miXed library. Does anyone know how to compile it? It's causing me a headache. Typing make from within the miXed library from the svn repository comes up with errors where it can't seem to find basic functions, e.g. some lines tot.c:(.text+0xbcf): undefined reference to `gensym' tot.c:(.text+0xbff): undefined reference to `canvas_getrootfor' tot.c:(.text+0xc17): undefined reference to `canvas_getrootfor' /tmp/ccctv252.o: In function `tot_getcanvas': tot.c:(.text+0xc5b): undefined reference to `canvas_class' tot.c:(.text+0xc69): undefined reference to `pd_findbyclass' tot.c:(.text+0xc79): undefined reference to `glist_getcanvas' tot.c:(.text+0xcb7): undefined reference to `loud_warning' tot.c:(.text+0xd48): undefined reference to `loud_error' some more lines It gives similar outpuit for tot, and for pddplink ? Ed -Jonathan ___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list ___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list
Re: [PD] Error: invalid command name pd
Looks more like the cyclone lib. I've succeeded to update it by simply replacing every pd with pdsend - but I haven't tested it extensively. cheers, jan On 30.7.13 21:03 , Ed Kelly wrote: First click Details on your error message and post the output. Doh! Of course. Sorry, I naturally distrust error reports having been brought up on Window$. Error messages are all the same in my patch: invalid command name pd invalid command name pd while executing pd [concat #hammergui _vised .x988db98.c 1 \;] invoked from within if {[hammergui_ispatcher .x988db98.c]} {pd [concat #hammergui _vised .x988db98.c 1 \;]} (command bound to event) I think the problem is with [tot .] from the miXed library. Does anyone know how to compile it? It's causing me a headache. Typing make from within the miXed library from the svn repository comes up with errors where it can't seem to find basic functions, e.g. some lines tot.c:(.text+0xbcf): undefined reference to `gensym' tot.c:(.text+0xbff): undefined reference to `canvas_getrootfor' tot.c:(.text+0xc17): undefined reference to `canvas_getrootfor' /tmp/ccctv252.o: In function `tot_getcanvas': tot.c:(.text+0xc5b): undefined reference to `canvas_class' tot.c:(.text+0xc69): undefined reference to `pd_findbyclass' tot.c:(.text+0xc79): undefined reference to `glist_getcanvas' tot.c:(.text+0xcb7): undefined reference to `loud_warning' tot.c:(.text+0xd48): undefined reference to `loud_error' some more lines It gives similar outpuit for tot, and for pddplink ? Ed -Jonathan ___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list ___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list
[PD] Error: invalid command name pd
Can anyone tell me what's causing this? Screenshot enclosed. Vanilla 0.44 on Debian Wheezy, with libraries...big patches with lots of GOP Ed Ninja Jamm - a revolutionary new music remix app from Ninja Tune and Seeper, for iPhone and iPad http://www.ninjajamm.com/ Gemnotes-0.2: Live music notation for Pure Data, now with dynamics! http://sharktracks.co.uk/ attachment: error.png___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list
Re: [PD] Error: invalid command name pd
On 07/28/2013 09:39 AM, Ed Kelly wrote: Can anyone tell me what's causing this? Screenshot enclosed. Probably an external that does its own messaging. Probably messaging between the GUI and pd. See if you're using any non-Vanilla GUI object classes. Messages from the gui like pd audio-properties got changed to pdsend audio-properties with the big GUI code remix awhile back. This _breaks_ user patches like yours and requires you or someone else to go in and make a permanent change to the source of the offending external. (And then that external will no longer work with old versions of Pd that expect pd instead of pdsend. I guess I'll go ahead and say that you should view all non-Vanilla GUI classes as expiremental and subject to arbitrary change, because they all use tcl/tk code inside that will probably break the next time someone separates more GUI from the core. -Jonathan Vanilla 0.44 on Debian Wheezy, with libraries...big patches with lots of GOP Ed Ninja Jamm - a revolutionary new music remix app from Ninja Tune and Seeper, for iPhone and iPad http://www.ninjajamm.com/ Gemnotes-0.2: Live music notation for Pure Data, now with dynamics! http://sharktracks.co.uk/ ___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list ___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list
Re: [PD] Error: invalid command name pd
Aha. So I probably need to go through a process of elimination to discover which external is causing this, and then maybe we can fix it relatively simply by changing the pd to pdsend in the source code? I dread to say anything like relatively simple - but these things needs to be sorted out incrementally. Will report back when I find anything... Cheers, Ed From: Jonathan Wilkes jancs...@yahoo.com To: pd-list@iem.at Sent: Sunday, 28 July 2013, 17:08 Subject: Re: [PD] Error: invalid command name pd On 07/28/2013 09:39 AM, Ed Kelly wrote: Can anyone tell me what's causing this? Screenshot enclosed. Probably an external that does its own messaging. Probably messaging between the GUI and pd. See if you're using any non-Vanilla GUI object classes. Messages from the gui like pd audio-properties got changed to pdsend audio-properties with the big GUI code remix awhile back. This _breaks_ user patches like yours and requires you or someone else to go in and make a permanent change to the source of the offending external. (And then that external will no longer work with old versions of Pd that expect pd instead of pdsend. I guess I'll go ahead and say that you should view all non-Vanilla GUI classes as expiremental and subject to arbitrary change, because they all use tcl/tk code inside that will probably break the next time someone separates more GUI from the core. -Jonathan Vanilla 0.44 on Debian Wheezy, with libraries...big patches with lots of GOP Ed Ninja Jamm - a revolutionary new music remix app from Ninja Tune and Seeper, for iPhone and iPad http://www.ninjajamm.com/ Gemnotes-0.2: Live music notation for Pure Data, now with dynamics! http://sharktracks.co.uk/ ___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list ___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list ___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list
Re: [PD] Error: invalid command name pd
From: Ed Kelly morph_2...@yahoo.co.uk To: Jonathan Wilkes jancs...@yahoo.com; pd-list@iem.at pd-list@iem.at Sent: Sunday, July 28, 2013 12:15 PM Subject: Re: [PD] Error: invalid command name pd Aha. So I probably need to go through a process of elimination to discover which external is causing this, and then maybe we can fix it relatively simply by changing the pd to pdsend in the source code? First click Details on your error message and post the output. -Jonathan___ Pd-list@iem.at mailing list UNSUBSCRIBE and account-management - http://lists.puredata.info/listinfo/pd-list