Opened 6 months ago

Closed 7 days ago

#68615 closed defect (fixed)

MenuMeters @1.1.6.1: error: a function declaration without a prototype is deprecated in all versions of C

Reported by: pipp8 (Pippo Cattaneo) Owned by: Tatsh (Andrew Udvare)
Priority: Normal Milestone:
Component: ports Version: 2.8.1
Keywords: sonoma Cc: landonf (Landon Fuller)
Port: MenuMeters

Description (last modified by ryandesign (Ryan Carsten Schmidt))

After upgrading Mac Port menumeters disappered from the menu bar and the new installation fails with the following message (no detail in the log file):

bash-3.2# port install MenuMeters
--->  Fetching archive for MenuMeters
--->  Attempting to fetch MenuMeters-2.1.6.1_0.darwin_23.x86_64.tbz2 from https://packages.macports.org/MenuMeters
--->  Attempting to fetch MenuMeters-2.1.6.1_0.darwin_23.x86_64.tbz2 from http://fco.it.packages.macports.org/MenuMeters
--->  Attempting to fetch MenuMeters-2.1.6.1_0.darwin_23.x86_64.tbz2 from https://fra.de.packages.macports.org/MenuMeters
--->  Staging MenuMeters into destroot
Error: Failed to destroot MenuMeters: command execution failed

Log file has a huge list of command but no error.

Did you plan a port to sonoma in next releases ?

thank you

Attachments (1)

main.log.zip (42.9 KB) - added by pipp8 (Pippo Cattaneo) 6 months ago.
Installation main.log file

Download all attachments as: .zip

Change History (7)

comment:1 Changed 6 months ago by ryandesign (Ryan Carsten Schmidt)

Component: contribports
Description: modified (diff)
Milestone: MacPorts Future
Owner: set to Tatsh
Status: newassigned
Type: updatedefect

Please attach the main.log file. If it is large, compress it first.

Changed 6 months ago by pipp8 (Pippo Cattaneo)

Attachment: main.log.zip added

Installation main.log file

comment:2 Changed 6 months ago by ryandesign (Ryan Carsten Schmidt)

Port: MenuMeters added; mennumeters removed
Summary: Installing MenuMeters on Mac Book with Sonoma (x86_64)MenuMeters @1.1.6.1: error: a function declaration without a prototype is deprecated in all versions of C

Ok, your log matches what we see on our Sonoma buildbot worker:

:info:destroot /opt/local/var/macports/build/_opt_local_var_macports_sources_distfiles.macports.org_ports_aqua_MenuMeters/MenuMeters/work/MenuMeters-2.1.6.1/hardware_reader/smc_reader.m:6:22: error: a function declaration without a prototype is deprecated in all versions of C [-Werror,-Wstrict-prototypes]
:info:destroot kern_return_t SMCOpen()
:info:destroot                      ^
:info:destroot                       void
:info:destroot /opt/local/var/macports/build/_opt_local_var_macports_sources_distfiles.macports.org_ports_aqua_MenuMeters/MenuMeters/work/MenuMeters-2.1.6.1/hardware_reader/smc_reader.m:38:23: error: a function declaration without a prototype is deprecated in all versions of C [-Werror,-Wstrict-prototypes]
:info:destroot kern_return_t SMCClose()
:info:destroot                       ^
:info:destroot                        void
:info:destroot /opt/local/var/macports/build/_opt_local_var_macports_sources_distfiles.macports.org_ports_aqua_MenuMeters/MenuMeters/work/MenuMeters-2.1.6.1/hardware_reader/applesilicon_hardware_reader.m:57:28: error: a function declaration without a prototype is deprecated in all versions of C [-Werror,-Wstrict-prototypes]
:info:destroot static void initEventSystem(){
:info:destroot                            ^
:info:destroot                             void

The fix for this issue was submitted to the developers of MenuMeters here but has not yet been accepted:

https://github.com/yujitach/MenuMeters/pull/305/commits/5c44f1f9bf97d01a75a73d7aaddea049564ad68c

We could apply that patch to MacPorts.

comment:3 Changed 6 months ago by Tatsh (Andrew Udvare)

I am wondering why the developer wants to use -Werror in a release build. Could turn that off to avoid further issues caused by -Werror in the future.

comment:4 Changed 6 months ago by ryandesign (Ryan Carsten Schmidt)

Yes that would be a good idea too.

comment:5 Changed 8 days ago by llucps

Just a quick update and menumeters compiles and works fine under Sonoma.

comment:6 Changed 7 days ago by ryandesign (Ryan Carsten Schmidt)

Cc: landonf added
Resolution: fixed
Status: assignedclosed

Landon fixed it in [652acb048c7600087fe177948beb1f142da4c9ad/macports-ports]. Thanks, Landon!

Note: See TracTickets for help on using tickets.