<div dir="ltr"><div class="gmail_quote">On Tue, 3 Mar 2015 at 08:25 Luca Bruno <<a href="mailto:lucab@debian.org">lucab@debian.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Friday 27 February 2015 10:23:32 Angus Lees wrote:<br>
<br>
> now building with system LLVM. It passes my local build tests, but if<br>
> there's any issues then revert at will.<br>
<br>
This is great news, as we pushed quite a bit in order to have alpha2 in sync<br>
with 3.6. Big thanks to Björn for patching the last missing pieces in<br>
rustllvm.<br>
<br>
I'm however a bit fearful, as when asked upstream doesn't seem to have a<br>
single vision on this[0], and compatibility plans looked a bit unreal.<br>
<br>
[0] <a href="http://internals.rust-lang.org/t/targeted-llvm-for-1-0/1371" target="_blank">http://internals.rust-lang.<u></u>org/t/targeted-llvm-for-1-0/<u></u>1371</a><br>
<br>
Plus, patches against LLVM are again starting piling up (currently 2, both<br>
rust-specific). Maybe we should note them down on a wiki page and periodically<br>
recheck them, to avoid suddenly ending in a major screwed due to some<br>
important patches missing on our LLVM.<br></blockquote><div><br></div><div>Yeah agreed. I expect this will require diligence from us and we'll flip to/from using system llvm occasionally given the priorities/pace of rust upstream. I suspect anything we can do to automate/track this will be worthwhile.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
> * Patch upstream manpages to address minor troff issues<br>
<br>
This is good and I saw your (still open) PR.<br>
However, I'd like to stick to a policy common to other packaging teams:<br>
don't carry any patches generated on our own (expect when debian-specific),<br>
but only backport them once committed upstream.<br>
This is because I'd like to avoid constantly rebasing patches, and for code<br>
stuff avoid big screw-ups á-la openssl.<br></blockquote><div><br></div><div>Understood. I figured this was only manpage markup and it was fixing a lintian error - but I accept your polite admonishment and will commit via upstream in future ;)</div><div><br></div><div> - Gus</div></div></div>