<div dir="ltr">Hi, thanks for doing this, I appreciate it.<div><br></div><div>For those not aware, this has been a long discussion, with many turns and lots of new information to me. Let me summarize.</div><div><br></div><div>When trying to merge cyclone, the libloader failed, cause we had both a path and a library with the same name to load. As I see it, the options here are:</div><div>1- fix the loader</div><div>2- get rid of the library for purr data (we can have a workaround with hexloader, and this would also mean a separate branch)</div><div>3- make cyclone into a single library named cyclone</div><div>4- rename the library</div><div><br></div><div>But why do we have now both a path and a library? The library is for loading operators with non-alphanumeric characters, an issue that is only solvable in vanilla with a library, or a third party external dependency (hexloader). The reason we have both it and a set of single binaries is that we inherited the single binaries from the last maintenance phase. </div><div><br></div><div>If I'm not mistaken, cyclone used to come as a single library binary, but got chopped off into separate binaries in Pd Extended because of some Pd-Extended issue I don't know about. </div><div><br></div><div>So, on to my opinion on the solutions. I've listed them in order of personal preference, kinda... </div><div><br></div><div>Fixing the loader is option number 1, it would introduce the least change, and would bring benefits of improving the loader system. </div><div><br></div><div>A second option would be to create a branch in Purr Data that just loads files with hexloader (like it is doing with zexy). This is actually something jonathan proposed first. This forces the creation of a separate branch for the Purr Data help files and all, but I don't think it is that much of a hassle and I could work on that, no problem.</div><div><br></div><div>The most complicated issue, on my opinion, is thinking about what to do with the "operators library". This just opens a can of worms in my head... and I'm sorry if it is hard for me to take a decision about it, I'm just not sure which is the best one. </div><div><br></div><div>One issue is that I already forced a name change. In cyclone 0.2 it was called 'nettles', I hated that name to the guts (I never knew what it meant, it was never part of cyclone before, I felt it was hard to relate that name to cyclone), so I thought it was best to revert back to the original "cyclone" library (which used to call these objects).</div><div><br></div><div>I actually also have an issue on the need of these two part deal, loading cyclone both as a set of separate binaries, where you need to include cyclone's path, but also a library only for a subset of objects. That was never a good design decision. It would be preferable if it'd be just one thing, either all separate binaries, or just a single binary pack.</div><div><br></div><div>Again, vanilla can't load non-alphanumeric, so that's out. I'm dreaming we could try and make this into vanilla, but I should get real...</div><div><br></div><div>and well, I never discarded the idea of making cyclone into a single binary pack. That would make it simpler, cleaner, and would also fix the loading issue in Purr Data. So it feels like a good option. </div><div><br></div><div>Renaming the library yet once again is something that would be a last resource kind of thing for me. In an outburst of impulse and anxiety, I did propose and changed the library name to 'cyclops' (standing for 'cyclone operators'), but jonathan felt this could potentially cause name clashes with user defined abstractions and proposes "cyclone_ops" - one can also consider going back to 'nettles' (yikes). One way or another, again, touching the cyclone name is the most delicate issue to me, and it's what gives me more headaches.</div><div><br></div><div>So, yeah, I'd love to hear more opinions, please Derek and Matt, share your cents, and anybody else on this list. I'll keep thinking about this in the meantime.</div><div><br></div><div>cheers</div></div><div class="gmail_extra"><br><div class="gmail_quote">2017-06-14 15:56 GMT-03:00 Jonathan Wilkes <span dir="ltr"><<a href="mailto:jancsika@yahoo.com" target="_blank">jancsika@yahoo.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div style="color:#000;background-color:#fff;font-family:Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif;font-size:13px"><div>Actually, let me clarify to avoid a digression--</div><div><br></div><div>This vote isn't about whether pd-cyclone should accept the pull request.</div><div id="m_-7400375510780695963yui_3_16_0_1_1497465999456_8203"><br></div><div id="m_-7400375510780695963yui_3_16_0_1_1497465999456_8204">This vote is about whether Purr Data should merge pd-cyclone with the "cyclops" binary name, or <br></div><div dir="ltr">the one from the revision with the "cyclone_ops" name.</div><div dir="ltr" id="m_-7400375510780695963yui_3_16_0_1_1497465999456_8229"><br></div><div dir="ltr" id="m_-7400375510780695963yui_3_16_0_1_1497465999456_8230">Thanks,</div><div dir="ltr" id="m_-7400375510780695963yui_3_16_0_1_1497465999456_8231">Jonathan<br></div><div class="hm HOEnZb"><div id="m_-7400375510780695963yui_3_16_0_1_1497465999456_8191"><span></span></div><div class="m_-7400375510780695963qtdSeparateBR" id="m_-7400375510780695963yui_3_16_0_1_1497465999456_8190"><br><br></div></div><div class="m_-7400375510780695963yahoo_quoted" id="m_-7400375510780695963yui_3_16_0_1_1497465999456_8184" style="display:block"><div class="hm HOEnZb">  </div><div style="font-family:Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif;font-size:13px" id="m_-7400375510780695963yui_3_16_0_1_1497465999456_8183"><div class="hm HOEnZb"> </div><div style="font-family:HelveticaNeue,Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif;font-size:16px" id="m_-7400375510780695963yui_3_16_0_1_1497465999456_8182"><div class="hm HOEnZb"> <div dir="ltr" id="m_-7400375510780695963yui_3_16_0_1_1497465999456_8189"> <font id="m_-7400375510780695963yui_3_16_0_1_1497465999456_8188" size="2" face="Arial"> <hr size="1"> <b><span style="font-weight:bold">From:</span></b> Jonathan Wilkes <<a href="mailto:jancsika@yahoo.com" target="_blank">jancsika@yahoo.com</a>><br> <b><span style="font-weight:bold">To:</span></b> Alexandre Torres Porres <<a href="mailto:porres@gmail.com" target="_blank">porres@gmail.com</a>> <br><b><span style="font-weight:bold">Cc:</span></b> Derek Kwan <<a href="mailto:derek.x.kwan@gmail.com" target="_blank">derek.x.kwan@gmail.com</a>>; "<a href="mailto:l2ork-dev@disis.music.vt.edu" target="_blank">l2ork-dev@disis.music.vt.edu</a>" <<a href="mailto:l2ork-dev@disis.music.vt.edu" target="_blank">l2ork-dev@disis.music.vt.edu</a>><wbr>; "<a href="mailto:brbrofsvl@gmail.com" target="_blank">brbrofsvl@gmail.com</a>" <<a href="mailto:brbrofsvl@gmail.com" target="_blank">brbrofsvl@gmail.com</a>><br> <b><span style="font-weight:bold">Sent:</span></b> Wednesday, June 14, 2017 2:51 PM<br> <b><span style="font-weight:bold">Subject:</span></b> cyclone merge resolution (Re: [L2Ork-dev] updating cyclone)<br> </font> </div></div><div><div class="h5"> <div class="m_-7400375510780695963y_msg_container" id="m_-7400375510780695963yui_3_16_0_1_1497465999456_8181"><br><div id="m_-7400375510780695963yiv3864808965"><div id="m_-7400375510780695963yui_3_16_0_1_1497465999456_8234"><div style="color:#000;background-color:#fff;font-family:Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif;font-size:13px" id="m_-7400375510780695963yui_3_16_0_1_1497465999456_8233"><div id="m_-7400375510780695963yiv3864808965yui_3_16_0_1_1497465999456_4036">Hi everyone,</div><div>I've got a pull request open on pd-cyclone to change the name of the binary for loading the ops.</div><div id="m_-7400375510780695963yui_3_16_0_1_1497465999456_8232"><br clear="none"></div><div>You can read about it here:</div><div dir="ltr" id="m_-7400375510780695963yiv3864808965yui_3_16_0_1_1497465999456_3937"><a rel="nofollow" shape="rect" class="m_-7400375510780695963yiv3864808965" id="m_-7400375510780695963yiv3864808965yui_3_16_0_1_1497465999456_3936" href="https://github.com/porres/pd-cyclone/pulls" target="_blank">https://github.com/porres/pd-<wbr>cyclone/pulls</a></div><div><br clear="none"></div><div dir="ltr" id="m_-7400375510780695963yiv3864808965yui_3_16_0_1_1497465999456_4007">I want to know if it should be accepted before I merge pd-cyclone into Purr Data and make the next release.</div><div dir="ltr" id="m_-7400375510780695963yiv3864808965yui_3_16_0_1_1497465999456_4086"><br clear="none"></div><div dir="ltr" id="m_-7400375510780695963yiv3864808965yui_3_16_0_1_1497465999456_4085">Albert, Ico, Alexandre, Matt, Derek? How do you vote?</div><div dir="ltr" id="m_-7400375510780695963yiv3864808965yui_3_16_0_1_1497465999456_4069"><br clear="none"></div><div dir="ltr" id="m_-7400375510780695963yiv3864808965yui_3_16_0_1_1497465999456_4070">I abstain.</div><div class="m_-7400375510780695963yiv3864808965yqt7744329860" id="m_-7400375510780695963yiv3864808965yqtfd56662"><div dir="ltr" id="m_-7400375510780695963yiv3864808965yui_3_16_0_1_1497465999456_4083"><br clear="none"></div><div dir="ltr" id="m_-7400375510780695963yiv3864808965yui_3_16_0_1_1497465999456_4084">-Jonathan<br clear="none"></div><div dir="ltr" id="m_-7400375510780695963yiv3864808965yui_3_16_0_1_1497465999456_4008"><br clear="none"></div></div></div></div></div><br><br></div> </div></div></div> </div>  </div></div></div></blockquote></div><br></div>