pcp
[Top] [All Lists]

Re: [pcp] pmmgr build failure on 32-bit Fedora release 20 [was 17]

To: Nathan Scott <nathans@xxxxxxxxxx>
Subject: Re: [pcp] pmmgr build failure on 32-bit Fedora release 20 [was 17]
From: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date: Thu, 13 Mar 2014 08:47:25 +1100
Cc: pcp@xxxxxxxxxxx
Delivered-to: pcp@xxxxxxxxxxx
In-reply-to: <1924983593.25657501.1394585473440.JavaMail.zimbra@xxxxxxxxxx>
References: <53101471.4000200@xxxxxxxxxxxxxxxx> <20140305150207.GB27666@xxxxxxxxxx> <5317BE58.6020603@xxxxxxxxxxxxxxxx> <20140306020106.GA6053@xxxxxxxxxx> <531EC7C8.3060109@xxxxxxxxxxxxxxxx> <20140311120502.GA13756@xxxxxxxxxx> <531F6B2E.4050602@xxxxxxxxxxxxxxxx> <1322230163.25595355.1394572885929.JavaMail.zimbra@xxxxxxxxxx> <1924983593.25657501.1394585473440.JavaMail.zimbra@xxxxxxxxxx>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.3.0
On 12/03/14 11:51, Nathan Scott wrote:


----- Original Message -----
----- Original Message -----
[...]
Nathan, at your next git pull from my tree you should see these two
commits flow, so no need to chase 'em in Frank's fche/i686 branch.

Little review comment; this bit could be further tidied...

+# We used to override --target=FOO, but it's better to let the
+# local build distro defaults do their jobs.
  pack_pcp : rpmmacros $(SPEC)
         DEFS=`grep '^--define' rpmmacros`; \
         eval $(RPMPROG) -ba $$DEFS \
-               --target $(TARGET_CPU)-$(TARGET_VENDOR)-$(TARGET_OS) \

That was the last reference to TARGET_CPU and TARGET_VENDOR (all the
way back to configure AFAICT)... they could possibly be removed now?

Good catch Nathan.

I am testing a change that
(a) removes 'em from builddefs
(b) stops munging with the (pre-defined) $target_cpu and $build_cpu definitions in configure.in

I'll push the commit if this works.

<Prev in Thread] Current Thread [Next in Thread>