-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Fixup tito tagger and builder libs #11196
Conversation
[test] |
1 similar comment
[test] |
Evaluated for origin test up to 606d580 |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/9583/) |
@sdodson @maxamillion Does this look ok to you? |
Looks good to me. so ldflags is completely gone now! yay |
Yep. They've been out of the spec file for a while, I just never got around to removing the code. |
[merge] |
1 similar comment
[merge] |
@openshift-bot, the last build failed from the following flakes:
re[merge] |
Evaluated for origin merge up to 606d580 |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/9720/) (Image: devenv-rhel7_5142) |
Update origin.spec os_git_vars on tag
Do not do -dirty during rpm builds
Clean out build flags commands since they are no longer used in origin.spec file