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.

Sign up for GitHub

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

Jump to bottom

Chatterino has a compilation time issue on Windows CI #4623

Closed
4 tasks done
Mm2PL opened this issue May 11, 2023 · 4 comments
Closed
4 tasks done

Chatterino has a compilation time issue on Windows CI #4623

Mm2PL opened this issue May 11, 2023 · 4 comments
Labels
bug Something isn't working as intended, or works in a confusing/unintuitive way for the user ci Issues related to the ci

Comments

@Mm2PL
Copy link
Collaborator

Mm2PL commented May 11, 2023

Checklist

  • I'm reporting a problem with Chatterino
  • I've verified that I'm running the most recent nightly build or stable release
  • I've looked for my problem on the wiki
  • I've searched the issues and pull requests for similar looking reports

Describe your issue

Chatterino takes 15 to 30 minutes !! depending on the compiler on CI. This is not acceptable.

Screenshots

From #4622 :
image

OS and Chatterino Version

1f0aa80
@Mm2PL Mm2PL added the bug Something isn't working as intended, or works in a confusing/unintuitive way for the user label May 11, 2023
@Felanbird Felanbird changed the title Chatterino has a compilation time issue Chatterino has a compilation time issue on Windows CI May 11, 2023
@Felanbird Felanbird added the ci Issues related to the ci label May 11, 2023
@Felanbird
Copy link
Collaborator

The ubuntu/mac ones are ok, the windows one is the truly unreasonable one

Fix was attempted in #4606 by axing LTO but it did not resolve the super slow windows build.

@fourtf
Copy link
Member

fourtf commented May 11, 2023 •
edited
Loading

Clangs --ftime-trace is useful for figuring out ways to reduce compile time

Otherwise upgrading the build infra to more cores

@pajlada
Copy link
Member

pajlada commented May 11, 2023

Duplicate of #3452 (your own issue)

@pajlada pajlada closed this as not planned Won't fix, can't repro, duplicate, stale May 11, 2023
@Mm2PL
Copy link
Collaborator Author

Mm2PL commented May 11, 2023 •
edited
Loading

same one complaint xd

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working as intended, or works in a confusing/unintuitive way for the user ci Issues related to the ci
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@pajlada @fourtf @Mm2PL @Felanbird