Skip to content

Fix CommandLineSpecTests #88

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

Merged
merged 1 commit into from
Feb 6, 2025
Merged

Conversation

neonichu
Copy link
Collaborator

@neonichu neonichu commented Feb 4, 2025

I broke these in #31 since that added a new setting that needs to be manually applied in this particular test.

@neonichu
Copy link
Collaborator Author

neonichu commented Feb 4, 2025

@swift-ci please test

@neonichu
Copy link
Collaborator Author

neonichu commented Feb 5, 2025

The output is quite difficult to parse, but I think appHostedTestTargetRebuild() had a couple of known issues, but also one unexpected one?

@owenv
Copy link
Collaborator

owenv commented Feb 5, 2025

I think the actual failure is that buildCommandWithXcodeWorkspace() crashed

@neonichu
Copy link
Collaborator Author

neonichu commented Feb 5, 2025

Ah yes, I made the mistake of looking at the rendered output without noticing that it is actually just showing the tail of the log...

@neonichu
Copy link
Collaborator Author

neonichu commented Feb 5, 2025

@swift-ci please test

I broke these in #31 since that added a new setting that needs to be manually applied in this particular test.
@neonichu neonichu force-pushed the bbuegling/fix-swiftTaskConstruction branch from 4c101cc to ac91508 Compare February 5, 2025 17:23
@neonichu neonichu changed the title Fix CommandLineSpecTests.swiftTaskConstruction() Fix CommandLineSpecTests Feb 5, 2025
@neonichu
Copy link
Collaborator Author

neonichu commented Feb 5, 2025

@swift-ci please test

@owenv
Copy link
Collaborator

owenv commented Feb 5, 2025

@swift-ci test macOS

@neonichu
Copy link
Collaborator Author

neonichu commented Feb 5, 2025

Failure is blockUndeclaredInputOrOutputFromDisk which is known to be flaky, see 98

@neonichu neonichu enabled auto-merge (squash) February 5, 2025 23:36
@neonichu
Copy link
Collaborator Author

neonichu commented Feb 5, 2025

I guess I need to rebase since the macOS job is now required.

@owenv
Copy link
Collaborator

owenv commented Feb 6, 2025

@swift-ci test macOS

2 similar comments
@owenv
Copy link
Collaborator

owenv commented Feb 6, 2025

@swift-ci test macOS

@owenv
Copy link
Collaborator

owenv commented Feb 6, 2025

@swift-ci test macOS

@owenv
Copy link
Collaborator

owenv commented Feb 6, 2025

test issues should be fixed now

@owenv
Copy link
Collaborator

owenv commented Feb 6, 2025

@swift-ci test macOS

@neonichu neonichu merged commit 45e66f0 into main Feb 6, 2025
1 of 2 checks passed
@neonichu neonichu deleted the bbuegling/fix-swiftTaskConstruction branch February 6, 2025 22:11
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

Successfully merging this pull request may close these issues.

4 participants