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

Modify default spec template #10

Merged
merged 1 commit into from
Aug 24, 2016

Conversation

kasicka
Copy link
Contributor

@kasicka kasicka commented Aug 17, 2016

No description provided.

Name: nodejs-%{npm_name}
Version: $VERSION
Release: $RELEASE%{?dist}
Summary: $SUMMARY
License: $LICENSE
Group: Development/Libraries
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is deprecated.

@dLobatog
Copy link
Member

@kasicka 👍 to everything but removing the doc section - why?

Thanks for the PR - it's great to see this being used outside of https://github.com/theforeman/foreman-packaging/ which was the main motivation, what do you use it for (if you can share it)

@kasicka
Copy link
Contributor Author

kasicka commented Aug 24, 2016

I'd like to use it for Fedora/EPEL and software collections packaging.

@dLobatog
Copy link
Member

Fair enough, I guess I'll create our own template for foreman-packaging and leave this as a general one. Thanks @kasicka ! Let me know if you need any more fixes or help with packaging for Fedora/EPEL, I'd be glad to help (newbie though 😄)

@dLobatog dLobatog merged commit 1e5c0c8 into theforeman:master Aug 24, 2016
@kasicka kasicka deleted the update-spec-template branch August 25, 2016 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants