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

Issue 6880189: Fixed recursive variable in Makefile (Closed)

Created:
9 years, 8 months ago by taysom
Modified:
9 years, 7 months ago
CC:
chromium-os-reviews_chromium.org
Visibility:
Public.

Description

Fixed recursive variable in Makefile DEFECT=13360 TEST=compiled with "emerge-x86-mario ktop" Change-Id: I76d877eac3cbb66fb334f3a4cb8f3fa1ef4ba1c6 Merge branch 'master' of ssh://gitrw.chromium.org:9222/ktop into ktop Added missing makefile. BUG=13360 Change-Id: Id60a7273762e764e33f120005f9db01cd86db249 Changes to kplot for ebuild BUG=13360 TEST=compiled with emerge and ran on cr48 Change-Id: I585ea57b7b3b86abe6096f4eade724d5ea12ab9f R=msb@chromium.org,semenzato@chromium.org,ups@chromium.org Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=bde760a

Patch Set 1 #

Total comments: 2

Patch Set 2 : Deleted commented out code. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -3 lines) Patch
M Makefile View 1 3 chunks +6 lines, -3 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
taysom
9 years, 8 months ago (2011-04-25 23:22:29 UTC) #1
Luigi Semenzato
LGTM with nits: Can you elaborate just a bit more on the "recursive variable"? It ...
9 years, 8 months ago (2011-04-25 23:28:45 UTC) #2
taysom
9 years, 8 months ago (2011-04-26 00:01:41 UTC) #3
Paul T
PTAL http://codereview.chromium.org/6880189/diff/1/Makefile File Makefile (right): http://codereview.chromium.org/6880189/diff/1/Makefile#newcode28 Makefile:28: name := $(basename $(notdir $(PWD))) On 2011/04/25 23:28:45, ...
9 years, 8 months ago (2011-04-26 00:02:26 UTC) #4
Luigi Semenzato
9 years, 8 months ago (2011-04-26 00:09:23 UTC) #5
LGTM

On 2011/04/26 00:02:26, Paul T wrote:
> PTAL
> 
> http://codereview.chromium.org/6880189/diff/1/Makefile
> File Makefile (right):
> 
> http://codereview.chromium.org/6880189/diff/1/Makefile#newcode28
> Makefile:28: name    := $(basename $(notdir $(PWD)))
> On 2011/04/25 23:28:45, Luigi Semenzato wrote:
> > I'd be bold and remove the previous line instead of commenting it out.
> 
> Fixed.

Powered by Google App Engine
This is Rietveld 408576698