lint: stop ignoring LIEF imports #27507
pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:unignore_lief changing 6 files +6 −4-
fanquake commented at 9:32 am on April 21, 2023: memberType stubs are now available as of 0.13.0. See https://github.com/lief-project/LIEF/issues/650.
-
DrahtBot commented at 9:32 am on April 21, 2023: contributor
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
Reviews
See the guideline for information on the review process.
Type Reviewers ACK TheCharlatan If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
Conflicts
No conflicts as of last run.
-
fanquake commented at 9:40 am on April 21, 2023: member
ERROR: Could not find a version that satisfies the requirement lief==0.13.0 (from versions: 0.8.0.post7, 0.8.1.post1, 0.8.2.post1, 0.8.3.post3, 0.9.0, 0.10.0, 0.10.1, 0.11.0, 0.11.1, 0.11.2, 0.11.3, 0.11.4, 0.11.5, 0.12.0, 0.12.1, 0.12.2, 0.12.3) ERROR: No matching distribution found for lief==0.13.0
Looks like LIEF 0.13.0 requires 3.8+, which is why this is failing to install. Guess this will be solved after #27483.
-
DrahtBot added the label CI failed on Apr 21, 2023
-
fanquake force-pushed on Apr 21, 2023
-
fanquake marked this as a draft on Apr 21, 2023
-
fanquake referenced this in commit d89aca1bdb on Apr 28, 2023
-
maflcko added the label Needs rebase on Apr 28, 2023
-
maflcko commented at 12:14 pm on April 28, 2023: member
0contrib/devtools/security-check.py:246: error: "object" has no attribute "header" [attr-defined]
-
DrahtBot removed the label Needs rebase on Apr 28, 2023
-
fanquake force-pushed on May 1, 2023
-
fanquake commented at 1:42 pm on May 1, 2023: member
contrib/devtools/security-check.py:246: error: “object” has no attribute “header” [attr-defined]
The script is working as expected. This seems to either be a bug in mypy, or an issue with the LIEF stubs. Am following up.
-
fanquake commented at 8:28 am on May 3, 2023: member
or an issue with the LIEF stubs. Am following up.
This has now been resolved upstream in LIEF. See https://github.com/lief-project/LIEF/issues/909 & https://github.com/lief-project/LIEF/commit/2e06bdb4af4586d1f5dcb9e9ccb027b55d457e24.
-
hebasto commented at 7:47 pm on May 28, 2023: member
or an issue with the LIEF stubs. Am following up.
This has now been resolved upstream in LIEF. See lief-project/LIEF#909 & lief-project/LIEF@2e06bdb.
LIEF 0.13.1 has just been released with the fix.
-
lint: stop ignoring LIEF imports
Type stubs are now available as of 0.13.0. See https://github.com/lief-project/LIEF/issues/650.
-
fanquake force-pushed on May 29, 2023
-
fanquake marked this as ready for review on May 29, 2023
-
fanquake commented at 9:27 am on May 29, 2023: memberRebased and updated to use LIEF 0.13.1.
-
DrahtBot removed the label CI failed on May 29, 2023
-
fanquake requested review from TheCharlatan on May 29, 2023
-
TheCharlatan approved
-
TheCharlatan commented at 4:05 pm on May 29, 2023: contributorACK 015cc5e588fa25f65f6ea2ed03701def8dfd5444
-
fanquake merged this on May 29, 2023
-
fanquake closed this on May 29, 2023
-
fanquake deleted the branch on May 29, 2023
-
hebasto commented at 5:22 pm on May 29, 2023: memberPost-merge ACK 015cc5e588fa25f65f6ea2ed03701def8dfd5444.
-
sidhujag referenced this in commit bc532b9581 on May 29, 2023
-
fanquake referenced this in commit ec822e88dd on Jun 28, 2023
-
sidhujag referenced this in commit 61be3dddfb on Jun 30, 2023
-
knst referenced this in commit 63bea741ce on Oct 19, 2023
-
knst referenced this in commit 09b8836dc0 on Oct 19, 2023
-
PastaPastaPasta referenced this in commit 6e7b402fe9 on Oct 23, 2023
-
bitcoin locked this on May 28, 2024
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-24 00:12 UTC
More mirrored repositories can be found on mirror.b10c.me