Opened 13 years ago
Closed 13 years ago
#1746 closed defect (fixed)
errron buildUtil.py replaceVersion
| Reported by: | Owned by: | James Burke | |
|---|---|---|---|
| Priority: | high | Milestone: | 0.9 |
| Component: | BuildTools | Version: | 0.4 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: |
Description
If I supply a -Dversion property when I build it dies with the following error:
[script] Setting version for Dojo to: 0.4 [script] Traceback (innermost last): [script] File "<string>", line 13, in ? [script] File "c:devdojo runkuildscripts.uildUtil.py", line 182, in replaceVersion [script] TypeError?: unsupported operand type(s) for +: 'str' and 'int'
BUILD FAILED C:devdojo runkuildscriptsuild.xml:523: Traceback (innermost last): File "<string>", line 13, in ? File "c:devdojo runkuildscripts.uildUtil.py", line 182, in replaceVers ion TypeError?: unsupported operand type(s) for +: 'str' and 'int'
I dont know python but it seems that when it splits the version on '.' and tests the array for a value for each of majorVersion, minorVersion etc., a bogus value is sneaking through - maybe its a null or undefined.
Sam
Change History (2)
comment:1 Changed 13 years ago by
| Milestone: | → 0.5 |
|---|---|
| Owner: | changed from alex to James Burke |
comment:2 Changed 13 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
This was fixed in r7003, now n.n version numbers seem to work, so no need to add a special message.
We require a three version number, and support a four version number. We don't support a two version number. Maybe we can message this better instead of throwing up. I do not see this as critical, so moving to 0.5.