----- Original Message -----
> From: "Jan Stancek" <jstan...@redhat.com>
> To: "Xiaoguang Wang" <wangxg.f...@cn.fujitsu.com>
> Cc: "Cyril Hrubis" <chru...@suse.cz>, ltp-list@lists.sourceforge.net
> Sent: Monday, 20 July, 2015 1:54:53 PM
> Subject: Re: [LTP] [PATCH] syscalls/getrandom: add configure check for 
> linux/random.h
> Hmm, I tried older RHEL5 release, which gave me:
>   configure: WARNING: linux/random.h: present but cannot be compiled
>   configure: WARNING: linux/random.h:     check for missing prerequisite

I see the difference. RHEL5.11GA has autoconf-2.59-12, which is too old
for LTP, it requires autoconf >= 2.61. Test we run internally upgrades
it to higher version.

If I use the absolute minimum of autoconf/automake versions, I get:
 configure: WARNING: linux/random.h: present but cannot be compiled
 configure: WARNING: linux/random.h:     check for missing prerequisite headers?
 configure: WARNING: linux/random.h: see the Autoconf documentation
 configure: WARNING: linux/random.h:     section "Present But Cannot Be 
Compiled"
 configure: WARNING: linux/random.h: proceeding with the preprocessor's result
 configure: WARNING: linux/random.h: in the future, the compiler will take 
precedence

Notice the last line. I think that's why it worked for me - I used version, 
where
it already took compiler result as default.

So I think your patch is still needed for older versions, but I'd remove
the call to getrandom() and keep the name "HAVE_LINUX_RANDOM_H".

Regards,
Jan

------------------------------------------------------------------------------
Don't Limit Your Business. Reach for the Cloud.
GigeNET's Cloud Solutions provide you with the tools and support that
you need to offload your IT needs and focus on growing your business.
Configured For All Businesses. Start Your Cloud Today.
https://www.gigenetcloud.com/
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to