breach / thrust

Chromium-based cross-platform / cross-language application framework
MIT License
2.77k stars 121 forks source link

Debian Build Error #163

Closed morganrallen closed 9 years ago

morganrallen commented 10 years ago

I've made it throught to attempting the build to get an error related to Grit.

morgan@debian:~/devel/chromium/src$ ./build/gyp_chromium exo_browser/exo_browser.gyp 
Updating projects from gyp files...
------------------------------------------------------------------------------
Resource id not set for exo_browser/exo_browser_resources.grd (messages)!
Please update ../tools/gritsettings/resource_ids to include an entry for exo_browser/exo_browser_resources.grd.  See the comments in resource_ids for information on why you need to update that file.
------------------------------------------------------------------------------
Exception: u'messages'
Traceback (most recent call last):
  File "/home/morgan/devel/chromium/src/tools/gyp/pylib/gyp/input.py", line 469, in CallLoadTargetBuildFile
    includes, depth, check, False)
  File "/home/morgan/devel/chromium/src/tools/gyp/pylib/gyp/input.py", line 385, in LoadTargetBuildFile
    build_file_data, PHASE_EARLY, variables, build_file_path)
  File "/home/morgan/devel/chromium/src/tools/gyp/pylib/gyp/input.py", line 1234, in ProcessVariablesAndConditionsInDict
    build_file)
  File "/home/morgan/devel/chromium/src/tools/gyp/pylib/gyp/input.py", line 1249, in ProcessVariablesAndConditionsInList
    ProcessVariablesAndConditionsInDict(item, phase, variables, build_file)
  File "/home/morgan/devel/chromium/src/tools/gyp/pylib/gyp/input.py", line 1234, in ProcessVariablesAndConditionsInDict
    build_file)
  File "/home/morgan/devel/chromium/src/tools/gyp/pylib/gyp/input.py", line 1249, in ProcessVariablesAndConditionsInList
    ProcessVariablesAndConditionsInDict(item, phase, variables, build_file)
  File "/home/morgan/devel/chromium/src/tools/gyp/pylib/gyp/input.py", line 1234, in ProcessVariablesAndConditionsInDict
    build_file)
  File "/home/morgan/devel/chromium/src/tools/gyp/pylib/gyp/input.py", line 1253, in ProcessVariablesAndConditionsInList
    expanded = ExpandVariables(item, phase, variables, build_file)
  File "/home/morgan/devel/chromium/src/tools/gyp/pylib/gyp/input.py", line 873, in ExpandVariables
    replacement = str(py_module.DoMain(parsed_contents[1:])).rstrip()
  File "/home/morgan/devel/chromium/src/tools/grit/grit_info.py", line 138, in DoMain
    options.target_platform)
  File "/home/morgan/devel/chromium/src/tools/grit/grit_info.py", line 68, in Inputs
    first_ids_file=ids_file, target_platform=target_platform)
  File "/home/morgan/devel/chromium/src/tools/grit/grit/grd_reader.py", line 214, in Parse
    handler.root.AssignFirstIds(filename_or_stream, defines)
  File "/home/morgan/devel/chromium/src/tools/grit/grit/node/misc.py", line 460, in AssignFirstIds
    raise e
KeyError: u'messages'

I've added an entry to ./tools/gritsettings/resource_ids pointing to exo_browser/exo_browser_resources.grd (copied from chromes) but I still get the message and error.

I found this related bug with similar error message but the fix doesn't seem to apply.

https://codereview.chromium.org/253443002

The file it patches no longer contains mimetypes.init

azer commented 10 years ago

I'm hitting the same error unfortunately

azer commented 10 years ago

adding this fixes:

"exo_browser/exo_browser_resources.grd": {
+    "includes": [32000],
+    "structures": [32100],
+    "messages": [32200],
+  },
spolu commented 10 years ago

Did you run the diff on the chromium repo?

spolu commented 9 years ago

@azer @morganrallen can you attempt to rebuild the recent master branch to see if this reproduces?

see https://github.com/breach/thrust#building-thrust

Thanks!

spolu commented 9 years ago

Fixed by v0.7