<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Sun, 1 May 2016 at 16:12 Amaury SECHET <<a href="mailto:deadalnix@gmail.com">deadalnix@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Some update on this.<br><div><div class="gmail_extra"><br><div class="gmail_quote"></div></div></div></div><div dir="ltr"><div><div class="gmail_extra"><div class="gmail_quote">2016-04-12 18:48 GMT-07:00 Andrew Wilkins <span dir="ltr"><<a href="mailto:axwalk@gmail.com" target="_blank">axwalk@gmail.com</a>></span>:<br><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"><span><div dir="ltr">On Wed, 13 Apr 2016 at 09:38 Amaury SECHET <<a href="mailto:deadalnix@gmail.com" target="_blank">deadalnix@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">I'd be happy to do it, but this is a bit much high level for me to be actionable. Can you explain me what I should do to reintroduce them int he debian packaging ?</div></blockquote><div><br></div></span><div>On the CMake side, I'm not sure. I think it's just a matter of using the "install()" functions to install them into the appropriate place in the tree. They weren't installed by CMake in the first place; I was going to do that, which is when I was told they shouldn't be distributed.</div><div><br></div></div></div></blockquote><div><br></div></div></div></div></div><div dir="ltr"><div><div class="gmail_extra"><div class="gmail_quote"><div>Turns out, these executable are installed when using -DLLVM_INSTALL_UTILS=1 with cmake, so that simplify this.<br></div></div></div></div></div><div dir="ltr"><div><div class="gmail_extra"><div class="gmail_quote"><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>For Debian packaging, there are a few things.</div><div>1) revive the llvm-X.Y-tools.* files, which were in LLVM 3.7. See: <a href="http://anonscm.debian.org/viewvc/pkg-llvm/llvm-toolchain/branches/3.7/debian/" target="_blank">http://anonscm.debian.org/viewvc/pkg-llvm/llvm-toolchain/branches/3.7/debian/</a>.</div></div></div></blockquote><div><br></div></div></div></div></div><div dir="ltr"><div><div class="gmail_extra"><div class="gmail_quote"><div>This is where I'm not sure how to proceed. How do I contribute to that repository ? Is that debian owned ? What is the process to submit a patch ?</div></div></div></div></div></blockquote><div><br></div><div>Yes, it is Debian-owned. When I made the changes to use CMake, I sent patches directly to Sylvestre Ledru (CCd), who maintains the LLVM/Clang/etc. packages for Debian.</div><div><br></div><div>FWIW, I second Reid's sentiment about having things like "not" in the standard $PATH. They are there in the older versions of LLVM, albeit with version suffixes (e.g. /usr/bin/not-3.7). If it's possible, it would be nice to hide them away in /usr/lib/llvm-<version>/bin, or perhaps a utils-specific directory. I think it would then just be a matter of adding that to $PATH before running lit tests.</div><div><br></div><div>I would offer to help write patches, but you would be at the mercy of my near non-existent availability.</div><div><br></div><div>Cheers,</div><div>Andrew</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div class="gmail_extra"><div class="gmail_quote"><div><span style="line-height:1.5"> </span></div></div></div></div></div><div dir="ltr"><div class="gmail_extra"><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 class="gmail_quote"><div>2) add the llvm-X.Y-tools entry back into control. Compare <a href="http://anonscm.debian.org/viewvc/pkg-llvm/llvm-toolchain/branches/3.7/debian/control?revision=1843&view=markup" target="_blank">http://anonscm.debian.org/viewvc/pkg-llvm/llvm-toolchain/branches/3.7/debian/control?revision=1843&view=markup</a> and <a href="http://anonscm.debian.org/viewvc/pkg-llvm/llvm-toolchain/branches/3.8/debian/control?revision=1881&view=markup" target="_blank">http://anonscm.debian.org/viewvc/pkg-llvm/llvm-toolchain/branches/3.8/debian/control?revision=1881&view=markup</a>.</div><div>3) update the "rules" script as necessary to support the above, copying files into the places that the Debian package layout expects</div><div><br></div><div>HTH,</div><div>Andrew</div><div><div><div> <br></div></div></div></div></div></blockquote></div></div></div></blockquote></div></div>