<div dir="ltr"><div>Hi Jonathan,</div><div><br></div><div>thanks for clearing that up. I've rebuilt the package with 0.46.3 now (on Arch x86_64). Most simple patches load and work fine with it, but lots of the more advanced stuff is broken. E.g., the About patch comes up empty and (adding insult to injury) the window cannot be closed any more, giving the error message shown in the screenshot:</div><div><br></div><div><div><img src="cid:ii_kc251th00" alt="image.png" width="476" height="300"><br></div><div><br></div><div>In fact, drawing of data structures seems to be completely broken (not just the purr-data svg drawing extensions, but plain old vanilla data structures, too).</div><div><br></div><div>Gem also stopped working. Even the most basic examples crash Purr Data (segfault) as soon as you try to create a Gem window.</div><div><br></div><div>This is all stuff that worked fine in rev. 4b7b73cd from 2 weeks ago.</div><div><br></div><div>I hate to say it, but these latest changes are not ready for prime time yet. IMHO, they absolutely need to be moved back to their own feature branch and tested out *much* more thoroughly before they can be merged again. Right now, if anyone wants to try the latest from the master branch, she can only conclude that Purr Data is broken (at least on Linux, I didn't do any testing on Mac or Windows). That's not the impression we want to give, do we?<br></div><div><br></div><div>I can understand that you want to move on with these things, but if a changeset is highly experimental and carries with it so much potential for breakage, then it needs to be given some serious regression testing. Please don't break the master branch, that's what feature branches are for. ;-)</div><div><br></div><div>To get us out of here quickly and easily, you might just branch off a new "stable" branch at rev. 4b7b73cd and present that as our new official main branch to users (and also turn that inthe default branch of the repo)? Whereas "master" will become our development branch?<br></div></div><div><br></div><div>Albert</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jun 30, 2020 at 2:14 PM Jonathan Wilkes <<a href="mailto:jon.w.wilkes@gmail.com">jon.w.wilkes@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tue, Jun 30, 2020 at 6:30 AM Albert Graef <<a href="mailto:aggraef@gmail.com" target="_blank">aggraef@gmail.com</a>> wrote:<br>
><br>
> Hi Sam,<br>
><br>
> No, I'm using 0.24.4, which is the default version of nw.js on Linux that's supported in our build system. In any case, I'm using the same version as with the last release and with the last-known-working revision.<br>
><br>
> Perhaps I've missed a stealth update of the nw.js version in recent times? ;-) If so, then the update *must* be reflected in the build system, and we should all be very cautious about all kinds of possible regressions due to the version update.<br>
<br>
So my plan was to do a smaller bugfix release yesterday with a few<br>
patches which Ico had told me don't depend on a<br>
newer version of nw.js.<br>
<br>
But his "patchset from hell" has inscrutable interdependencies on each<br>
other. So even those merges<br>
still required the updated version.<br>
<br>
I can confirm running HEAD against a local 0.46.3 nw.js on aarch64<br>
does indeed work to load and display patches.<br>
<br>
I'll update the binaries and build script.<br>
<br>
On a related note-- apparently "feature-branch" isn't enough to convey<br>
the dev process I want here.<br>
<br>
What do I need to add to the contributor's guide to make it clear what<br>
a desirable merge request branch should look like?<br>
<br>
-Jonathan<br>
<br>
<br>
><br>
> Best,<br>
> Albert<br>
><br>
><br>
> On Tue, Jun 30, 2020 at 11:42 AM Sam Thursfield <<a href="mailto:ssssam@gmail.com" target="_blank">ssssam@gmail.com</a>> wrote:<br>
>><br>
>> Hi Albert,<br>
>><br>
>> On Tue, Jun 30, 2020 at 9:12 AM Albert Graef <<a href="mailto:aggraef@gmail.com" target="_blank">aggraef@gmail.com</a>> wrote:<br>
>> > The program still builds fine, launches and I can still open new patch windows (^n), but "About Pd-L2ork" doesn't work any more and I can't open existing patches either (apparently the patches do get opened in the engine, but no window is mapped).<br>
>><br>
>> Is it possible that you are using a version of nw.js >= 0.42.3 ?<br>
>> This issue sounds a bit like<br>
>> <a href="https://git.purrdata.net/jwilkes/purr-data/-/issues/572" rel="noreferrer" target="_blank">https://git.purrdata.net/jwilkes/purr-data/-/issues/572</a><br>
>> Sam<br>
>> _______________________________________________<br>
>> L2Ork-dev mailing list<br>
>> <a href="mailto:L2Ork-dev@disis.music.vt.edu" target="_blank">L2Ork-dev@disis.music.vt.edu</a><br>
>> <a href="https://disis.music.vt.edu/listinfo/l2ork-dev" rel="noreferrer" target="_blank">https://disis.music.vt.edu/listinfo/l2ork-dev</a><br>
><br>
><br>
><br>
> --<br>
> Dr. Albert Gr"af<br>
> Computer Music Research Group, JGU Mainz, Germany<br>
> Email: <a href="mailto:aggraef@gmail.com" target="_blank">aggraef@gmail.com</a>, web: <a href="https://agraef.github.io/" rel="noreferrer" target="_blank">https://agraef.github.io/</a><br>
> _______________________________________________<br>
> L2Ork-dev mailing list<br>
> <a href="mailto:L2Ork-dev@disis.music.vt.edu" target="_blank">L2Ork-dev@disis.music.vt.edu</a><br>
> <a href="https://disis.music.vt.edu/listinfo/l2ork-dev" rel="noreferrer" target="_blank">https://disis.music.vt.edu/listinfo/l2ork-dev</a><br>
_______________________________________________<br>
L2Ork-dev mailing list<br>
<a href="mailto:L2Ork-dev@disis.music.vt.edu" target="_blank">L2Ork-dev@disis.music.vt.edu</a><br>
<a href="https://disis.music.vt.edu/listinfo/l2ork-dev" rel="noreferrer" target="_blank">https://disis.music.vt.edu/listinfo/l2ork-dev</a></blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr">Dr. Albert Gr"af<br>Computer Music Research Group, JGU Mainz, Germany<br>Email: <a href="mailto:aggraef@gmail.com" target="_blank">aggraef@gmail.com</a>, web: <a href="https://agraef.github.io/" target="_blank">https://agraef.github.io/</a></div></div></div></div></div></div>