-
Notifications
You must be signed in to change notification settings - Fork 3.7k
GH-46562: [GLib][CI] Enable ARROW_GLIB_WERROR by default #46563
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
base: main
Are you sure you want to change the base?
GH-46562: [GLib][CI] Enable ARROW_GLIB_WERROR by default #46563
Conversation
|
Oh, It seems that we need to fix many warnings before this flag change. Change to draft. |
https://github.com/apache/arrow/actions/runs/15199666662/job/42751369386?pr=46563
|
7e324b7
to
8420764
Compare
We need to modify newline string on the Windows Platform. arrow/c_glib/arrow-glib/meson.build Lines 181 to 196 in 153da30
|
1eca642
to
305732e
Compare
305732e
to
43eb26d
Compare
Rationale for this change
In GLib buiding, disabled
-Werror
compiler flag by default.What changes are included in this PR?
Change
-Werror
by default. This change makes debugging easier.Are these changes tested?
Yes.
Are there any user-facing changes?
No.