<div dir="ltr"><div>Hi o/<br></div><div><br></div><div>First of all, thank you for the fixes and letting me know about cowbuilder-dist :-)</div><div><br></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 7 Dec 2020 at 17:51, Michael Crusoe <<a href="mailto:michael.crusoe@gmail.com">michael.crusoe@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"><div dir="ltr"><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>I'd be really grateful if you could take a look at my changes and let me know if they look fine.</div></div></blockquote><div><br></div><div>I'll leave comments on Salsa<br></div></div></div></blockquote><div><br></div><div>Thanks a lot, cherry-picked + pulled the changes mentioned.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div></div><div><br></div><div>Have you been sending PRs and otherwise forwarding your patches upstream?<br></div></div></div></blockquote><div><br></div><div>Yeah, I almost always do the needed forwards. I did 2 simde fixes before this one - for plast and ngmlr.</div><div>I forwarded the patch for plast upstream[1] and I'll forward for ngmlr in some more days - since this seems to need more changes in the upstream codebase (I don't exactly remember what exactly, but this is in my TODO)<br></div><div><br></div><div>[1]: <a href="https://github.com/PLAST-software/plast-library/pull/8">https://github.com/PLAST-software/plast-library/pull/8</a></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><br></div><div>Also, another question: This package has a MPL-2.0 license and AFAIK, MPL is a restrictive Free software license, so does this qualify for adding in a "Built-Using" field?<br></div></div></blockquote></div><div class="gmail_quote"><br></div><div class="gmail_quote">Re-reading <a href="https://wiki.debian.org/SIMDEverywhere#Approach" target="_blank">https://wiki.debian.org/SIMDEverywhere#Approach</a> item 6, we see <br></div><div class="gmail_quote">"if the source package requires the full source code be available"</div><div class="gmail_quote"><br></div><div class="gmail_quote">Does the MPL have this requirement?<br></div></div></blockquote><div><br></div><div>I skimmed through /usr/share/common-licenses/MPL-2.0 but admittedly, it is still not very clear to me. From what I read, it looks like the answer is yes for this question, but I'm not completely confident about it.</div><div>Do you think we should ask debian-legal? - This seems like the last blocker for an upload now<br></div><div><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 7 Dec 2020 at 18:06, Michael Crusoe <<a href="mailto:michael.crusoe@gmail.com">michael.crusoe@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"><div dir="ltr">Don't forget step 8 of <a href="https://wiki.debian.org/SIMDEverywhere#Approach" target="_blank">https://wiki.debian.org/SIMDEverywhere#Approach</a> ; I've updated the sample scripts for multi-compiling x86 at different SIMD levels<br></div></blockquote><div> </div>Done and pushed. it took me quite a while to figure this out. Two questions though:<div>1) We are fundamentally doing this because we want to build this for all possible compilation flags right? - and in the dispatch script we are trying to figure out the most efficient implementation and use the binary for the same?</div><div>2) Could you please review the changes once? Works fine everywhere IMO</div><div><br></div><div>BTW I noticed that the rules file which was initially just around 10 lines increased by approximately an order of magnitude because I had to almost handcraft the makefile to do various stuff, and I observed a similar phenomenon for kalign and mmseqs2 where you made similar changes.</div><div>My intent here is to say - I (vaguely)wonder can we make a tool/helper that can _automate_ the entire process while using simde to reduce handcrafted stuff?</div><div>Because fundamentally we seem to be making similar changes everywhere.</div><div>And ofcourse, apologies if this doesn't sound good.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Please also update <a href="https://wiki.debian.org/SIMDEverywhere#Packages_Status" target="_blank">https://wiki.debian.org/SIMDEverywhere#Packages_Status</a> & <a href="https://wiki.debian.org/SIMDEverywhere#Candidate_packages" target="_blank">https://wiki.debian.org/SIMDEverywhere#Candidate_packages</a> as you add SIMDe support to packages.</div></div></blockquote><div><br></div><div>I'll definitely update this in the next couple of days.</div><div><br></div><div>Kind Regards,</div><div>Nilesh<br></div></div></div><div><br></div></div></div>