All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mats Liljegren <mats.liljegren@enea.com>
To: Mike Frysinger <vapier@gentoo.org>
Cc: "ltp-list@lists.sourceforge.net" <ltp-list@lists.sourceforge.net>
Subject: Re: [LTP] [PATCH] ltp: Fix cross-compile issue
Date: Mon, 16 Feb 2015 08:15:22 +0000	[thread overview]
Message-ID: <0247700D01F14443B9209F90AD938CC524E65CBB@SESTOEX04.enea.se> (raw)
In-Reply-To: <20150216051704.GB4075@vapier>

My change never made it into the repository. Instead, Cyril prepended the path with ${SYSROOT}, where your cross-compile environment needs to setup SYSROOT path in order to make a proper cross-compile of LTP. See commit d4ab5d27d5ad666547d7b4d8e269e00921960c58. Cludgy, but it is simple and it works. This "=" thing I proposed apparently didn't work for everyone.

As for using pkg-config, I have no idea how that will work out in a cross-compile environment. But I guess it should be possible to go down that route as well...

/Mats
________________________________________
From: Mike Frysinger [vapier@gentoo.org]
Sent: Monday, February 16, 2015 6:17 AM
To: Mats Liljegren
Cc: ltp-list@lists.sourceforge.net
Subject: Re: [LTP] [PATCH] ltp: Fix cross-compile issue

On 16 Dec 2014 15:34, Mats Liljegren wrote:
> --- a/m4/ltp-tirpc.m4
> +++ b/m4/ltp-tirpc.m4
> @@ -25,7 +25,7 @@ AC_DEFUN([LTP_CHECK_TIRPC],[
>       TIRPC_LIBS=""
>
>       AC_CHECK_HEADER(tirpc/netconfig.h,[
> -             TIRPC_CPPFLAGS="-I/usr/include/tirpc"
> +             TIRPC_CPPFLAGS="-I=/usr/include/tirpc"
>               AC_DEFINE(HAVE_LIBTIRPC, 1, [Define to 1 if you have libtirpc headers installed])
>               AC_CHECK_LIB(tirpc, rpcb_set, [TIRPC_LIBS="-ltirpc"])])

this m4 is fatally flawed and tweaking the flags won't help.  we should switch
to use pkg-config here.  a cursory glance suggests acl/cap should also be
updated.
-mike

------------------------------------------------------------------------------
Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
from Actuate! Instantly Supercharge Your Business Reports and Dashboards
with Interactivity, Sharing, Native Excel Exports, App Integration & more
Get technology previously reserved for billion-dollar corporations, FREE
http://pubads.g.doubleclick.net/gampad/clk?id=190641631&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

  reply	other threads:[~2015-02-16  8:27 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-16 14:34 [LTP] [PATCH] ltp: Fix cross-compile issue Mats Liljegren
2014-12-16 14:34 ` Mats Liljegren
2014-12-17 12:09   ` Cyril Hrubis
     [not found]     ` <20141217154739.69e91b2a@mats-desktop>
2014-12-17 14:54       ` Cyril Hrubis
2015-02-16  5:17   ` Mike Frysinger
2015-02-16  8:15     ` Mats Liljegren [this message]
2015-02-16  9:10       ` Mike Frysinger
2015-02-17 16:49     ` Cyril Hrubis

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=0247700D01F14443B9209F90AD938CC524E65CBB@SESTOEX04.enea.se \
    --to=mats.liljegren@enea.com \
    --cc=ltp-list@lists.sourceforge.net \
    --cc=vapier@gentoo.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.