<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">On Feb 12, 2016, at 1:08 PM, Chandler Carruth <<a href="mailto:chandlerc@gmail.com" class="">chandlerc@gmail.com</a>> wrote:<br class=""><div><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class=""><div class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class=""><div class=""><div class="">I got permission internally and contributed the header under the LLVM license, I don’t think that whether it originated from swift repo or not makes any difference here.</div><div class="">If you could elaborate more on your concerns it would be helpful.</div></div></div></blockquote><div class=""><br class=""></div><div class="">How do we know whether any other contributor to Swift authored some of the code in this file? Does Apple get copyright assignment for all Swift contributions? Only if Apple has copyright assignment for all contributions to Swift or only people from Apple have every contributed to this part of Swift is asking internally enough.</div><div class=""><br class=""></div><div class="">And since this is an open source project, it would seem polite (even if not necessary) to also ask the community rather than just asking internally. We don't all work at Apple. =/</div></div></div></div></blockquote></div></div></blockquote></div></div></div></blockquote><div><br class=""></div><div>The code is 100% copyrighted by Apple. The full history in the swift repo is here:</div><div><a href="https://github.com/apple/swift/commits/master/include/swift/Basic/OptionSet.h" class="">https://github.com/apple/swift/commits/master/include/swift/Basic/OptionSet.h</a></div><div><br class=""></div><div>There are two trivial patches by non-apple contributors. Both are both single line patches that adjust comments (one is the first line of the file, one is the copyright date):</div><div><a href="https://github.com/apple/swift/commit/1339b5403bbaf6205abb2bfdf7fabadef1aacc70#diff-1e8f8df8addd9510deea20d4bea2eda2" class="">https://github.com/apple/swift/commit/1339b5403bbaf6205abb2bfdf7fabadef1aacc70#diff-1e8f8df8addd9510deea20d4bea2eda2</a></div><div><a href="https://github.com/apple/swift/commit/e3a4147ac94e55fcab1d14e949f572b53d9eb638#diff-1e8f8df8addd9510deea20d4bea2eda2" class="">https://github.com/apple/swift/commit/e3a4147ac94e55fcab1d14e949f572b53d9eb638#diff-1e8f8df8addd9510deea20d4bea2eda2</a></div><div><br class=""></div><div>That said, both of those patches are irrelevant to the discussion, because Argyrios removed the swift header and replaced it with the LLVM header when he checked it in.</div><div><br class=""></div><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class=""><div class=""><blockquote type="cite" class=""></blockquote><div class="">+ Chris, for comment.</div></div></div></blockquote><div class=""><br class=""></div><div class="">Also, just to be clear (since I think my original email wasn't terribly): I think something as simple as this code review thread would have been fine if you had waited for the code review, and made it clear why it was OK to contribute (only Apple folks have touched it, or whatever).</div><div class=""><br class=""></div><div class="">I'm not suggesting we need to have some big or complex discussion. =] We have plenty of those on harder subjects.</div></div></div>
</div></blockquote></div><div class=""><br class=""></div><div class="">I agree that the header should get the normal patch review!</div><div class=""><br class=""></div><div class="">-Chris</div></body></html>