Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(137)

Issue 8394006: Import with keyword for 2.5 python as well as backport os.path.relpath() from python unittest2 (Closed)

Created:
9 years, 2 months ago by codefu
Modified:
9 years, 2 months ago
Reviewers:
mmendez
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Import with keyword for 2.5 python as well as backport os.path.relpath() from python unittest2 R=mmendez BUG= TEST= Committed: https://code.google.com/p/dart/source/detail?r=708

Patch Set 1 #

Patch Set 2 : Backing out tools change #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -0 lines) Patch
M compiler/generate_source_list.py View 1 chunk +1 line, -0 lines 0 comments Download
M compiler/generate_systemlibrary_list.py View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
codefu
9 years, 2 months ago (2011-10-25 18:24:43 UTC) #1
mmendez
On 2011/10/25 18:24:43, codefu wrote: Like we discussed over IM, lets address the compiler issues ...
9 years, 2 months ago (2011-10-25 18:30:18 UTC) #2
codefu
On 2011/10/25 18:30:18, mmendez wrote: > On 2011/10/25 18:24:43, codefu wrote: > > Like we ...
9 years, 2 months ago (2011-10-25 18:31:15 UTC) #3
mmendez
9 years, 2 months ago (2011-10-25 18:41:21 UTC) #4
LGTM - Please double check that python 2.6 doesn't emit any warnings as a
result.

On 2011/10/25 18:31:15, codefu wrote:
> On 2011/10/25 18:30:18, mmendez wrote:
> > On 2011/10/25 18:24:43, codefu wrote:
> > 
> > Like we discussed over IM, lets address the compiler issues in one change. 
> The
> > os.path.relpath is a separate matter.
> 
> patch 2 removed the relpath issue.

Powered by Google App Engine
This is Rietveld 408576698