Fluidsynth soundfont patch

Developer's Forum, for discussion of bugs, code, and other developmental aspects of DOSBox.

Re: Fluidsynth soundfont patch

Postby Dominus » 2017-1-27 @ 22:09

It does sound as if lukeman has a dependency from somewhere else. Maybe from another dosbox built.
User avatar
Dominus
DOSBox Moderator
 
Posts: 7538
Joined: 2002-10-03 @ 09:54
Location: Vienna or Ludwigsburg

Re: Fluidsynth soundfont patch

Postby Laukku » 2017-1-28 @ 12:02

lukeman3000 wrote:
Yesterplay80 wrote:Yes and yes! ☺

After I got home I downloaded your fluidsynth build onto my desktop and tried to run dosbox.exe. This is the error message I received:

Image


I get the same error. The non-fluidsynth ECE build works fine. I have a 64-bit Windows 7 Home Premium SP1.
My YouTube account, with miscellanous DOS game stuff: http://www.youtube.com/user/LaukkuTheGreit
User avatar
Laukku
Newbie
 
Posts: 49
Joined: 2011-8-31 @ 06:27

Re: Fluidsynth soundfont patch

Postby lukeman3000 » 2017-1-29 @ 01:27

Yesterplay80 wrote:I don't have any idea why it asks for this file on your machine. I just tried my build on a completely new install of Windows 7 in a VM and it ran right from the start without asking for any other DLLs. I even searched the entire HDD for this file to make sure it isn't installed. So the problem doesn't seem to come from DOSBox. Maybe you already have another version of any of the provided DLLs installed that is looking for a libiconv-2.dll? Just in case, I'll attach the file it is looking for, just copy it in the DOSBox directory where you extraxted my build into.
libiconv-2.7z

Does someone else have this this error, too?

UPDATE: It works on a fresh Windows 10 x64 as well here.

After trying with the attached file, I now get this message:

Image
lukeman3000
Member
 
Posts: 190
Joined: 2009-3-17 @ 00:59

Re: Fluidsynth soundfont patch

Postby Yesterplay80 » 2017-1-29 @ 02:33

I don't get it. I reinstalled my notebook (W10 x64) yesterday, from scratch. Tried DOSBox and it worked without having to addd anything.
However, try this version of the dll, please:

libiconv-2.7z
(507.21 KiB) Downloaded 23 times
My full-featured DOSBox SVN builds (without debugger) for Windows: Vanilla DOSBox and DOSBox ECE (Enhanced Community Edition)
User avatar
Yesterplay80
Member
 
Posts: 297
Joined: 2016-2-23 @ 11:02
Location: Germany

Re: Fluidsynth soundfont patch

Postby lukeman3000 » 2017-1-29 @ 05:08

Yesterplay80 wrote:I don't get it. I reinstalled my notebook (W10 x64) yesterday, from scratch. Tried DOSBox and it worked without having to addd anything.
However, try this version of the dll, please:

libiconv-2.7z

Gives me the same error message
lukeman3000
Member
 
Posts: 190
Joined: 2009-3-17 @ 00:59

Re: Fluidsynth soundfont patch

Postby Yesterplay80 » 2017-1-30 @ 13:08

Please try the attached DOSBox.exe, I created it using other header files and dugans modified patch:
DOSBox.7z
(1.61 MiB) Downloaded 28 times
My full-featured DOSBox SVN builds (without debugger) for Windows: Vanilla DOSBox and DOSBox ECE (Enhanced Community Edition)
User avatar
Yesterplay80
Member
 
Posts: 297
Joined: 2016-2-23 @ 11:02
Location: Germany

Could someone make a patch for FluidLite?

Postby Yesterplay80 » 2017-2-07 @ 12:14

I tried to get FluidLite working, I could compile a binary as well, but when using fluidsynth as midi device, the music won't play. I think this is because FluidLite doesn't put out the audio to a specific driver but the application it is compiled with. Could someone maybe provide a patch to change the existing one(s) to work with FluidLite? It has several improvements over the "old" FluidSynth, like being a static library, not having all those dependencies FluidSynth has, and even SF3 support (OGG encoded SoundFonts).
My full-featured DOSBox SVN builds (without debugger) for Windows: Vanilla DOSBox and DOSBox ECE (Enhanced Community Edition)
User avatar
Yesterplay80
Member
 
Posts: 297
Joined: 2016-2-23 @ 11:02
Location: Germany

Re: Fluidsynth soundfont patch

Postby Dominus » 2017-2-07 @ 12:34

yes, that sounds neat (can't help with getting it to work, though)
User avatar
Dominus
DOSBox Moderator
 
Posts: 7538
Joined: 2002-10-03 @ 09:54
Location: Vienna or Ludwigsburg

Re: Fluidsynth soundfont patch

Postby hail-to-the-ryzen » 2017-11-27 @ 02:02

Attached fluidsynth dll binary + patch. Based on version 116 without glib dependency:
https://github.com/kirbyfan64/fluidsynth

Additional changes include those derived from above posts and:
https://github.com/FluidSynth/fluidsynth

User takes responsibility for any testing of contents.
Attachments
fluidsynth-noGlib-bin.zip
fluidsynth dll binary without glib dependency + patch
(144.93 KiB) Downloaded 19 times
hail-to-the-ryzen
Member
 
Posts: 154
Joined: 2017-3-09 @ 01:34

Re: Fluidsynth soundfont patch

Postby Dominus » 2017-11-27 @ 04:38

Thanks. Sounds as if you didn’t include the source, please add it.
User avatar
Dominus
DOSBox Moderator
 
Posts: 7538
Joined: 2002-10-03 @ 09:54
Location: Vienna or Ludwigsburg

Re: Fluidsynth soundfont patch

Postby hail-to-the-ryzen » 2017-11-27 @ 06:04

The zip archive already has source code patch for application against kirbyfan64 branch.
hail-to-the-ryzen
Member
 
Posts: 154
Joined: 2017-3-09 @ 01:34

Re: Fluidsynth soundfont patch

Postby Dominus » 2017-11-27 @ 06:24

I'm not sure under what license Fluidsynth is, but at least for GPL, a patch is not enough. So, please just add an archive with the full source you used for making this binary.
And don't get me wrong, I appreciate you doing this, I really do. But I do stand for this part of the GPL, mostly because I see the practical side of this. If you provide the source as well, it is likely that anyone can compile it even in a few years. A patch might be dead by then because you don't necessarily know for which git commit it was valid or the repository is long gone by then.
User avatar
Dominus
DOSBox Moderator
 
Posts: 7538
Joined: 2002-10-03 @ 09:54
Location: Vienna or Ludwigsburg

Re: Fluidsynth soundfont patch

Postby hail-to-the-ryzen » 2017-11-27 @ 07:30

The source code for the kirbyfan64 branch is here:
https://github.com/kirbyfan64/fluidsynt ... emoval.zip

Downloaded the archive from that above link and attached it without modification. The patch in the archive I posted above is applied to this code. Also, verified that it applies without error.
Attachments
fluidsynth-glib_removal.zip
kirbyfan64 branch of fluidsynth
(738.76 KiB) Downloaded 15 times
hail-to-the-ryzen
Member
 
Posts: 154
Joined: 2017-3-09 @ 01:34

Re: Fluidsynth soundfont patch

Postby Uriel_Dagda » 2017-12-03 @ 05:11

I tried using this, but it seems to have it's own built in MIDI soundfont, and ignores the soundfont I list in midiconfig completely.
Uriel_Dagda
Newbie
 
Posts: 17
Joined: 2013-6-21 @ 19:16

Previous

Return to DOSBox Development

Who is online

Users browsing this forum: No registered users and 1 guest