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

expected array length 2 but got 4 #91

Open
blacktop opened this issue Jul 11, 2024 · 8 comments
Open

expected array length 2 but got 4 #91

blacktop opened this issue Jul 11, 2024 · 8 comments

Comments

@blacktop
Copy link

Using Symbolicator.pkl

With xnu.pkl

I'm getting the error: expected array length 2 but got 4

?

Looking at your docs I can't see what I'm doing wrong?

@blacktop
Copy link
Author

Adding amends "../pkl/Symbolicator.pkl" to the top of xnu.pkl seemed to fix it, but I have no idea why?

Should I be using import or something else? What would others use if they had their own signatures ?

@HT154
Copy link
Contributor

HT154 commented Jul 11, 2024

Heads up that your linked files don't appear to be visible publicly, so it's hard to try and diagnose what's happening here.

@blacktop
Copy link
Author

@HT154 ?? I just opened this issue in a private tab and can get to both files just fine?

@HT154
Copy link
Contributor

HT154 commented Jul 12, 2024

Huh. The Github app errors on the links, but browser is fine. Sorry about that 😓

@blacktop
Copy link
Author

closing due to this mostly just being me not understanding how pkls work, but that error wasn't very helpful ;)

@bioball
Copy link
Contributor

bioball commented Jul 12, 2024

Re-opening this; that error message indicates that there is at least some error condition that we should be catching but aren't.

@bioball bioball reopened this Jul 12, 2024
@bioball
Copy link
Contributor

bioball commented Jul 12, 2024

@blacktop: is there a commit that you can point us to that reproduces this? On that branch, removing the amends header gives me another error.

But, yeah--the standard flow is to add the amends header like you are doing.

@blacktop
Copy link
Author

@bioball this should be the commit right before I added amend blacktop/ipsw@302e96e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants