scons: Don't use isdir in AddLocalRPATH.
authorGabe Black <gabeblack@google.com>
Tue, 12 Mar 2019 12:00:41 +0000 (05:00 -0700)
committerGabe Black <gabeblack@google.com>
Tue, 12 Mar 2019 15:43:38 +0000 (15:43 +0000)
commit91195ae7f637d1d4879cc3bf0860147333846e75
treec1638f69aaf37fd0dfd84e606fb6a08c92db08c6
parent4ec5b85a31ded2afdf784555a7c736e61c8b594a
scons: Don't use isdir in AddLocalRPATH.

isdir isn't a nice way to check if an FS.Base is a File or a Dir as was
initially assumed, it literally checks if a path can be stat-ed and is
reported as a directory by stat. This means that if a directory is
going to be created as part of the build, the result of that test will
change depending on whether that part of the build has happened
successfully before.

A better check which behaves as originally intended is to check whether
the Node is an instance of the SCons.Node.FS.Dir class.

Change-Id: Id041917d50b768a8205769c0a05320f92b09993c
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/17128
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Gabe Black <gabeblack@google.com>
SConstruct