[Patch] BaseTools: Fix a ParseDevPathValue function issue.

[Patch] BaseTools: Fix a ParseDevPathValue function issue.

[Feng, Bob C]

In ParseDevPathValue, the binary data need to convert to string
for python3 and python2 compatiblity.

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Bob Feng <bob.c.feng@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
---
 BaseTools/Source/Python/Common/Misc.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/BaseTools/Source/Python/Common/Misc.py b/BaseTools/Source/Python/Common/Misc.py
index 565a5ad40b..c7daf5417c 100644
--- a/BaseTools/Source/Python/Common/Misc.py
+++ b/BaseTools/Source/Python/Common/Misc.py
@@ -1030,10 +1030,11 @@ def ParseFieldValue (Value):
             subprocess._cleanup()
             p.stdout.close()
             p.stderr.close()
         if err:
             raise BadExpression("DevicePath: %s" % str(err))
+        out = out.decode()
         Size = len(out.split())
         out = ','.join(out.split())
         return '{' + out + '}', Size
 
     if "{CODE(" in Value:
-- 
2.18.0.windows.1

Re: [Patch] BaseTools: Fix a ParseDevPathValue function issue.

[Gao, Liming]

Reviewed-by: Liming Gao <liming.gao@intel.com>

>-----Original Message-----
>From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
>Feng, Bob C
>Sent: Saturday, February 16, 2019 11:38 PM
>To: edk2-devel@lists.01.org
>Cc: Gao, Liming <liming.gao@intel.com>
>Subject: [edk2] [Patch] BaseTools: Fix a ParseDevPathValue function issue.
>
>In ParseDevPathValue, the binary data need to convert to string
>for python3 and python2 compatiblity.
>
>Contributed-under: TianoCore Contribution Agreement 1.1
>Signed-off-by: Bob Feng <bob.c.feng@intel.com>
>Cc: Liming Gao <liming.gao@intel.com>
>---
> BaseTools/Source/Python/Common/Misc.py | 1 +
> 1 file changed, 1 insertion(+)
>
>diff --git a/BaseTools/Source/Python/Common/Misc.py
>b/BaseTools/Source/Python/Common/Misc.py
>index 565a5ad40b..c7daf5417c 100644
>--- a/BaseTools/Source/Python/Common/Misc.py
>+++ b/BaseTools/Source/Python/Common/Misc.py
>@@ -1030,10 +1030,11 @@ def ParseFieldValue (Value):
>             subprocess._cleanup()
>             p.stdout.close()
>             p.stderr.close()
>         if err:
>             raise BadExpression("DevicePath: %s" % str(err))
>+        out = out.decode()
>         Size = len(out.split())
>         out = ','.join(out.split())
>         return '{' + out + '}', Size
>
>     if "{CODE(" in Value:
>--
>2.18.0.windows.1
>
>_______________________________________________
>edk2-devel mailing list
>edk2-devel@lists.01.org
>https://lists.01.org/mailman/listinfo/edk2-devel