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

[SR-7796] Assertion failed: : file /Users/buildnode/jenkins/workspace/oss-swift-4.2-package-osx/swiftpm/Sources/PackageGraph/PackageGraphRoot.swift, line 124 #4813

Closed
weissi opened this issue May 29, 2018 · 10 comments
Labels

Comments

@weissi
Copy link
Member

weissi commented May 29, 2018

Previous ID SR-7796
Radar rdar://problem/40617721
Original Reporter @weissi
Type Bug
Status Closed
Resolution Done
Additional Detail from JIRA
Votes 0
Component/s Package Manager
Labels Bug
Assignee None
Priority Medium

md5: a03b8ff2f96f13dc54e0cf819000c161

Issue Description:

I have a checked out directory that I tested with a few different Swift versions and now typing swift test in there crashes at least Swift 4.1 release and also Apple Swift version 4.2-dev (LLVM ff3ae623c3, Clang c2feda0b5c, Swift 58f7399ba5):

Assertion failed: : file /Users/buildnode/jenkins/workspace/oss-swift-4.2-package-osx/swiftpm/Sources/PackageGraph/PackageGraphRoot.swift, line 124
/Users/johannes/local/bin/jw-swift-latest: line 3:  3494 Illegal instruction: 4  jw-other-swift latest "$@"

I attached a tarball with everything.

@belkadan
Copy link

Attachment seems to have failed. :-(

@weissi
Copy link
Member Author

weissi commented May 29, 2018

thanks @belkadan, exceeds 10MB, uploading to dropbox, will attach link

@ankitspd
Copy link
Member

@swift-ci create

@ankitspd
Copy link
Member

@shahmishal should we increase the attachment limit? 10MB seems very small for the build data.

@weissi
Copy link
Member Author

weissi commented May 29, 2018

@ankitspd
Copy link
Member

Ah. I was overly strict and added an assertion to check that revision dependencies have git hash length of 40. I don't think that check is actually needed because we do validate the git ref. You're hitting this because OSS toolchains have assertions enabled. To workaround, use full git hash for your revision based dependencies.

@weissi
Copy link
Member Author

weissi commented May 30, 2018

@aciidb0mb3r hmm, am I even using any git hash dependencies?

@ankitspd
Copy link
Member

Yeah:

26: .package(url: "https://github.com/pointfreeco/swift-prelude.git", .revision("3b521bd")),
27: .package(url: "https://github.com/pointfreeco/swift-snapshot-testing.git", .revision("08c0976")),

@weissi
Copy link
Member Author

weissi commented May 30, 2018

@aciidb0mb3r oh thanks! Totally forgot I used this 3rd party package. sorry about that

@ankitspd
Copy link
Member

#1683

@swift-ci swift-ci transferred this issue from apple/swift-issues Apr 25, 2022
@shahmishal shahmishal transferred this issue from apple/swift May 4, 2022
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants