Very good question. Thanks for giving it a bash.

Could you have a look at what goes wrong in the sconslog? I've tested it on 
quite a flavour of systems, but obviously that was not enough.


From: Nilay Vaish <[email protected]<mailto:[email protected]>>
Reply-To: Nilay Vaish <[email protected]<mailto:[email protected]>>
Date: Friday, 28 June 2013 06:26
To: Nilay Vaish <[email protected]<mailto:[email protected]>>, Andreas Hansson 
<[email protected]<mailto:[email protected]>>, Default 
<[email protected]<mailto:[email protected]>>
Subject: Re: Review Request 1933: scons: Use python-config instead of distutils

This is an automatically generated e-mail. To reply, visit: 
http://reviews.gem5.org/r/1933/


Andreas, I was testing this patch on my machine and I got the following error. 
Is the problem at my
end or is something wrong with the patch?

[nilay@ribera] (26)$ scons -j 15 --no-lto PROTOCOL=MESI_CMP_directory 
build/X86_MESI_CMP_directory/gem5.debug
scons: Reading SConscript files ...
Checking for leading underscore in global variables...(cached) no
Checking for C header file Python.h... (cached) yes
Checking for C library pthread... yes
Checking for C library dl... yes
Checking for C library util... yes
Checking for C library m... yes
Checking for C library python2.7... no
Error: can't find library python2.7 required by python


- Nilay Vaish


On June 27th, 2013, 2:11 p.m. UTC, Andreas Hansson wrote:

Review request for Default.
By Andreas Hansson.

Updated June 27, 2013, 2:11 p.m.

Repository: gem5
Description

Changeset 9798:320114314d30
---------------------------
scons: Use python-config instead of distutils

This patch changes how we determine the Python-related compiler and
linker flags. The previous approach used the internal LINKFORSHARED
which is not intended as part of the external API
(http://bugs.python.org/issue3588) and causes failures on recent OSX
installations.

Instead of using distutils we now rely on python-config and scons
ParseConfig. For backwards compatibility we also parse out the
includes and libs although this could safely be dropped. The drawback
of this patch is that Python 2.5 is now required, but hopefully that
is an acceptable compromise as any system with gcc 4.4 most likely
will have Python >= 2.5.



Testing

Compiled on OSX 10.8, RHE5, Ubuntu 10/12.04, OpenSUSE 12.3, Fedora19


Diffs

 *   SConstruct (9cd5f91e7a79)

View Diff<http://reviews.gem5.org/r/1933/diff/>


-- IMPORTANT NOTICE: The contents of this email and any attachments are 
confidential and may also be privileged. If you are not the intended recipient, 
please notify the sender immediately and do not disclose the contents to any 
other person, use it for any purpose, or store or copy the information in any 
medium. Thank you.
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to