Open RetroSven opened 6 years ago
I'll look into it. I'm not very familiar with programming on Linux so it may take me a little while. I also have to setup my Linux system again so that will also take some time.
I'll get back to you in a few days.
Daniel.
From: Sven notifications@github.com Sent: 21 September 2018 03:13 To: M3wP/XSID Cc: Subscribed Subject: [M3wP/XSID] static libs x64 linux build (#1)
Will you please make the included so libs in the x64 linux build static libraries instead of dynamic? I don't have the version of libstdc++ you used to compile and the program will therefore not run. Thanks!
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/M3wP/XSID/issues/1, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AJtEd6ZVrNBJwiFsWCKLQhfyj6ERO6LMks5uc8yegaJpZM4WykC6.
Hmm... Just looking at this now. The programs actually require quite a few libraries. I don't see libstdc++ in the list but I do see libc.
I think it may be easier to compile it for yourself on your system so that you will have the required libraries (you will need to install Lazarus and a few other things).
Or, perhaps I can compile it for your system myself if you can tell me what you are running? Which version of libc do you have?
Daniel.
From: Sven notifications@github.com Sent: 21 September 2018 03:13 To: M3wP/XSID Cc: Subscribed Subject: [M3wP/XSID] static libs x64 linux build (#1)
Will you please make the included so libs in the x64 linux build static libraries instead of dynamic? I don't have the version of libstdc++ you used to compile and the program will therefore not run. Thanks!
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/M3wP/XSID/issues/1, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AJtEd6ZVrNBJwiFsWCKLQhfyj6ERO6LMks5uc8yegaJpZM4WykC6.
Ahh! Of course! Now I understand where your problem comes from. Sorry!
It would be in either LibReSID.so or LibSIDPlay.so. They were written in C++. I didn't even think of this, initially. You even seem to mention it, now that I look again. I didn't write those so I didn't think of them right away (I only modified them for my purposes). That's what I get for trying to do this stuff at 02:00.
I'll look at it again in a little while.
Daniel.
From: Sven notifications@github.com Sent: 21 September 2018 03:13 To: M3wP/XSID Cc: Subscribed Subject: [M3wP/XSID] static libs x64 linux build (#1)
Will you please make the included so libs in the x64 linux build static libraries instead of dynamic? I don't have the version of libstdc++ you used to compile and the program will therefore not run. Thanks!
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/M3wP/XSID/issues/1, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AJtEd6ZVrNBJwiFsWCKLQhfyj6ERO6LMks5uc8yegaJpZM4WykC6.
Okay, I've done what you asked and made libC++ statically linked in libReSIDFP.so and libSIDPlay.so for the Linux x64 distribution.
There was also another problem with XSIDPlay in that distribution. I think it was a x86 version instead of the x64 version. I'm not entirely sure. I updated it and tested it and it works for me, now.
I'm concerned that you may still have problems because of a missing libGCC_s. Let me know if it works for you now, though.
Daniel.
Thanks much - I can now run the programs (at least SIDConvert - which is the only one I've tried so far). A few more issues, though - will open separate tickets.
Thanks again!
There was just one additional issue as it turns out (entered in different ticket). At first I thought something strange was going on with the SID to XSID process because I was getting seemingly the same sized output XSID files regardless of how long the time selection override was. It turns out the length of the SID as it appears in the HVSC song length DB is wrong.
Anyway everything works great - it does a better job of conversion than sid2midi which had a resulting output MID with many garbled notes near the end of the songs I've been converting.
If you have the time and are so inclined, what do you think about adding the following CLI-friendly features?
SIDConvert : ability to run from command line (no GUI) - add command line parameters -i input_filename -t sid_track_number -l song_length_in_seconds -c compression_mode -o output_filename
e.g.:
SIDConvert -i Legacy_of_the_Ancients.sid -t 1 -l 156 -c 0 -o Legacy_of_the_Ancients.xsid
compression_mode 0 = none, 1 = deflate, 2 = lz (or just drop compression mode and default to 0/none)
Similar for XSIDToMIDI
XSIDToMIDI -i Legacy_of_the_Ancients.xsid -o Legacy_of_the_Ancients.mid
which would convert based on all the default values you're currently using for XSIDToMIDI
Great stuff - very useful :)
Will you please make the included so libs in the x64 linux build static libraries instead of dynamic? I don't have the version of libstdc++ you used to compile and the program will therefore not run. Thanks!