Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: provide more field support for rpms #79

Merged

Conversation

djgilcrease
Copy link
Contributor

@djgilcrease djgilcrease commented Oct 3, 2019

New RPM fields

  • Description
  • OS <= info.Platform
  • Licence
  • URL <= info.Homepage
  • Vendor
  • Packager <= info.Maintainer
  • Provides
  • Require <= info.Depends
  • Obsolete <= info.Replaces
  • Suggests
  • Conflicts

Fixes

  • RPM Compression types are supported again
  • RPM Config files are supported again
    • rpmpack supports all the rpm file types, so we could extend this more latter

Closes: #71 #54

@caarlos0
Copy link
Member

caarlos0 commented Oct 4, 2019

LGTM so far :D

Dj Gilcrease added 2 commits October 8, 2019 11:45
New RPM fields
---
* Description
* OS <= info.Platform
* Licence
* URL <= info.Homepage
* Vendor
* Packager <= info.Maintainer
* Provides
* Require <= info.Depends
* Obsolete <= info.Replaces
* Suggests
* Conflicts

Fixes
---
* RPM Compression types are supported again
* RPM Config files are supported again
  * `rpmpack` will support all the rpm file types, so we could extend this more latter
@djgilcrease djgilcrease force-pushed the feature/support-more-rpm-features branch from e4b8762 to 4b772d8 Compare October 8, 2019 18:47
@djgilcrease djgilcrease changed the title WIP: provide more field support for rpms provide more field support for rpms Oct 8, 2019
@djgilcrease
Copy link
Contributor Author

Should be all good now

Copy link
Member

@caarlos0 caarlos0 left a comment

Choose a reason for hiding this comment

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

Thanks for all the hard work! It is looking great so far!

I did a few comments, let me know what you think!

Cheers

acceptance/testdata/rpm.overrides.dockerfile Outdated Show resolved Hide resolved
acceptance/testdata/rpm.min.dockerfile Outdated Show resolved Hide resolved
acceptance/testdata/rpm.dockerfile Outdated Show resolved Hide resolved
acceptance/testdata/rpm.complex.dockerfile Outdated Show resolved Hide resolved
acceptance/testdata/release.rpm.dockerfile Outdated Show resolved Hide resolved
acceptance/acceptance_test.go Outdated Show resolved Hide resolved
acceptance/acceptance_test.go Outdated Show resolved Hide resolved
acceptance/acceptance_test.go Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
Makefile Show resolved Hide resolved
@codecov-io
Copy link

codecov-io commented Oct 9, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@7a174b2). Click here to learn what that means.
The diff coverage is 78.75%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #79   +/-   ##
=========================================
  Coverage          ?   71.09%           
=========================================
  Files             ?        5           
  Lines             ?      467           
  Branches          ?        0           
=========================================
  Hits              ?      332           
  Misses            ?       84           
  Partials          ?       51
Impacted Files Coverage Δ
rpm/rpm.go 71.63% <78.75%> (ø)

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 7a174b2...12f49a7. Read the comment docs.

@caarlos0 caarlos0 changed the title provide more field support for rpms feat: provide more field support for rpms Oct 9, 2019
@caarlos0
Copy link
Member

caarlos0 commented Oct 9, 2019

awesome, thanks!

@caarlos0 caarlos0 merged commit 03193b2 into goreleaser:master Oct 9, 2019
@caarlos0
Copy link
Member

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@goreleaser goreleaser locked as resolved and limited conversation to collaborators Nov 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

master appears to have broken rpm installs on Centos 7
3 participants