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

buildroot and %clean section is not needed #409

Merged
merged 1 commit into from
Aug 2, 2012

Conversation

xsuchy
Copy link
Contributor

@xsuchy xsuchy commented Aug 1, 2012

see http://fedoraproject.org/wiki/Packaging:Guidelines#BuildRoot_tag
It is needed only for EL5, but we do not support server on EL5


# we need to keep RHEL compatibility
BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)

Copy link
Contributor

Choose a reason for hiding this comment

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

So I assume its not needed for RHEL6 anymore. But how about RHEL5? I am not sure if we support this for CLI.

@lzap
Copy link
Contributor

lzap commented Aug 2, 2012

👍 Looks OK to me. Only katello-agent need to support RHEL5 I guess.

@xsuchy
Copy link
Contributor Author

xsuchy commented Aug 2, 2012

fixed, in fact in katello-agent is was removed week ago in commit 30a0a56
so I added it there back as we really build katello-agent for el5 (alas only in CFSE).

@lzap
Copy link
Contributor

lzap commented Aug 2, 2012

Oh good. All others are fine I guess.

lzap added a commit that referenced this pull request Aug 2, 2012
buildroot and %clean section is not needed
@lzap lzap merged commit 8812e3e into Katello:master Aug 2, 2012
lzap added a commit that referenced this pull request Apr 27, 2013
buildroot and %clean section is not needed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants