Opened 4 years ago

Last modified 4 years ago

#4192 closed defect

Build fails when using git repository — at Version 5

Reported by: greiner Assignee: kzar
Priority: P1 Milestone:
Module: Automation Keywords:
Cc: sebastian, kzar, kvas, trev Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):



Ubuntu 14.04

How to reproduce

git clone
cd adblockpluschrome
python -t chrome build

Observed behaviour

Build not created due to the following error:

Traceback (most recent call last):
  File "./", line 18, in <module>, sys.argv)
  File "/home/thomas/tmp/adblockpluschrome/buildtools/", line 613, in processArgs
    commands[command](baseDir, scriptName, opts, args, type)
  File "/home/thomas/tmp/adblockpluschrome/buildtools/", line 55, in __call__
    return self._handler(baseDir, scriptName, opts, args, type)
  File "/home/thomas/tmp/adblockpluschrome/buildtools/", line 207, in runBuild
    releaseBuild=releaseBuild, keyFile=keyFile)
  File "/home/thomas/tmp/adblockpluschrome/buildtools/", line 326, in createBuild
    version = getBuildVersion(baseDir, metadata, releaseBuild, buildNum)
  File "/home/thomas/tmp/adblockpluschrome/buildtools/", line 60, in getBuildVersion
    if len(buildNum) > 0:
TypeError: object of type 'int' has no len()

Note that this doesn't occur when using the Mercurial repository.

Expected behaviour

Build created without errors.

Change History (5)

comment:1 Changed 4 years ago by kzar

  • Cc kvas trev added
  • Component changed from Unknown to Build-and-Release-Tools
  • Priority changed from Unknown to P1
  • Ready set

Can reproduce, seems to be a regression caused by Issue 4099 - Don't enforce Gecko build IDs to be numerical.

comment:2 Changed 4 years ago by kzar

  • Owner set to kzar

comment:3 Changed 4 years ago by kzar

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:4 Changed 4 years ago by abpbot

A commit referencing this issue has landed:
Issue 4192 - getBuildNum must return a string

comment:5 Changed 4 years ago by kzar

  • Review URL(s) modified (diff)
Note: See TracTickets for help on using tickets.