Re: [fluid-dev] Adding Poly/mono functionality to FluidSynth

[jean-jacques.ceresa]

Hi Ben and RL Horn,

Thanks to R.L Horn to adapt the polymono patch the right way.
I am sorry for the wasted time. I am reading the patch coming from
http://lists.nongnu.org/archive/html/fluid-dev/2016-07/msg00011.html
to understand the right way:

What i understand  is
assuming,
./fluidsynth-1.1.6/src   the original fluidynth directory source tree 
(i.e coming from git repository).
./fluidsynth-1.1.6_xx/src   the modified fluidynth directory source 
tree. (i.e original files modified or new files added).

1) To add new files in the sources tree (eg: 
./fluidsynth-1.1.6_xx/src/synth/f1.c,f2.c)
we need to edit the file:  ./fluidsynth-1.1.6_xx/src/CmakeLists.txt like 
this:

In the section starting: set ( libfluidsynth_SOURCES, add the following 
lines: synth/f1.c  synth/f2.c

2) Then we need to use  the following diff command
diif    -Nur    ./fluidsynth-1.1.6/src   ./fluidsynth-1.1.6_xx/src > 
./new_patch_xx.patch

is it sufficient ?

Many thanks for your advise

jjc

Le 01/07/2016 14:22, jean-jacques.ceresa a écrit :
> Hi,
>
> Theses 2 new files have be forgotten: fluid_synth_mono.c, 
> fluid_synth_polymono.c.
> Theses are parts of the complete poylymono-patch.
>
> For the 2 news file (fluid_synth_mono.c, fluid_synth_polymono.c) , i 
> assume that it sufficient to put them in
> the initial source directory fluid_synth-1.1.6/src/synth to get them 
> compiled but i am not sure as it depends on themake command.
>
> Because you put theses new files in the initial source directory, you 
> need to re-execute commands "cmake" and "make".
>
> Note: Thanks for sending the complete steps you have already done with 
> the "patch command".
>
> jjc
>
> Le 01/07/2016 12:41, Ben Gonzales a écrit :
> > OK, I fixed the error below, I then ran cmake and make, and got an 
> > error.
> >
> > See the full make log on:
> >
> > http://gonzos.net/ewi/makelog.txt
> >
> > Suggestions please...
> >
> > Ben
> >
> > On 01/07/16 20:08, Ben Gonzales wrote:
> > > Pardonnez moi, je devrais avoir expliqué (high-school French)
> > >
> > > I did the patch successfully (I think) on all files except the 
> > > synth.h where I got the error:
> > > "Invalid cross-device link "
> > >
> > > I don't know what it means.
> > >
> > > Ben
> > >
> > > On 01/07/16 19:49, jean-jacques.ceresa wrote:
> > > > >Tried that,
> > > > Sorry, as you don't explain what you are trying i can't understand ?
> > > > jjc
> > > >
> > > > Le 01/07/2016 10:30, Ben Gonzales a écrit :
> > > > > Aah... found it
> > > > >
> > > > > Tried that, but got this:
> > > > > ../include/fluidsynth/synth.h: Invalid cross-device link
> > > > >
> > > > > Help...
> > > > >
> > > > > Ben
> > > > >
> > > > >
> > > > >
> > > > > On 01/07/16 17:35, jean-jacques.ceresa wrote:
> > > > > > Hi,
> > > > > > >Hmmm... I can't find the synth.h file (the last one actioned in 
> > > > > > the patch file). I patched the rest of them OK.
> > > > > >
> > > > > > This is the main header file for API in the following path: 
> > > > > > fluidsynth-1.1.6/include/fluidsynth.
> > > > > > If anyone want to write an application using fluidsynth library, 
> > > > > > this file is the only one that expose fluidsynth functions API.
> > > > > > The Poly/mono patch have added new API functions prototypes in 
> > > > > > this file.
> > > > > >
> > > > > > Please, note all the steps you are doing to apply this patch. It 
> > > > > > will be very useful to write a documentation procedure for news 
> > > > > > coming.
> > > > > > Thanks for your contribution.
> > > > > >
> > > > > > jjc
> > > > > >
> > > > > > Le 01/07/2016 02:32, Ben Gonzales a écrit :
> > > > > > > Hmmm... I can't find the synth.h file (the last one actioned in 
> > > > > > > the patch file). I patched the rest of them OK.
> > > > > > >
> > > > > > > Ben
> > > > > > > > > >> Hi, Ben
> > > > > > > > > >> 1)After a quick search on the Web, the command to apply a 
> > > > > > > > > patch is
> > > > > > > > > >> simply 'patch'.
> > > > > > > > > >> You need to apply this command on each original source file 
> > > > > > > > > concerned
> > > > > > > > > >> (the list is given in the pdf).
> > > > > > > > > >>
> > > > > > >
> > > > > > >
> > > > > > > _______________________________________________
> > > > > > > fluid-dev mailing list
> > > > > > > address@hidden
> > > > > > > https://lists.nongnu.org/mailman/listinfo/fluid-dev
> > > > > >
> > > > > >
> > > > > > _______________________________________________
> > > > > > fluid-dev mailing list
> > > > > > address@hidden
> > > > > > https://lists.nongnu.org/mailman/listinfo/fluid-dev
> > > > >
> > > > >
> > > > > _______________________________________________
> > > > > fluid-dev mailing list
> > > > > address@hidden
> > > > > https://lists.nongnu.org/mailman/listinfo/fluid-dev
> > > >
> > > >
> > > > _______________________________________________
> > > > fluid-dev mailing list
> > > > address@hidden
> > > > https://lists.nongnu.org/mailman/listinfo/fluid-dev
> >
> >
> > _______________________________________________
> > fluid-dev mailing list
> > address@hidden
> > https://lists.nongnu.org/mailman/listinfo/fluid-dev
>
>
> _______________________________________________
> fluid-dev mailing list
> address@hidden
> https://lists.nongnu.org/mailman/listinfo/fluid-dev