pcp
[Top] [All Lists]

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

To: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Subject: Re: [pcp] pmmgr build failure on 32-bit Fedora release 20 [was 17]
From: Nathan Scott <nathans@xxxxxxxxxx>
Date: Tue, 11 Mar 2014 20:51:13 -0400 (EDT)
Cc: pcp@xxxxxxxxxxx
Delivered-to: pcp@xxxxxxxxxxx
In-reply-to: <1322230163.25595355.1394572885929.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>
Reply-to: Nathan Scott <nathans@xxxxxxxxxx>
Thread-index: 5+Zio+zla1S6w6B5WsxJL3ecAfNntJk/4LOm
Thread-topic: pmmgr build failure on 32-bit Fedora release 20 [was 17]

----- 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?

cheers.

--
Nathan

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