mu-utils: Fix build with musl

djcb / mu

maildir indexer/searcher + emacs mail client + guile bindings

http://www.djcbsoftware.nl/code/mu

GNU General Public License v3.0

1.59k stars 384 forks source link

mu-utils: Fix build with musl #2677

Closed MatthewGentoo closed 3 months ago

MatthewGentoo commented 3 months ago

In musl, stdout is a macro that expands to (stdout), and ::(stdout) is not valid C++.

../mu-1.12.2/lib/utils/mu-utils.hh:268:32: error: expected id-expression before '(' token
 268 |                              ::stdout);
     |                                ^~~~~~

Nothing in the Mu namespace is named stdout, so it is safe to drop the ::.

Bug: https://bugs.gentoo.org/928361

MatthewGentoo commented 3 months ago

Would you accept a change to add Alpine Linux, or Gentoo Linux w/ musl to the CI workflow?

djcb commented 3 months ago

Merged, thanks.

djcb commented 3 months ago

Would you accept a change to add Alpine Linux, or Gentoo Linux w/ musl to the CI workflow?

Sure, assuming after this PR that would work.