Skip to content

rpm: Fix issue where no-cache is not respected#775

Merged
cpuguy83 merged 1 commit intoproject-dalec:mainfrom
cpuguy83:fix_rpm_nocache
Oct 2, 2025
Merged

rpm: Fix issue where no-cache is not respected#775
cpuguy83 merged 1 commit intoproject-dalec:mainfrom
cpuguy83:fix_rpm_nocache

Conversation

@cpuguy83
Copy link
Collaborator

@cpuguy83 cpuguy83 commented Oct 1, 2025

Makes sure --no-cache and --no-cache-filter is respected for rpm-based targets.

Signed-off-by: Brian Goff <cpuguy83@gmail.com>
@cpuguy83 cpuguy83 requested a review from a team as a code owner October 1, 2025 23:36
Copilot AI review requested due to automatic review settings October 1, 2025 23:36
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes a bug where the ignore-cache build flag was not being properly respected in RPM and DEB package building workflows. The fix ensures that cache control is consistently applied throughout the build pipeline.

  • Adds proper cache control to worker initialization and package building stages
  • Ensures cache options are properly propagated through the build pipeline
  • Fixes missing options propagation in DEB container building

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
targets/linux/rpm/distro/worker.go Passes IgnoreCache option to Worker method call
targets/linux/rpm/distro/pkg.go Adds cache control to BuildPkg and Build functions
targets/linux/rpm/distro/container.go Adds cache control to BuildContainer function
targets/linux/deb/distro/container.go Fixes missing options propagation to BuildPkg call

Copy link
Member

@sozercan sozercan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@cpuguy83 cpuguy83 merged commit b2993a3 into project-dalec:main Oct 2, 2025
22 checks passed
@cpuguy83 cpuguy83 deleted the fix_rpm_nocache branch October 2, 2025 02:04
@cpuguy83 cpuguy83 changed the title rpm: Fix issue where ignore-cache is not respected rpm: Fix issue where no-cache is not respected Oct 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants