Please consider the following change to the config.h for ruby. This sets the config-ppc64le properly for the ppc64le arch. index ce501db..b34e398 100644 --- a/config.h +++ b/config.h @@ -20,7 +20,12 @@ #elif defined(__ia64__) #include "ruby/config-ia64.h" #elif defined(__powerpc64__) +#include <endian.h> +#if __BYTE_ORDER__ == __ORDER_BIG_ENDIAN__ #include "ruby/config-ppc64.h" +#else +#include "ruby/config-ppc64le.h" +#endif #elif defined(__powerpc__) #include "ruby/config-ppc.h" #elif defined(__s390x__)
I'd rather have the patch get pushed upstream instead of having us carry it. Is there any reason it can't be included in ruby-core? If it can't be included in core, please tell us the source of the patch or whether you're the author.
Hi Sam, thanks for the speed response. I thought initially that the config.h was part of the fedora package (there are other packages that do similar) but as you point out, it is part of the upstream package. I'm happy to take this upstream but I suspect they won't drop a release as quick as I like. When I make the upstream submission, can I point you at it and ask that you temporarily bring it in?
Yep, we can absolutely do that. If the patch gets accepted then we can patch Fedora's ruby temporarily until we upgrade to the last release of core. If the patch doesn't get accepted by we deem it acceptable to carry with Fedora then that's another option, too. We always try to work upstream first, though. Just so I know - where did the patch come from? Are you the author? Is this picked out of someone else's repo?
I'm assigning this to myself so I can track it.
(In reply to Sam Kottler from comment #1) Sam, config.h is ours. And you can read through argument with upstream about this header file in bug 975660 though. (In reply to baude from comment #0) Thanks for the patch. It is committed in dist-git [1] for now. Do you need fresh build? Do you need this patch also for older releases? I'd prefer to keep it just in Rawhide. [1] http://pkgs.fedoraproject.org/cgit/ruby.git/commit/?id=c4c64c841ce1b36e531038f235af66162c14b5a0
If you just nvr bump rawhide, I'm happy. Thanks!
Yes, I did.