New issue
Advanced search Search tips

Issue 614747 link

Starred by 1 user
Status: Untriaged
Owner: ----
Cc:
dpranke@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

GN: allow nested accessors

Project Member Reported by sdefresne@chromium.org, May 25 2016 
See comment in chrome/chrome_repack_locales.gni

  # GN's subscript is too stupid to do invoker.output_locales[foo] so we need
  # to make a copy and do output_locales[foo].
  output_locales = invoker.output_locales

Filed after I was asked whether it is a bug or not when duplicating that workaround in https://codereview.chromium.org/2014473004/.

brettw: do we want to allow "invoker.output_locales[foo]"?

Comment 1 by brettw@chromium.org, May 25 2016 

Seems like invoker.output_locales[foo] should work, but fixing is a low priority.

Comment 2 by brettw@chromium.org, Aug 12 2016

Summary: GN: allow nested accessors (was: GN: allow to subscript invoker.variable)
Basically allowing nested accessors will enable things like this:
  foo.bar.baz
  foo.bar[0]

This is a nice to have but complicates the implementation and has very low impact.

Comment 3 by brettw@chromium.org, Aug 17 2016

Owner: ----
Status: Available (was: Assigned)
Project Member

Comment 4 by sheriffbot@chromium.org, Aug 18 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 5 by dtapu...@chromium.org, Nov 10 2017

Components: Build
►

Sign in to add a comment