diff options
Diffstat (limited to 'manifest_xml.py')
| -rw-r--r-- | manifest_xml.py | 35 |
1 files changed, 35 insertions, 0 deletions
diff --git a/manifest_xml.py b/manifest_xml.py index b26b0cac..cdab31e6 100644 --- a/manifest_xml.py +++ b/manifest_xml.py | |||
| @@ -1445,6 +1445,7 @@ https://gerrit.googlesource.com/git-repo/+/HEAD/docs/manifest-format.md | |||
| 1445 | 1445 | ||
| 1446 | repo_hooks_project = None | 1446 | repo_hooks_project = None |
| 1447 | enabled_repo_hooks = None | 1447 | enabled_repo_hooks = None |
| 1448 | failed_revision_changes = [] | ||
| 1448 | for node in itertools.chain(*node_list): | 1449 | for node in itertools.chain(*node_list): |
| 1449 | if node.nodeName == "project": | 1450 | if node.nodeName == "project": |
| 1450 | project = self._ParseProject(node) | 1451 | project = self._ParseProject(node) |
| @@ -1471,6 +1472,7 @@ https://gerrit.googlesource.com/git-repo/+/HEAD/docs/manifest-format.md | |||
| 1471 | remote = self._get_remote(node) | 1472 | remote = self._get_remote(node) |
| 1472 | dest_branch = node.getAttribute("dest-branch") | 1473 | dest_branch = node.getAttribute("dest-branch") |
| 1473 | upstream = node.getAttribute("upstream") | 1474 | upstream = node.getAttribute("upstream") |
| 1475 | base_revision = node.getAttribute("base-rev") | ||
| 1474 | 1476 | ||
| 1475 | named_projects = self._projects[name] | 1477 | named_projects = self._projects[name] |
| 1476 | if dest_path and not path and len(named_projects) > 1: | 1478 | if dest_path and not path and len(named_projects) > 1: |
| @@ -1484,6 +1486,13 @@ https://gerrit.googlesource.com/git-repo/+/HEAD/docs/manifest-format.md | |||
| 1484 | if groups: | 1486 | if groups: |
| 1485 | p.groups.extend(groups) | 1487 | p.groups.extend(groups) |
| 1486 | if revision: | 1488 | if revision: |
| 1489 | if base_revision: | ||
| 1490 | if p.revisionExpr != base_revision: | ||
| 1491 | failed_revision_changes.append( | ||
| 1492 | "extend-project name %s mismatch base " | ||
| 1493 | "%s vs revision %s" | ||
| 1494 | % (name, base_revision, p.revisionExpr) | ||
| 1495 | ) | ||
| 1487 | p.SetRevision(revision) | 1496 | p.SetRevision(revision) |
| 1488 | 1497 | ||
| 1489 | if remote_name: | 1498 | if remote_name: |
| @@ -1558,6 +1567,7 @@ https://gerrit.googlesource.com/git-repo/+/HEAD/docs/manifest-format.md | |||
| 1558 | if node.nodeName == "remove-project": | 1567 | if node.nodeName == "remove-project": |
| 1559 | name = node.getAttribute("name") | 1568 | name = node.getAttribute("name") |
| 1560 | path = node.getAttribute("path") | 1569 | path = node.getAttribute("path") |
| 1570 | base_revision = node.getAttribute("base-rev") | ||
| 1561 | 1571 | ||
| 1562 | # Name or path needed. | 1572 | # Name or path needed. |
| 1563 | if not name and not path: | 1573 | if not name and not path: |
| @@ -1571,6 +1581,13 @@ https://gerrit.googlesource.com/git-repo/+/HEAD/docs/manifest-format.md | |||
| 1571 | for projname, projects in list(self._projects.items()): | 1581 | for projname, projects in list(self._projects.items()): |
| 1572 | for p in projects: | 1582 | for p in projects: |
| 1573 | if name == projname and not path: | 1583 | if name == projname and not path: |
| 1584 | if base_revision: | ||
| 1585 | if p.revisionExpr != base_revision: | ||
| 1586 | failed_revision_changes.append( | ||
| 1587 | "remove-project name %s mismatch base " | ||
| 1588 | "%s vs revision %s" | ||
| 1589 | % (name, base_revision, p.revisionExpr) | ||
| 1590 | ) | ||
| 1574 | del self._paths[p.relpath] | 1591 | del self._paths[p.relpath] |
| 1575 | if not removed_project: | 1592 | if not removed_project: |
| 1576 | del self._projects[name] | 1593 | del self._projects[name] |
| @@ -1578,6 +1595,17 @@ https://gerrit.googlesource.com/git-repo/+/HEAD/docs/manifest-format.md | |||
| 1578 | elif path == p.relpath and ( | 1595 | elif path == p.relpath and ( |
| 1579 | name == projname or not name | 1596 | name == projname or not name |
| 1580 | ): | 1597 | ): |
| 1598 | if base_revision: | ||
| 1599 | if p.revisionExpr != base_revision: | ||
| 1600 | failed_revision_changes.append( | ||
| 1601 | "remove-project path %s mismatch base " | ||
| 1602 | "%s vs revision %s" | ||
| 1603 | % ( | ||
| 1604 | p.relpath, | ||
| 1605 | base_revision, | ||
| 1606 | p.revisionExpr, | ||
| 1607 | ) | ||
| 1608 | ) | ||
| 1581 | self._projects[projname].remove(p) | 1609 | self._projects[projname].remove(p) |
| 1582 | del self._paths[p.relpath] | 1610 | del self._paths[p.relpath] |
| 1583 | removed_project = p.name | 1611 | removed_project = p.name |
| @@ -1597,6 +1625,13 @@ https://gerrit.googlesource.com/git-repo/+/HEAD/docs/manifest-format.md | |||
| 1597 | "project: %s" % node.toxml() | 1625 | "project: %s" % node.toxml() |
| 1598 | ) | 1626 | ) |
| 1599 | 1627 | ||
| 1628 | if failed_revision_changes: | ||
| 1629 | raise ManifestParseError( | ||
| 1630 | "revision base check failed, rebase patches and update " | ||
| 1631 | "base revs for: ", | ||
| 1632 | failed_revision_changes, | ||
| 1633 | ) | ||
| 1634 | |||
| 1600 | # Store repo hooks project information. | 1635 | # Store repo hooks project information. |
| 1601 | if repo_hooks_project: | 1636 | if repo_hooks_project: |
| 1602 | # Store a reference to the Project. | 1637 | # Store a reference to the Project. |
