|
Bugzilla – Full Text Bug Listing |
| Summary: | pybindgen version number format | ||
|---|---|---|---|
| Product: | ns-3 | Reporter: | Tom Henderson <tomh> |
| Component: | build system | Assignee: | Gustavo J. A. M. Carneiro <gjcarneiro> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | ns-bugs |
| Priority: | P3 | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | All | ||
| Attachments: | possible patch | ||
Apologies for the breakage. This is due to me doing an official release. Yes, 3 number versions will occur again in the future, every time there is a release. The patch looks good enough. Possibly could be simplified slightly, but no big deal. pushed in changeset 12886:954349c7cf11 |
Created attachment 2845 [details] possible patch pybindgen version numbering recently changed, from this type of format: 0.17.0.post58+ngcf00cc0 to this: 0.18.0 If a user clones the tip of the repo, and tries to use with our wscript, './waf configure' will error out such as follows. if not (split_version(pybindgen_version) >= split_version(REQUIRED_PYBINDGEN_VERSION)): File "/home/buildslave/jenkins/workspace/daily-fedora-without-valgrind/label/Fedora-64-25/ns-3-allinone/bake/source/ns-3-dev/bindings/python/wscript", line 72, in split_version return (int(ver[0]), int(ver[1]), int(ver[2]), int(ver[3].split('post')[1])) IndexError: list index out of range The attached patch to wscript appears to support both formats.