-
Notifications
You must be signed in to change notification settings - Fork 937
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
OBS spec update for 5.1.0 release #1316
Conversation
…dists) Added workarround for buggy _sharedstatedir on openSUSE Leap. Fixed build error for RHEL7 dists Some SPEC clenup
%bcond_without sctp | ||
%bcond_without websocket | ||
%bcond_with sctp | ||
%bcond_with websocket | ||
%bcond_without xmlrpc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This reverts 612e96d. Unintentional?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This section is applicable to RHEL7 dist only, not CentOS7.
RHEL7 is not have libunistring package and build will fail.
Same is related to sctp.
RHEL not have lksctp-tools-devel
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What repo you used to build sctp
and websocket
modules on RHEL
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Created PR #1317
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Both libunistring-devel and lksctp-tools-devel are in the rhel7-server-optional repo, which my machine has enabled by default.
When bulding packages in EPEL they enable the same repo, ref
http://fedoraproject.org/wiki/EPEL/GuidelinesAndPolicies#Policy
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you help find way how to attache this optional repo on OpenSUSE build server.
I see OBS not provide this optional repo.
Without this optional repo we cannot build this packages on available infrastructure.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With a quick search I found some details at:
But not sure if that command can be used in OBS.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes Daniel
This is requires active RedHat support contact.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think add instruction how to build this modules to README file for RHEL dist.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@oyvindkolbu Please test instructions on commit 35f28d7
And let me know, please.
@sergey-safarov, @oyvindkolbu -- was all sorted out here? Should I merge this one? |
yes, Daniel. |
Also need do cherry-pick to 5.1 branch