All of lore.kernel.org
 help / color / mirror / Atom feed
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

  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.