From: "gaoliming" <gaoliming@byosoft.com.cn>
To: <devel@edk2.groups.io>, <lersek@redhat.com>, <bob.c.feng@intel.com>
Cc: "'Yuwei Chen'" <yuwei.chen@intel.com>,
"'Shenglei Zhang'" <shenglei.zhang@intel.com>
Subject: 回复: 回复: [edk2-devel] [Patch V2] BaseTools/Ecc: Fix an issue of path separator compatibility
Date: Wed, 2 Sep 2020 08:39:09 +0800 [thread overview]
Message-ID: <000e01d680c1$78c8fe50$6a5afaf0$@byosoft.com.cn> (raw)
In-Reply-To: <46e6486c-a9f2-49f6-a7fe-93ec67a664d8@redhat.com>
Laszlo:
Thank you!
Liming
> -----邮件原件-----
> 发件人: bounce+27952+64917+4905953+8761045@groups.io
> <bounce+27952+64917+4905953+8761045@groups.io> 代表 Laszlo Ersek
> 发送时间: 2020年9月2日 2:04
> 收件人: devel@edk2.groups.io; gaoliming@byosoft.com.cn;
> bob.c.feng@intel.com
> 抄送: 'Yuwei Chen' <yuwei.chen@intel.com>; 'Shenglei Zhang'
> <shenglei.zhang@intel.com>
> 主题: Re: 回复: [edk2-devel] [Patch V2] BaseTools/Ecc: Fix an issue of path
> separator compatibility
>
> On 09/01/20 16:42, gaoliming wrote:
> > Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>
>
> Merged in commit 751355992635, via
> <https://github.com/tianocore/edk2/pull/910>.
>
> Thanks
> Laszlo
>
> >> -----邮件原件-----
> >> 发件人: bounce+27952+64887+4905953+8761045@groups.io
> >> <bounce+27952+64887+4905953+8761045@groups.io> 代表 Bob Feng
> >> 发送时间: 2020年9月1日 18:23
> >> 收件人: devel@edk2.groups.io
> >> 抄送: Liming Gao <gaoliming@byosoft.com.cn>; Yuwei Chen
> >> <yuwei.chen@intel.com>; Shenglei Zhang <shenglei.zhang@intel.com>
> >> 主题: [edk2-devel] [Patch V2] BaseTools/Ecc: Fix an issue of path
> > separator
> >> compatibility
> >>
> >> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2904
> >>
> >> The path separator is different in Windows and Linux, the
> >> original code does not handle this difference. This patch
> >> is to fix this issue.
> >>
> >> Signed-off-by: Bob Feng <bob.c.feng@intel.com>
> >> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> >> Cc: Yuwei Chen <yuwei.chen@intel.com>
> >> Cc: Shenglei Zhang <shenglei.zhang@intel.com>
> >> ---
> >> V2
> >> Change to a better method to get path separator.
> >>
> >> BaseTools/Source/Python/Ecc/Check.py | 8 ++++----
> >> 1 file changed, 4 insertions(+), 4 deletions(-)
> >>
> >> diff --git a/BaseTools/Source/Python/Ecc/Check.py
> >> b/BaseTools/Source/Python/Ecc/Check.py
> >> index 0fdc7e35c1..6087abfa4d 100644
> >> --- a/BaseTools/Source/Python/Ecc/Check.py
> >> +++ b/BaseTools/Source/Python/Ecc/Check.py
> >> @@ -1101,15 +1101,15 @@ class Check(object):
> >> for Item in InfPathSet:
> >> if Item[0] not in InfPathList:
> >> InfPathList.append(Item[0])
> >> SqlCommand = """
> >> select ID, Path, FullPath from File where
> >> upper(FullPath) not in
> >> - (select upper(A.Path) || '\\' ||
> >> upper(B.Value1) from File as A, INF as B
> >> + (select upper(A.Path) || '%s' ||
> >> upper(B.Value1) from File as A, INF as B
> >> where A.ID in (select BelongsToFile
> from
> >> INF where Model = %s group by BelongsToFile) and
> >> B.BelongsToFile = A.ID and B.Model
> = %s)
> >> and (Model = %s or Model = %s)
> >> - """ % (MODEL_EFI_SOURCE_FILE,
> >> MODEL_EFI_SOURCE_FILE, MODEL_FILE_C, MODEL_FILE_H)
> >> + """ % (os.sep, MODEL_EFI_SOURCE_FILE,
> >> MODEL_EFI_SOURCE_FILE, MODEL_FILE_C, MODEL_FILE_H)
> >> RecordSet = EccGlobalData.gDb.TblInf.Exec(SqlCommand)
> >> for Record in RecordSet:
> >> Path = Record[1]
> >> Path = Path.upper().replace('\X64', '').replace('\IA32',
> >> '').replace('\EBC', '').replace('\IPF', '').replace('\ARM', '')
> >> if Path in InfPathList:
> >> @@ -1130,13 +1130,13 @@ class Check(object):
> >> if Pcd[3]:
> >> PcdName = Pcd[3]
> >> BelongsToFile = Pcd[4]
> >> SqlCommand = """
> >> select ID from File where FullPath in
> >> - (select B.Path || '\\' || A.Value1 from
> INF
> >> as A, File as B where A.Model = %s and A.BelongsToFile = %s
> >> + (select B.Path || '%s' || A.Value1 from
> >> INF as A, File as B where A.Model = %s and A.BelongsToFile = %s
> >> and B.ID = %s and (B.Model = %s or
> >> B.Model = %s))
> >> - """ % (MODEL_EFI_SOURCE_FILE,
> >> BelongsToFile, BelongsToFile, MODEL_FILE_C, MODEL_FILE_H)
> >> + """ % (os.sep,
> >> MODEL_EFI_SOURCE_FILE, BelongsToFile, BelongsToFile, MODEL_FILE_C,
> >> MODEL_FILE_H)
> >> TableSet =
> >> EccGlobalData.gDb.TblFile.Exec(SqlCommand)
> >> for Tbl in TableSet:
> >> TblName = 'Identifier' + str(Tbl[0])
> >> SqlCommand = """
> >> select Name, ID from %s where
> >> value like '%s' and Model = %s
> >> --
> >> 2.20.1.windows.1
> >>
> >>
> >>
> >
> >
> >
> >
> >
> >
>
>
>
next prev parent reply other threads:[~2020-09-02 0:39 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-09-01 10:23 [Patch V2] BaseTools/Ecc: Fix an issue of path separator compatibility Bob Feng
2020-09-01 14:42 ` 回复: [edk2-devel] " gaoliming
2020-09-01 18:03 ` Laszlo Ersek
2020-09-02 0:39 ` gaoliming [this message]
2020-09-02 0:44 ` Bob Feng
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='000e01d680c1$78c8fe50$6a5afaf0$@byosoft.com.cn' \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox