fix: processing symlinks in directories

[Felixoid]

This PR fixes #428 and depends on goreleaser/fileglob#24

appendGlobbedFiles now distinguish symlinks and add a proper Content object.

Tests in deb and rpm are removed because covered by files now.

I'll update the go.mod and convert it from the draft after the mentioned PR is merged.

[codecov]

Codecov Report

Merging #453 (ec46922) into main (53a3fc2) will increase coverage by 0.03%.
The diff coverage is 100.00%.

❗ Current head ec46922 differs from pull request most recent head bcac98a. Consider uploading reports for the commit bcac98a to get more accurate results

@@            Coverage Diff             @@
##             main     #453      +/-   ##
==========================================
+ Coverage   65.89%   65.93%   +0.03%     
==========================================
  Files          16       16              
  Lines        1903     1905       +2     
==========================================
+ Hits         1254     1256       +2     
  Misses        510      510              
  Partials      139      139              

Impacted Files	Coverage Δ
files/files.go	88.28% <100.00%> (+0.18%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 53a3fc2...bcac98a. Read the comment docs.

[Felixoid]

Woop-woop, it's green :-j

[caarlos0]

looks good, but I would keep the symlink tests in each packager too

[Felixoid]

Then I'd rather just put the static links and see if it's there, without helpers and creating. On one hand.

On another hand, there are still tests though https://github.com/goreleaser/nfpm/blob/main/apk/apk_test.go#L477, https://github.com/goreleaser/nfpm/blob/main/rpm/rpm_test.go#L641, and https://github.com/goreleaser/nfpm/blob/main/deb/deb_test.go#L685

Removed tests were there for particular checking if the destination files in archives are the same as the target files for symlinks. And this logic is completely dead now.

upd: the coverage report shows, that I haven't removed any tests that did check for anything

[Felixoid]

Dear @erikgeiser and @djgilcrease, can you review it too, please? Would be very helpful to have it merged!

[djgilcrease]

Overall looks good to me,

[caarlos0]

Thanks everyone!

[Felixoid]

Thank you, awesome!

I am looking forward to a new release!