From: Mike Frysinger <vapier@gentoo.org>
To: alexey.kodanev@oracle.com
Cc: vasily.isaenko@oracle.com, ltp-list@lists.sourceforge.net
Subject: Re: [LTP] [PATCH] configure: add configure checks to compile kernel modules
Date: Tue, 11 Jun 2013 13:04:45 -0400 [thread overview]
Message-ID: <201306111304.47061.vapier@gentoo.org> (raw)
In-Reply-To: <51B6EF41.5090702@oracle.com>
[-- Attachment #1.1: Type: Text/Plain, Size: 723 bytes --]
On Tuesday 11 June 2013 05:34:57 alexey.kodanev@oracle.com wrote:
> On 06/11/2013 05:31 AM, Mike Frysinger wrote:
> > defaulting like this won't work. you need to check $cross_compiling is
> > set to yes in order to default to `uname -r`.
> >
> > seems like it'd be simpler to just do:
> > AC_ARG_WITH([linux-version], [AC_HELP_STRING(...)],,
> > AS_IF([test "$cross_compiling" != no],
> > [with_linux_version=`uname -r`]))
> > LINUX_VERSION=$with_linux_version
> > AC_SUBST(LINUX_VERSION)
>
> May be defaulting it to `uname -r` when $cross_compiling is set to no?
yeah, i inverted the check here (i didn't test any of the code ... just typed
it into the e-mail). but you get the idea.
-mike
[-- Attachment #1.2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
[-- Attachment #2: Type: text/plain, Size: 184 bytes --]
------------------------------------------------------------------------------
This SF.net email is sponsored by Windows:
Build for Windows Store.
http://p.sf.net/sfu/windows-dev2dev
[-- Attachment #3: Type: text/plain, Size: 155 bytes --]
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
next prev parent reply other threads:[~2013-06-11 17:05 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-06 14:21 [LTP] [PATCH] configure: add configure checks to compile kernel modules Alexey Kodanev
2013-06-11 1:31 ` Mike Frysinger
2013-06-11 9:34 ` alexey.kodanev
2013-06-11 17:04 ` Mike Frysinger [this message]
2013-06-11 1:32 ` Mike Frysinger
-- strict thread matches above, loose matches on Subject: below --
2013-06-06 14:13 Alexey Kodanev
2013-06-05 12:01 Alexey Kodanev
2013-06-06 10:57 ` chrubis
[not found] ` <51B0716B.2090107@oracle.com>
2013-06-06 13:56 ` chrubis
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=201306111304.47061.vapier@gentoo.org \
--to=vapier@gentoo.org \
--cc=alexey.kodanev@oracle.com \
--cc=ltp-list@lists.sourceforge.net \
--cc=vasily.isaenko@oracle.com \
/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.