Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Plugins
  • Assigned To No-one
  • Operating System SW-codec
  • Severity Low
  • Priority Very Low
  • Reported Version Version 3.2
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by Wincent Balin - 2009-05-16
Last edited by Peter D'Hoye - 2009-05-22

FS#10214 - Pure Data for Rockbox plugin

This is a GSoC 2009 (and beyond) project about porting Pure Data audio framework to Rockbox. With a simple GUI around it will allow to turn a DAP into a portable synthesizer, effect device or any audio processing gadget you wish.

Closed by  Peter D'Hoye
2009-05-22 21:59
Reason for closing:  Accepted
Additional comments about closing:  

and now, your changes.. :)

Peter D'Hoye commented on 2009-05-18 10:13

great! I’ll check it as soon as I can…

Dominik Wenger commented on 2009-05-18 16:20

A few general Comments. I will take a deeper look, when i find time.

- The patch isnt complete, all the unmodified files from PDa (and others) are missing in the patch. (you have to svn add them, to get them included into svn diff)
- You should add a README.rockbox which contains the details about the imported code (exact version, licence, etc) and your modifications.
- It would be a good Idea to divide that into a few patches. Like one patch for unmodified PDa (not enabled for compiling) then a patch (or many) with the different modifications needed for compiling, etc.

Wincent Balin commented on 2009-05-18 17:15

Here are the patches for dbestfit and PDa, both unmodified.

Peter D'Hoye commented on 2009-05-20 22:19

Wincent, the patch misses the addition into the rockbox make system (adding your patches doesn’t cause them to get build)

If I do that (see attached patch) it turns out this code doesn’t build and even misses some files:

CC apps/plugins/pdbox/pdbox.c
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/pdbox.c: In function `plugin_start’:
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/pdbox.c:102: warning: unused variable `mem_size’ /home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/pdbox.c:103: warning: unused variable `mem_pool’ /home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/pdbox.c:106: warning: unused variable `filename’ CC apps/plugins/pdbox/pdbox-net.c
CC apps/plugins/pdbox/PDa/src/m_fixed.c
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:2:24: sys/socket.h: No such file or directory
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:3:24: netinet/in.h: No such file or directory
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:4:25: netinet/tcp.h: No such file or directory
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:5:19: netdb.h: No such file or directory
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:25: warning: function declaration isn’t a prototype
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c: In function `ipod_connect’:
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:26: error: storage size of ‘server’ isn’t known
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:39: warning: implicit declaration of function `socket’ /home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:39: error: `AF_INET’ undeclared (first use in this function)
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:39: error: (Each undeclared identifier is reported only once
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:39: error: for each function it appears in.)
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:39: error: `SOCK_DGRAM’ undeclared (first use in this function)
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:43: warning: implicit declaration of function `sys_sockerror’ /home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:50: warning: implicit declaration of function `gethostbyname’ /home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:50: warning: assignment makes pointer from integer without a cast
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:57: warning: implicit declaration of function `memcpy’ /home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:57: error: dereferencing pointer to incomplete type
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:57: error: dereferencing pointer to incomplete type
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:59: warning: implicit declaration of function `htons’ /home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:59: error: `u_short’ undeclared (first use in this function)
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:59: error: syntax error before “portno” /home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:60: warning: implicit declaration of function `connect’ /home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:63: warning: implicit declaration of function `sys_closesocket’ /home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:26: warning: unused variable `server’ /home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:31: warning: unused variable `intarg’ /home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c: In function `ipod_bang’:
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:75: warning: implicit declaration of function `sprintf’ /home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:76: warning: implicit declaration of function `send’ /home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:76: warning: implicit declaration of function `strlen’ /home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c: In function `pd_checkgui’:
/home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:111: warning: implicit declaration of function `strncmp’ /home/peter/rockbox/gsoc/2009/dev1/apps/plugins/pdbox/PDa/src/m_fixed.c:112: warning: implicit declaration of function `strcmp’ make: *** [/home/peter/rockbox/gsoc/2009/dev1/build/apps/plugins/pdbox/PDa/src/m_fixed.o] Error 1

Wincent Balin commented on 2009-05-20 23:44

Here is the patch for the build system

Wincent Balin commented on 2009-05-21 00:17

Here is the correct version of the pdbox initial patch

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing