CreaValix / ems_bus

A Buderus EMS bus driver for Python and Home Assistant
GNU General Public License v3.0
16 stars 6 forks source link

Fix compilation #7

Closed BubuOT closed 12 months ago

BubuOT commented 1 year ago

These changes are necessary for me to compile the project under gcc 11.4.0

CreaValix commented 12 months ago

Hi @BubuOT, thanks for your changes. client_id was supposed to be one symbol between rx.c and tx.c. I just corrected this in another push.

Why did you need -Wno-unused-result? According to the documentation , means to „Do not warn if a caller of a function marked with attribute warn_unused_result does not use its return value“.

The unused struct is fixed already.

CreaValix commented 12 months ago

My bad, I executed make manually. It seems that indeed setup.py sets warn_unused_result for functions somehow.

I fixed it by using the return value of write().

It should all compile on gcc 11 and higher now.