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

rpm: Add a option to insert raw Require statements #687

Merged

Conversation

vStone
Copy link
Contributor

@vStone vStone commented Apr 29, 2014

This can be useful to have requirements for different stages.

Examples:

--rpm-add-require 'Requires(post): /usr/sbin/alternatives'
--rpm-add-require 'Requires(preun): /usr/sbin/alternatives'

@jordansissel
Copy link
Owner

hmm, it's not really an 'add-require' if you specify Requires in the value. The rpm book calls this a tag. What about --rpm-tag 'Requires.....' ?

Citation: http://www.rpm.org/max-rpm/s1-rpm-inside-tags.html

Looking at a spec file, the first thing you'll see are a number of lines, all following the same basic format:
something:something-else
The is known as a "tag", because it is used by RPM to name or tag some data.

@vStone
Copy link
Contributor Author

vStone commented Jul 25, 2014

I'll try to run a quick sed -i on the code and update the pull request later today.

@vStone
Copy link
Contributor Author

vStone commented Jul 25, 2014

Rebased + adjusted commit.

This can be useful to have requirements for different stages.

Examples:

    --rpm-tag 'Requires(post): /usr/sbin/alternatives'
    --rpm-tag 'Requires(preun): /usr/sbin/alternatives'
vStone added a commit to vStone/fpm that referenced this pull request Sep 18, 2014
…w Require statements

* pr/687:
  rpm: Add a option to insert raw tag statements
jordansissel added a commit that referenced this pull request Nov 7, 2015
rpm: Add a option to insert raw Require statements
@jordansissel jordansissel merged commit 66e0501 into jordansissel:master Nov 7, 2015
@jordansissel
Copy link
Owner

Thanks for your work revising this patch <3

Merged!

@vStone vStone deleted the feature/rpm-custom-requires branch November 13, 2015 11:11
jordansissel added a commit that referenced this pull request Jun 20, 2016
rpm: Add a option to insert raw Require statements
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants