From 338d0ac83b51a476c7b1c3f50c75e1f3ebf80a7a Mon Sep 17 00:00:00 2001 From: Rohan Devasthale Date: Wed, 5 Feb 2025 15:33:55 -0500 Subject: [PATCH] Add debug message for expected build tag --- e2e/test_bootstrap_constraints.sh | 4 ++-- src/fromager/bootstrapper.py | 6 ++++++ src/fromager/packagesettings.py | 9 +++++++-- 3 files changed, 15 insertions(+), 4 deletions(-) diff --git a/e2e/test_bootstrap_constraints.sh b/e2e/test_bootstrap_constraints.sh index c5bbde3b..6487807b 100755 --- a/e2e/test_bootstrap_constraints.sh +++ b/e2e/test_bootstrap_constraints.sh @@ -38,7 +38,7 @@ fi $pass EXPECTED_LINES=" -pbr==6.1.0 +pbr==6.1.1 # ERROR stevedore==4.0.0 stevedore==5.2.0 @@ -51,4 +51,4 @@ for pattern in $EXPECTED_LINES; do fi done -$pass \ No newline at end of file +$pass diff --git a/src/fromager/bootstrapper.py b/src/fromager/bootstrapper.py index 27d76850..2a9621df 100644 --- a/src/fromager/bootstrapper.py +++ b/src/fromager/bootstrapper.py @@ -323,6 +323,12 @@ def _download_wheel_from_cache( wheelfile_name = pathlib.Path(urlparse(wheel_url).path) pbi = self.ctx.package_build_info(req) expected_build_tag = pbi.build_tag(resolved_version) + # Log the expected build tag for debugging + logger.info(f"{req.name}: has expected build tag {expected_build_tag}") + # Get changelogs for debug info + changelogs = pbi.get_changelog(resolved_version) + logger.debug(f"{req.name} has change logs {changelogs}") + dist_name, dist_version, build_tag, _ = wheels.extract_info_from_wheel_file( req, wheelfile_name ) diff --git a/src/fromager/packagesettings.py b/src/fromager/packagesettings.py index 773e7724..01ceeba9 100644 --- a/src/fromager/packagesettings.py +++ b/src/fromager/packagesettings.py @@ -627,11 +627,16 @@ def build_dir(self, sdist_root_dir: pathlib.Path) -> pathlib.Path: return sdist_root_dir / relative_build_dir return sdist_root_dir + def get_changelog(self, version: Version) -> list[str]: + pv = typing.cast(PackageVersion, version) + variant_changelog = self._variant_changelog + package_changelog = self._ps.changelog.get(pv, []) + return variant_changelog + package_changelog + def build_tag(self, version: Version) -> BuildTag: """Build tag for version's changelog and this variant""" pv = typing.cast(PackageVersion, version) - release = len(self._ps.changelog.get(pv, [])) - release += len(self._variant_changelog) + release = len(self.get_changelog(pv)) if release == 0: return () # suffix = "." + self.variant.replace("-", "_")