search

jjzhang / booth

The Booth Cluster Ticket Manager
GNU General Public License v2.0
16 stars 37 forks source link

Build error with -Werror=cast-align on ia64 #32

Open fghaas opened 12 years ago

fghaas commented 12 years ago

Debian still builds on ia64, among other platforms. On ia64, we're getting a cast-align warning that breaks the build as we're building with -Werror. The problem is evidently in the do_list and do_command functions in src/main.c.

Details here: https://buildd.debian.org/status/fetch.php?pkg=booth&arch=ia64&ver=0.1.0-1&stamp=1338167781

jjzhang commented 12 years ago

I'm also wondering if this warning indicates a real problem or not. If not, how about removing the -Werror ?;)

fghaas commented 12 years ago

You're the developer, you tell me. Silencing a warning just by disabling it sounds like less than a great idea to me.

fghaas commented 12 years ago

Building with -Wcast-align doesn't only break on ia64, but on a bunch of platforms:

https://buildd.debian.org/status/package.php?p=booth

I suggest to either fix those warnings, or (if safe to do so) disable the cast-align warnings.

jjzhang commented 12 years ago

Disable cast-align warning should be safe, so fix it now.