Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dotnQ-0001ka-Nf for pgsql-pkg-yum@arkaria.postgresql.org; Mon, 04 Sep 2017 16:00:20 +0000 Received: from localhost ([127.0.0.1] helo=postgresql.org) by malur.postgresql.org with smtp (Exim 4.84_2) (envelope-from ) id 1dotnP-0004J5-MC for pgsql-pkg-yum@arkaria.postgresql.org; Mon, 04 Sep 2017 16:00:19 +0000 Received: from makus.postgresql.org ([2001:4800:1501:1::229]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.84_2) (envelope-from ) id 1dotnP-0004G2-29 for pgsql-pkg-yum@postgresql.org; Mon, 04 Sep 2017 16:00:19 +0000 Received: from dd6038.kasserver.com ([85.13.130.233]) by makus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.84_2) (envelope-from ) id 1dotnI-000502-0Z for pgsql-pkg-yum@postgresql.org; Mon, 04 Sep 2017 16:00:17 +0000 Received: from niheb.land.credativ.lan (fw-front.credativ.com [62.154.226.94]) by dd6038.kasserver.com (Postfix) with ESMTPSA id A24EF2DC008C for ; Mon, 4 Sep 2017 18:00:08 +0200 (CEST) Message-ID: <1504540808.3248.44.camel@oopsware.de> Subject: RPMs for ppc64/ppc64le From: Bernd Helmle To: pgsql-pkg-yum@postgresql.org Date: Mon, 04 Sep 2017 18:00:08 +0200 Content-Type: multipart/mixed; boundary="=-47rxMkmMdhrj+S0WdpQD" X-Mailer: Evolution 3.24.5 (3.24.5-1.fc26) Mime-Version: 1.0 List-Archive: List-Help: List-ID: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: X-Mailing-List: pgsql-pkg-yum Precedence: bulk Sender: pgsql-pkg-yum-owner@postgresql.org --=-47rxMkmMdhrj+S0WdpQD Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Folks POWER support is finally there, that's great and thanks for all the guys involved to make that happen! However, i found the way how the SPEC file define the build parameters for POWER not very usable at the moment. Currently, we build per default with the IBM Advanced Toolchain (AT) if ppc64 and ppc64le architecture is defined and use those definitions in all SPEC files. I've got complaints from some people building from upstream source packages on their ppc64/ppc64le infrastructure, that the current behavior is suboptimal if you don't want to rely on IBM AT (not everyone is convinced that it is better to build with higher optimization levels) or even power8 CPU instructions. E.g. there are still users out there running PostgreSQL on dozens of POWER7 machines and the current behavior of the SPEC files breaks their build environments. The latter makes it currently hard on those older POWER CPUs to build packages based on current upstream source rpms, cause they install but can't be executed without patching all the SPEC files. There are several solutions to this situation which come to my mind which i want to discuss. Maybe there are some others, better ones, but let's start with this: The current behavior forces you to use the IBM Advanced Toolchain, so what i would like to see is to turn that feature easily off. We could make the procedure to choose the IBM AT conditional with the default to on , like i've done in the attached patch. Though this requires to specialize all SPEC files currently supported on ppc64/ppc64le. It basically moves all tests for ppc64/ppc64le into a single conditional and summarizes them with a new macro %ibmpower. That's used afterwards for checking if IBM AT or special flags are required. This allows rpmbuild to deselect the default IBM AT based build with --define "ibmpower 0", enabling the default compiler to be used. Another solution that come to my mind: use a single centralized rpmrc passed to rpmbuild (via --rcfile) during the build process. This file defines all necessary macros to build the packages like now, but in a single place. This would make it better maintainable in the future, say, when POWER9 comes along and would allow to easily override default settings with custom ones. We could make that a special setting to make (e.g RPMRC= make rpm96), but i haven't dug too deep into that approach yet. I'm willing to work on that, if we agree that we want to improve in that area. Opinions? Thanks, Bernd --=-47rxMkmMdhrj+S0WdpQD Content-Disposition: attachment; filename="power_optional.patch" Content-Type: text/x-patch; name="power_optional.patch"; charset="UTF-8" Content-Transfer-Encoding: base64 ZGlmZiAtLWdpdCBhL3JwbS9yZWRoYXQvOS42L3Bvc3RncmVzcWwvbWFzdGVy L3Bvc3RncmVzcWwtOS42LnNwZWMgYi9ycG0vcmVkaGF0LzkuNi9wb3N0Z3Jl c3FsL21hc3Rlci9wb3N0Z3Jlc3FsLTkuNi5zcGVjCmluZGV4IGU0ZGQ2MDcu LjIyNjBjMjcgMTAwNjQ0Ci0tLSBhL3JwbS9yZWRoYXQvOS42L3Bvc3RncmVz cWwvbWFzdGVyL3Bvc3RncmVzcWwtOS42LnNwZWMKKysrIGIvcnBtL3JlZGhh dC85LjYvcG9zdGdyZXNxbC9tYXN0ZXIvcG9zdGdyZXNxbC05LjYuc3BlYwpA QCAtNSw2ICs1LDEwIEBACiAlZ2xvYmFsIHBncGFja2FnZXZlcnNpb24gOS42 CiAlZ2xvYmFsIHBnaW5zdGRpciAvdXNyL3Bnc3FsLSV7cGdwYWNrYWdldmVy c2lvbn0KIAorIyB1c2UgSUJNIEFkdmFuY2VkIFRvb2xjaGFpbiBwZXIgZGVm YXVsdAorJWlmYXJjaCBwcGM2NCBwcGM2NGxlCisleyE/aWJtcG93ZXI6JWds b2JhbCBpYm1wb3dlciAxfQorJWVuZGlmCiAKICVnbG9iYWwgYmV0YSAwCiAl ez9iZXRhOiVnbG9iYWwgX19vc19pbnN0YWxsX3Bvc3QgL3Vzci9saWIvcnBt L2JycC1jb21wcmVzc30KQEAgLTU0LDcgKzU4LDcgQEAKICV7IT9zZWxpbnV4 OiVnbG9iYWwgc2VsaW51eCAwfQogJWVsc2UKICV7IT9zeXN0ZW1kX2VuYWJs ZWQ6JWdsb2JhbCBzeXN0ZW1kX2VuYWJsZWQgMX0KLSVpZmFyY2ggcHBjNjQg cHBjNjRsZQorJWlmICVpYm1wb3dlcgogJXshP3NkdDolZ2xvYmFsIHNkdCAw fQogJWVsc2UKICV7IT9zZHQ6JWdsb2JhbCBzZHQgMX0KQEAgLTY1LDcgKzY5 LDcgQEAKICVnbG9iYWwgX2hhcmRlbmVkX2J1aWxkIDEKICVlbmRpZgogCi0l aWZhcmNoIHBwYzY0IHBwYzY0bGUKKyVpZiAlaWJtcG93ZXIKICMgRGVmaW5l IHRoZSBBVCB2ZXJzaW9uIGFuZCBwYXRoLgogJWdsb2JhbCBhdHN0cmluZwlh dDEwLjAKICVnbG9iYWwgYXRwYXRoCQkvb3B0LyV7YXRzdHJpbmd9CkBAIC0x MDksNyArMTEzLDcgQEAgUGF0Y2g2OgkJJXtzbmFtZX0tJXtwZ21ham9ydmVy c2lvbn0tcGVybC1ycGF0aC5wYXRjaAogQnVpbGRSZXF1aXJlczoJcGVybCBn bGliYy1kZXZlbCBiaXNvbiBmbGV4ID49IDIuNS4zMQogQnVpbGRSZXF1aXJl czoJcGVybChFeHRVdGlsczo6TWFrZU1ha2VyKQogCi0laWZhcmNoIHBwYzY0 IHBwYzY0bGUKKyVpZiAlaWJtcG93ZXIKIEJ1aWxkUmVxdWlyZXM6CWFkdmFu Y2UtdG9vbGNoYWluLSV7YXRzdHJpbmd9LWRldmVsCiAlZW5kaWYKIApAQCAt MTQyLDcgKzE0Niw3IEBAIEJ1aWxkUmVxdWlyZXM6CXpsaWItZGV2ZWwgPj0g MS4wLjQKICVpZiAlc3NsCiAjIFdlIGRlcGVuZCB1biB0aGUgU1NMIGxpYnJh cmllcyBwcm92aWRlZCBieSBBZHZhbmNlIFRvb2xjaGFpbiBvbiBQUEMsCiAj IHNvIHVzZSBvcGVuc3NsLWRldmVsIG9ubHkgb24gb3RoZXIgcGxhdGZvcm1z OgotJWlmbmFyY2ggcHBjNjQgcHBjNjRsZQorJWlmIDAlez9pYm1wb3dlcn0K IEJ1aWxkUmVxdWlyZXM6CW9wZW5zc2wtZGV2ZWwKICVlbmRpZgogJWVuZGlm CkBAIC0yMjksNyArMjMzLDcgQEAgUmVxdWlyZXMocG9zdHVuKToJJXtfc2Jp bmRpcn0vdXBkYXRlLWFsdGVybmF0aXZlcwogQnVpbGRSb290Ogkle190bXBw YXRofS8le25hbWV9LSV7dmVyc2lvbn0tJXtyZWxlYXNlfS1yb290LSUoJXtf X2lkX3V9IC1uKQogUHJvdmlkZXM6CXBvc3RncmVzcWwKIAotJWlmYXJjaCBw cGM2NCBwcGM2NGxlCislaWYgJWlibXBvd2VyCiBBdXRvUmVxOgkwCiBSZXF1 aXJlczoJYWR2YW5jZS10b29sY2hhaW4tJXthdHN0cmluZ30tcnVudGltZQog JWVuZGlmCkBAIC0yNTIsNyArMjU2LDcgQEAgU3VtbWFyeToJVGhlIHNoYXJl ZCBsaWJyYXJpZXMgcmVxdWlyZWQgZm9yIGFueSBQb3N0Z3JlU1FMIGNsaWVu dHMKIEdyb3VwOgkJQXBwbGljYXRpb25zL0RhdGFiYXNlcwogUHJvdmlkZXM6 CXBvc3RncmVzcWwtbGlicyA9ICV7cGdwYWNrYWdldmVyc2lvbn0KIAotJWlm YXJjaCBwcGM2NCBwcGM2NGxlCislaWYgJWlibXBvd2VyCiBBdXRvUmVxOgkw CiBSZXF1aXJlczoJYWR2YW5jZS10b29sY2hhaW4tJXthdHN0cmluZ30tcnVu dGltZQogJWVuZGlmCkBAIC0yODksNyArMjkzLDcgQEAgUmVxdWlyZXM6CS91 c3Ivc2Jpbi91c2VyYWRkLCAvc2Jpbi9jaGtjb25maWcKICVlbmRpZgogUHJv dmlkZXM6CXBvc3RncmVzcWwtc2VydmVyCiAKLSVpZmFyY2ggcHBjNjQgcHBj NjRsZQorJWlmICVpYm1wb3dlcgogQXV0b1JlcToJMAogUmVxdWlyZXM6CWFk dmFuY2UtdG9vbGNoYWluLSV7YXRzdHJpbmd9LXJ1bnRpbWUKICVlbmRpZgpA QCAtMzE5LDcgKzMyMyw3IEBAIFJlcXVpcmVzOgkle25hbWV9JXs/X2lzYX0g PSAle3ZlcnNpb259LSV7cmVsZWFzZX0KIFJlcXVpcmVzOgkle25hbWV9LWxp YnMlez9faXNhfSA9ICV7dmVyc2lvbn0tJXtyZWxlYXNlfQogUHJvdmlkZXM6 CXBvc3RncmVzcWwtY29udHJpYgogCi0laWZhcmNoIHBwYzY0IHBwYzY0bGUK KyVpZiAlaWJtcG93ZXIKIEF1dG9SZXE6CTAKIFJlcXVpcmVzOglhZHZhbmNl LXRvb2xjaGFpbi0le2F0c3RyaW5nfS1ydW50aW1lCiAlZW5kaWYKQEAgLTM1 OCwxMyArMzYyLDEzIEBAIFN1bW1hcnk6CVRoZSBQZXJsIHByb2NlZHVyYWwg bGFuZ3VhZ2UgZm9yIFBvc3RncmVTUUwKIEdyb3VwOgkJQXBwbGljYXRpb25z L0RhdGFiYXNlcwogUmVxdWlyZXM6CSV7bmFtZX0tc2VydmVyJXs/X2lzYX0g PSAle3ZlcnNpb259LSV7cmVsZWFzZX0KIFJlcXVpcmVzOglwZXJsKDpNT0RV TEVfQ09NUEFUXyUoZXZhbCAiYCV7X19wZXJsfSAtVjp2ZXJzaW9uYCI7IGVj aG8gJHZlcnNpb24pKQotJWlmYXJjaCBwcGMgcHBjNjQKKyVpZiAlaWJtcG93 ZXIKIEJ1aWxkUmVxdWlyZXM6CXBlcmwtZGV2ZWwKICVlbmRpZgogT2Jzb2xl dGVzOglwb3N0Z3Jlc3FsJXtwZ21ham9ydmVyc2lvbn0tcGwKIFByb3ZpZGVz Oglwb3N0Z3Jlc3FsLXBscGVybAogCi0laWZhcmNoIHBwYzY0IHBwYzY0bGUK KyVpZiAlaWJtcG93ZXIKIEF1dG9SZXE6CTAKIFJlcXVpcmVzOglhZHZhbmNl LXRvb2xjaGFpbi0le2F0c3RyaW5nfS1ydW50aW1lCiAlZW5kaWYKQEAgLTM4 NSw3ICszODksNyBAQCBSZXF1aXJlczoJJXtuYW1lfS1zZXJ2ZXIlez9faXNh fSA9ICV7dmVyc2lvbn0tJXtyZWxlYXNlfQogT2Jzb2xldGVzOgkle25hbWV9 LXBsCiBQcm92aWRlczoJcG9zdGdyZXNxbC1wbHB5dGhvbgogCi0laWZhcmNo IHBwYzY0IHBwYzY0bGUKKyVpZiAlaWJtcG93ZXIKIEF1dG9SZXE6CTAKIFJl cXVpcmVzOglhZHZhbmNlLXRvb2xjaGFpbi0le2F0c3RyaW5nfS1ydW50aW1l CiAlZW5kaWYKQEAgLTQwNiw3ICs0MTAsNyBAQCBSZXF1aXJlczoJJXtuYW1l fS1zZXJ2ZXIlez9faXNhfSA9ICV7dmVyc2lvbn0tJXtyZWxlYXNlfQogT2Jz b2xldGVzOgkle25hbWV9LXBsCiBQcm92aWRlczoJcG9zdGdyZXNxbC1wbHB5 dGhvbjMKIAotJWlmYXJjaCBwcGM2NCBwcGM2NGxlCislaWYgJWlibXBvd2Vy CiBBdXRvUmVxOgkwCiBSZXF1aXJlczoJYWR2YW5jZS10b29sY2hhaW4tJXth dHN0cmluZ30tcnVudGltZQogJWVuZGlmCkBAIC00MjgsNyArNDMyLDcgQEAg UmVxdWlyZXM6CXRjbAogT2Jzb2xldGVzOgkle25hbWV9LXBsCiBQcm92aWRl czoJcG9zdGdyZXNxbC1wbHRjbAogCi0laWZhcmNoIHBwYzY0IHBwYzY0bGUK KyVpZiAlaWJtcG93ZXIKIEF1dG9SZXE6CTAKIFJlcXVpcmVzOglhZHZhbmNl LXRvb2xjaGFpbi0le2F0c3RyaW5nfS1ydW50aW1lCiAlZW5kaWYKQEAgLTQ0 Nyw3ICs0NTEsNyBAQCBSZXF1aXJlczoJJXtuYW1lfS1zZXJ2ZXIlez9faXNh fSA9ICV7dmVyc2lvbn0tJXtyZWxlYXNlfQogUmVxdWlyZXM6CSV7bmFtZX0t ZGV2ZWwlez9faXNhfSA9ICV7dmVyc2lvbn0tJXtyZWxlYXNlfQogUHJvdmlk ZXM6CXBvc3RncmVzcWwtdGVzdAogCi0laWZhcmNoIHBwYzY0IHBwYzY0bGUK KyVpZiAlaWJtcG93ZXIKIEF1dG9SZXE6CTAKIFJlcXVpcmVzOglhZHZhbmNl LXRvb2xjaGFpbi0le2F0c3RyaW5nfS1ydW50aW1lCiAlZW5kaWYKQEAgLTQ4 Miw3ICs0ODYsNyBAQCBiZW5jaG1hcmtzLgogJWVuZGlmCiAKIENGTEFHUz0i JHtDRkxBR1M6LSVvcHRmbGFnc30iCi0laWZhcmNoIHBwYzY0IHBwYzY0bGUK KyVpZiAlaWJtcG93ZXIKIAlDRkxBR1M9IiR7Q0ZMQUdTfSAkKGVjaG8gJXtf X2dsb2JhbF9jZmxhZ3N9IHwgc2VkICdzLy1PMi8tTzMvZycpIC1tNjQgLW1j cHU9cG93ZXI4IC1tdHVuZT1wb3dlcjggLUkle2F0cGF0aH0vaW5jbHVkZSIK IAlDWFhGTEFHUz0iJHtDWFhGTEFHU30gJChlY2hvICV7X19nbG9iYWxfY2Zs YWdzfSB8IHNlZCAncy8tTzIvLU8zL2cnKSAtbTY0IC1tY3B1PXBvd2VyOCAt bXR1bmU9cG93ZXI4IC1JJXthdHBhdGh9L2luY2x1ZGUiCiAJTERGTEFHUz0i LUwle2F0cGF0aH0vJXtfbGlifSIKQEAgLTU3MSw3ICs1NzUsNyBAQCBleHBv cnQgUFlUSE9OPS91c3IvYmluL3B5dGhvbjMKICVpZiAle3N5c3RlbWRfZW5h YmxlZH0KIAktLXdpdGgtc3lzdGVtZCBcCiAlZW5kaWYKLSVpZmFyY2ggcHBj NjQgcHBjNjRsZQorJWlmICVpYm1wb3dlcgogCS0td2l0aC1pbmNsdWRlcz0l e2F0cGF0aH0vaW5jbHVkZSBcCiAJLS13aXRoLWxpYnJhcmllcz0le2F0cGF0 aH0vbGliNjQgXAogJWVuZGlmCkBAIC02NjIsNyArNjY2LDcgQEAgdW5zZXQg UFlUSE9OCiAlaWYgJXtzeXN0ZW1kX2VuYWJsZWR9CiAJLS13aXRoLXN5c3Rl bWQgXAogJWVuZGlmCi0laWZhcmNoIHBwYzY0IHBwYzY0bGUKKyVpZiAlaWJt cG93ZXIKIAktLXdpdGgtaW5jbHVkZXM9JXthdHBhdGh9L2luY2x1ZGUgXAog CS0td2l0aC1saWJyYXJpZXM9JXthdHBhdGh9L2xpYjY0IFwKICVlbmRpZgo= --=-47rxMkmMdhrj+S0WdpQD Content-Type: text/plain Content-Disposition: inline Content-Transfer-Encoding: 8bit MIME-Version: 1.0 -- Sent via pgsql-pkg-yum mailing list (pgsql-pkg-yum@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-pkg-yum --=-47rxMkmMdhrj+S0WdpQD--