-
Notifications
You must be signed in to change notification settings - Fork 8
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
Fix c warnings #14
Comments
A lot of the "blame" for this falls on upstream shoulders. If I understand this correctly fixing these would make it harder to sync upstream changes. Is that correct @simoncozens? |
Right. Let’s do the sync first (things may be fixed), and then see if it’s already fixed. If not, we can raise an issue. (texk is on GitHub these days; it used to be just SVN and a mailing list.) |
Upstream is on GitHub? Do you know where, specifically? |
Upstream project name is AFAIK the mailing list is still the primary means of communication (though there may be some Japanese communication channels I don't know about). |
I'm a TeX Live contributor and I have a write access to TeX Live repo. If you can send me a patch to fix warnings, it's welcome. |
Currently compiling throws a bunch of c warnings. Some of them may be suppressible, others look like they are legitimate concerns. I'm not enough of a C master to know the differences, but one way or another the build should be setup so it doesn't throw any warnings by default. It's really hard to debug downstream packaging this way.
Note some distro build systems by default enable all warnings as errors and you have to setup overrides to get anything with warnings to even go ahead with the build.
The text was updated successfully, but these errors were encountered: