goreleaser / nfpm

nFPM is Not FPM - a simple deb, rpm, apk, ipk, and arch linux packager written in Go
https://nfpm.goreleaser.com/
MIT License
2.15k stars 157 forks source link

fix: better handle files owned by the fs #834

Closed caarlos0 closed 3 months ago

caarlos0 commented 3 months ago

forcefully set dirs owned by the FS as implicit dirs.

the rpm implementation should then not add them, which should be the right behavior I think.

this way these files will still be owned by the filesystem package instead of the package being built.

this only affects type: tree.

also added logrotate to the list of "common" paths.

closes #829

cloudflare-workers-and-pages[bot] commented 3 months ago

Deploying nfpm with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1965e83
Status: ✅  Deploy successful!
Preview URL: https://ae13cc0b.nfpm.pages.dev
Branch Preview URL: https://issue-829.nfpm.pages.dev

View logs

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 73.12%. Comparing base (784fa6c) to head (1965e83).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #834 +/- ## ========================================== + Coverage 73.01% 73.12% +0.10% ========================================== Files 22 22 Lines 2487 2489 +2 ========================================== + Hits 1816 1820 +4 + Misses 464 463 -1 + Partials 207 206 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.