On 25/09/17 14:30, Eric Engestrom wrote:
I pushed the rest of the series.
See below for discussion on this patch.
On Wednesday, 2017-09-20 17:05:21 +0000, Jose Fonseca wrote:
On 19/09/17 15:14, Eric Engestrom wrote:
Signed-off-by: Eric Engestrom <eric.engest...@imgtec.com>
---
scons/custom.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scons/custom.py b/scons/custom.py
index 0767ba936d410167116d..978ee5f9ec7c23a74cb9 100644
--- a/scons/custom.py
+++ b/scons/custom.py
@@ -257,7 +257,7 @@ def parse_source_list(env, filename, names=None):
sym_table = parser.parse(src.abspath)
if names:
- if isinstance(names, basestring):
+ if isinstance(names, str):
names = [names]
symbols = names
I'm not sure if this won't give the wrong results for unicode strings, but
at any rate, I don't think that should ever happen in practice.
Are you replying to Ilia [1] here?
He left a comment on this patch, saying:
This might be python3-compatible, but it's not the same thing. str !=
unicode. Not sure where "names" can come from, but if it can come in
as a unicode string, this won't work.
My knowledge of python is quite basic, so I'd rather you discuss between
you two rather than me trying to forward a conversation with each of you :P
[1] https://lists.freedesktop.org/archives/mesa-dev/2017-September/170130.html
I just checked all uses of ParseSourceList and it's never used with
unicode strings. It's always used with plain strings or lists of plain
strings.
So on 2nd thought, I think this patch should be safe. Ilia, would you
agree?
basestring doesn't exist on Python 3, so this will be necessary eventually.
BTW, thank you for looking into this Eric.
Jose
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
