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

Add Analyze Compile Flags #3387

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from
Draft

Conversation

AiraYumi
Copy link
Contributor

@AiraYumi AiraYumi commented Jan 9, 2025

Fixed version of #3321

@marchcat
Copy link
Contributor

Just checked, and it throws the same errors on my mac builds as before. Let's keep this one open until we fix the builds.

@AiraYumi
Copy link
Contributor Author

I can build it with the GHA build, but I don't know why an error occurs on the actual device.
I don't own a Mac, so I can't check the error.

@marchcat
Copy link
Contributor

Right, because GHA builds use "Release" configuration.

@marchcat
Copy link
Contributor

@AiraYumi, are your local linux builds fine with these changes and "ReleaseWithDebInfo" configuration? I don't have a linux dev setup right now, so can't verify that.

@AiraYumi
Copy link
Contributor Author

oops, Relwithdebinfoes options did not exist in the Linux version and an error occurred. You need to fix it.

@AiraYumi AiraYumi marked this pull request as draft January 10, 2025 22:40
@AiraYumi
Copy link
Contributor Author

Changed to Draft.
You need to fix all errors related to DEB

@AiraYumi
Copy link
Contributor Author

I think this is necessary.
e84a180

@marchcat
Copy link
Contributor

marchcat commented Jan 10, 2025

Right, it will allow you to test this change as well.

@AiraYumi
Copy link
Contributor Author

AiraYumi commented Jan 11, 2025

I checked. There were a lot of build errors caused by adding the debug option.
This will probably take a fair amount of time to fix. maybe.

https://github.com/AiraYumi/viewer/actions/runs/12720005335/job/35461069394

@AiraYumi
Copy link
Contributor Author

Can you merge this into a branch other than develop?
We expect a large number of CVE-related errors to occur, so we cannot determine the scale of the fix in a short time...

@AiraYumi
Copy link
Contributor Author

I built it with GCC_DISABLE_FATAL_WARNINGS enabled.
analyzer.log

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

Successfully merging this pull request may close these issues.

2 participants