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

Provide Copr releaser and fix building of nodejs-brace #1709

Closed
wants to merge 1,135 commits into from

Conversation

xsuchy
Copy link

@xsuchy xsuchy commented Jun 29, 2017

For plugin updates, please indicate which repos this should be built into:

  • Nightly
  • 1.15
  • 1.14
  • 1.13

Dominic Cleal and others added 30 commits January 27, 2014 09:52
- Fix few smaller issues (mhulan@redhat.com)
- Fix few smaller issues (mhulan@redhat.com)
- Merge pull request theforeman#62 from ares/master (ares@igloonet.cz)
- Fix dumping 'false' default values (mhulan@redhat.com)
- Merge pull request theforeman#62 from ares/master (ares@igloonet.cz)
- Fix dumping 'false' default values (mhulan@redhat.com)
- Display fatal errors on STDOUT (mhulan@redhat.com)
- Display fatal errors on STDOUT (mhulan@redhat.com)
- Update to v0.0.2 (dcleal@redhat.com)
- Bump to 0.0.18 (mbacovsk@redhat.com)
- Bump to 0.0.16 (mbacovsk@redhat.com)
- Bump to 0.0.18 (mbacovsk@redhat.com)
- Bump to 0.0.17 (mbacovsk@redhat.com)
- Bump to 0.0.18 (mbacovsk@redhat.com)
- Bump to 0.0.16 (mbacovsk@redhat.com)
- Bump to 0.0.18 (mbacovsk@redhat.com)
- Bump to 0.0.17 (mbacovsk@redhat.com)
- Bump to 0.1.11 (mbacovsk@redhat.com)
- Bump to 0.1.11 (mbacovsk@redhat.com)
- Bump to 0.1.10 (mbacovsk@redhat.com)
- Bump to 0.1.11 (mbacovsk@redhat.com)
- Bump to 0.1.10 (mbacovsk@redhat.com)
- Fix --no-colors issues with older highline (mhulan@redhat.com)
- Fix --no-colors issues with older highline (mhulan@redhat.com)
- rebase to 2.6.2 release (dcleal@redhat.com)
- rebase to 2.6.0 release (msuchy@redhat.com)
- Update to v0.1.4 (dcleal@redhat.com)
- Update to v0.1.4 (dcleal@redhat.com)
Adding foreman_reserve plugin to comps
- Rebase to rbovirt 0.0.22 (dcleal@redhat.com)
- rebase to 2.6.3 release (dcleal@redhat.com)
- Rebase to rbovirt 0.0.22 (dcleal@redhat.com)
- Rebase to rbovirt 0.0.23 (dcleal@redhat.com)
- Rebase to rbovirt 0.0.23 (dcleal@redhat.com)
@@ -1 +1 @@
0.10.0-1.fm1_15 nodejs-brace/
0.10.0-2.fm1_16 nodejs-brace/
Copy link
Member

Choose a reason for hiding this comment

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

This is unneeded, tito will generate this when we tag.

Copy link
Author

Choose a reason for hiding this comment

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

I done it with tito tag this PR include even the tag.

Copy link
Member

Choose a reason for hiding this comment

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

How do I see that the PR includes a tag? I didn't even think it was possible to push a tag to a PR? I appreciate trying to get ahead, but I honestly think its easier to just revert the tito tag and let us do that when we do a release post merge.

@@ -53,6 +52,9 @@ cp -pf LICENSE README.md ../../
%doc README.md

%changelog
* Thu Jun 29 2017 Miroslav Suchý <msuchy@redhat.com> 0.10.0-2
- correctly provides bundled() (msuchy@redhat.com)
Copy link
Member

Choose a reason for hiding this comment

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

This is unneeded, tito will generate this when we tag.

@@ -22,8 +22,7 @@ ExclusiveArch: %{nodejs_arches} noarch
ExclusiveArch: %{ix86} x86_64 %{arm} noarch
%endif
Provides: npm(%{npm_name}) = %{version}
Provides: bundled-npm(brace) = 0.10.0
Copy link
Member

Choose a reason for hiding this comment

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

Intentional to remove brace dependency?

Copy link
Author

Choose a reason for hiding this comment

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

ahh npm(brace) != nodejs(brace) good catch

@ehelms
Copy link
Member

ehelms commented Jul 1, 2017

[test]

@xsuchy
Copy link
Author

xsuchy commented Jul 11, 2017

removed the tag and updated

@mmoll
Copy link

mmoll commented Jul 11, 2017

[test]

@lzap
Copy link
Member

lzap commented Jul 12, 2017

So I assume this will work with all the packages except the special ones we have extra releases written for (foreman, foreman-installer, smart-proxy...), will it?

The webpack build error is unrelated. +1

foreman/README Outdated
@@ -0,0 +1,2 @@
this package can be built using:
tito release copr-nightly --arg jenkins_job=test_develop
Copy link
Member

Choose a reason for hiding this comment

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

I think this would live better in a similar section to https://github.com/theforeman/foreman-packaging/blob/rpm/develop/README.md#with-koji-access as its likely to just get lost here and be thought of as a source file.

@mmoll
Copy link

mmoll commented Aug 19, 2017

@xsuchy please rebase

Copy link
Member

@dLobatog dLobatog left a comment

Choose a reason for hiding this comment

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

I agree with @ehelms on the need to move the README file to the general README. @xsuchy Are you still interested on maintaining this Copr? Who else has access to it?

@xsuchy
Copy link
Author

xsuchy commented Sep 26, 2017

?? I rebased and this get closed?
I will create new one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
9 participants