Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BaseTools: Incremental build issue for included ASI file's deletion. #1102

Merged
merged 1 commit into from Nov 10, 2020

Conversation

BobCF
Copy link
Member

@BobCF BobCF commented Nov 10, 2020

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2972

When using incremental build to delete an included xxx.asi file from
the ASL file, the xxx.asl.trim.deps file generated by previous build
process will not be deleted from the OUTPUT directory, which caused
the dependency file still include the xxx.asl.trim.deps file.

If the include file is deleted and DEPs is empty.

Signed-off-by: Mingyue Liang mingyuex.liang@intel.com
Cc: Bob Feng bob.c.feng@intel.com
Cc: Liming Gao gaoliming@byosoft.com.cn
Cc: Yuwei Chen yuwei.chen@intel.com

Reviewed-by: Bob Feng bob.c.feng@intel.com

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2972

When using incremental build to delete an included xxx.asi file from
the ASL file, the xxx.asl.trim.deps file generated by previous build
process will not be deleted from the OUTPUT directory, which caused
the dependency file still include the xxx.asl.trim.deps file.

If the include file is deleted and DEPs is empty.

Signed-off-by: Mingyue Liang <mingyuex.liang@intel.com>
Cc: Bob Feng <bob.c.feng@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Yuwei Chen <yuwei.chen@intel.com>

Reviewed-by: Bob Feng <bob.c.feng@intel.com>
@BobCF BobCF added the push Auto push patch series in PR if all checks pass label Nov 10, 2020
@mergify mergify bot merged commit 0af7f8e into tianocore:master Nov 10, 2020
@BobCF BobCF deleted the pr1110 branch April 25, 2022 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
push Auto push patch series in PR if all checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants