search

AllStarLink / app_rpt

Refactoring and upgrade of AllStarLink's app_rpt, etc.
4 stars 2 forks source link

sys.io warnings Raspberry Pi 5 #285

Closed tsawyer closed 4 months ago

tsawyer commented 4 months ago

Many of these...

app_rpt/app_rpt.h:30:2: warning: #warning sys.io is not available on this architecture and some functionality will be disabled [-Wcpp]
   30 | #warning sys.io is not available on this architecture and some functionality will be disabled
      |  ^~~~~~~
In file included from app_rpt/rpt_bridging.c:36:
InterLinked1 commented 4 months ago

I guess this should be some kind of gcc "notice", rather than a warning, this is really just informational.

It should compile fine without dev mode, since warnings are only errors in dev mode, but I'll get that changed. Do you think we need documentation somewhere else about the limitations of using these platforms, or will such users already understand?

tsawyer commented 4 months ago

Should the Rpi architecture be detected and compiled accordingly?

InterLinked1 commented 4 months ago

Should the Rpi architecture be detected and compiled accordingly?

Currently, it is. The question is more if users need to be made aware of the differing support in any other way.

tsawyer commented 4 months ago

It's well known that io functions are different on the Raspberry Pi

InterLinked1 commented 4 months ago

How were you compiling in this case, was it in dev mode?

I don't think these warnings will ever be treated as errors, because of -Wcpp., so there isn't actually an issue here. The warnings are purely informational, about the degraded functionality. No need to worry about them.

tsawyer commented 4 months ago

How were you compiling in this case, was it in dev mode?

It was a phreaknet install. If this is no concern issue may be closed.