-
Notifications
You must be signed in to change notification settings - Fork 43
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
make a new 0.4.1 bug-fix release #142
Comments
Sure, when @chrisisonwildcode fixes that we should do a 0.4.1 I'm awaiting debian/ubuntu to flood us with bugs as well. |
After #141 is fixed, should we do a 0.4.1 release or wait a bit more? |
Don't have strong opinion for it. |
Wait a bit more. I've looked through the code and saw nothing obvious, as in the code paths for selecting a song within a multi-song file show no obvious errors. I'll be looking at the files that were sent later today and if they show nothing we've missed then I'll be gdbing the thing and tracing what's going on. At the very least I'll have an answer today, at most a fix. Sent from my iPhone
|
BTW: afritz1/OpenTESArena#38 (comment) Most of this was answered by Chris/Kcat (OpenAL-Soft dev), please give it a once over to see if he was correct? It implies that XMI support is only partial because it doesn't support a translation of XMI's looping mechanism. There are at least 2 files that seem to go on forever. One with 4~5m pause and another with an hour pause. Apparently freepats has missing instruments which might cause problems for other users of WildMIDI. |
I'm going to look into this. I'm sure we can support xmi loops converted into custom midi instructions ... Aka meta events ... Or even custom controller everts. If it can be pulled off the way I'm thinking then XMI's converted to MIDI via wildmidi would playback converted xmi loops as loops. Of course we'd need an option to ignore/enable loops for sanity. Thoughts? Sent from my iPhone
|
I can add support for xmidi controllers 116 & 117 and have WildMIDI support those from its midi output. But we're going to need a "gate keeper" for the midi output due to those controllers being undefined in midi and therefore ignored by default to avoid undesirable behaviours. We could add a text meta event "XMIDI" to be that gate keeper at the start of the file to denote xmidi controllers 116 & 117 be enabled. Again thoughts |
Possibly related? #146 Looks like there are some unimplemented (looping based) MIDI messages/events. |
From doing my work with OpenBSD, it told me the following:
Something to worry about? Good advise? Will we break something? |
strlcat warning is an openbsd advice. The rest are common advice.
If we are manipulating our buffers correctly, then there is nothing to
worry about. (That also means that it wouldn't hurt revisiting those
parts of the code and revising.)
|
Great news: https://buildd.debian.org/status/package.php?p=wildmidi 0.4.0 is finally uploaded to debian and it is built on all known platforms! (note, no tests were performed other than being built) |
On 8/16/16, Bret Curtis [email protected] wrote:
Are packages depending on libwildmidi proted properly too? |
Slomo (Debian) knows of the API changes, so that has been taken care of for us. To be fair, this is a downstream issue, but it is good to let them know ahead of time. ;) |
Should we make a 0.4.1 release? None of the open bug reports are regressions, including #141, already. |
Sorry for delays, life and all that comes with it. I'll be looking into features and bugs this week as I will have plenty of time on my hands. Hope to resolve at least #146 #149 #158 Chris Sent from my iPhone
|
rejoice! :) |
I'm looking to, with @sezero blessing, begin work on on unsf integration so that we have some support for sf2. This will likely be for 0.4.2 unless 0.4.1 is still not yet released by the time I'm ready to merge. ;) |
On 10/3/16, Bret Curtis [email protected] wrote:
My blessing (oh boy..) is irrelevant, but there is an inherent |
I was just trying to be funny. ;) Three things:
|
Any chance that we'll see those bug fixes? Or should we make a bugfix release from what we already have? |
@chrisisonwildcode I think he was referring to #146 #149 #158 |
Yes, we will see these fixes on Sunday night (AEST). I have been really busy with life, I apologise for the delay.
…Sent from my iPhone
On 24 Nov. 2016, at 6:04 pm, Bret Curtis ***@***.***> wrote:
@chrisisonwildcode I think he was referring to #146 #149 #158
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
I am dedicating this week to get stuff happening. I must point out that fixing devtest has been a priority due to it having timing issues making it harder to debug formats. Plan is to:
Any other request for the next release then please speak up. |
Do we have an ETA yet? |
Will this take forever? |
Hi,
I do apologise. To be honest with myself I haven't had the motivation lately to do much. In saying that I am getting back on track however I am having an operation in a couple of days time. The recovery period after release from hospital will give me plenty of time to work on this.
I will give you an eta of 3-6 weeks dependent on my recovery.
Again sorry for the delay.
Sincerely
Chris
…Sent from my iPhone
On 9 Jan 2017, at 6:13 am, sezero ***@***.***> wrote:
Will this take forever?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Gives me time to sneak in some stuff. ;) I'll keep myself busy with unsf. I want to get to the point where we can ask unsf for specific patches instead of dumping all of them at once. |
Nothing is happening. Should we release what we have as 0.4.1 ? |
Sorry, I know I keep saying that then something in my other life happens. If you want to release do so, I will be bug fixing as time allows. I have been working on stuff but have nothing ready.
Chris
…Sent from my iPhone
On 17 Mar 2017, at 5:25 pm, sezero ***@***.***> wrote:
Nothing is happening. Should we release what we have as 0.4.1 ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Made the 0.4.1 release; also created a new "0.4.2" milestone. Closing this one. |
We should make a 0.4.1 followup bugfix release:
One bug I fixed is commit 335dc04.
Bug #141 stands out, which is waiting for @chrisisonwildcode
The text was updated successfully, but these errors were encountered: