Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1682)

Issue 4336041: [PATCH] Add libstdc++-4.4 patch for C++0x to the website (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 8 months ago by Jeffrey Yasskin
Modified:
13 years, 4 months ago
Reviewers:
rjmccall
CC:
cfe-commits_cs.uiuc.edu
Base URL:
https://llvm.org/svn/llvm-project/cfe/trunk/
Visibility:
Public.

Patch Set 1 #

Patch Set 2 : Add license summary to libstdc++4.4-clang0x.patch #

Unified diffs Side-by-side diffs Delta from patch set Stats (+374 lines, -0 lines) Patch
M www/cxx_status.html View 1 chunk +5 lines, -0 lines 0 comments Download
A www/libstdc++4.4-clang0x.patch View 1 1 chunk +369 lines, -0 lines 0 comments Download

Messages

Total messages: 4
Jeffrey Yasskin
Let me know if there's a better place to put the patch and the link ...
13 years, 8 months ago (2011-03-29 19:22:14 UTC) #1
Jeffrey Yasskin
Whoops, this should have gone to cfe-commits. The patch is at http://codereview.appspot.com/download/issue4336041_1.diff On Tue, Mar ...
13 years, 8 months ago (2011-03-29 21:33:21 UTC) #2
rjmccall_apple.com
On Mar 29, 2011, at 2:33 PM, Jeffrey Yasskin wrote: > Whoops, this should have ...
13 years, 8 months ago (2011-03-29 21:45:37 UTC) #3
Jeffrey Yasskin
13 years, 8 months ago (2011-03-29 22:32:32 UTC) #4
On 2011/03/29 21:45:37, rjmccall_apple.com wrote:
> On Mar 29, 2011, at 2:33 PM, Jeffrey Yasskin wrote:
> > Whoops, this should have gone to cfe-commits. The patch is at
> > http://codereview.appspot.com/download/issue4336041_1.diff
> 
> You should probably note that this patch is offered under the same
> modified GPLv3 as libstdc++-4.4.  I don't think that's a problem for
> http://llvm.org hosting it, though.

Sure. Done and committed. Let me know if there are any problems, or if the link
should go somewhere else.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b