`mtime` is not used from config - Githubissues

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.13k stars 157 forks source link

`mtime` is not used from config #750

Closed keliramu closed 8 months ago

keliramu commented 8 months ago

What happened?

After recent changes, in your code you do not copy c.MTime over to info.MTime. Also, you do not enforce mtime onto info.Content Files.

I would expect that mtime from config (or from env variable) to be enforce on all timestamps.

How can we reproduce this?

Code location: https://github.com/goreleaser/nfpm/blob/main/nfpm.go#L123 here you need to info.MTime = c.MTime

nfpm version

using Dec 7 commit: 7439013fb5dc9357e84aabb36823a21c1b97e8ba

Search

[X] I did search for other open and closed issues before opening this.

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Additional context

No response

caarlos0 commented 8 months ago

hmm, seems like mergo is ignoring that field (and many others??)

will investigate