x11/polybar: Many fixes.

[rigoletto]

Summary

PR: 226507

• remove ALSA from OPTIONS_DEFAULT (broken)
• add mpd to working modules list (tested)
• add menu to working modules (tested)
• add xworkspaces to working modules (not extensively tested)
• add github to working modules (tested)
• add TESTS OPTION
• fix IPC_DESC. That is a built-in IPC, not i3 related
• move cpu, memory and others modules to broken (tested)
• while here do some alphabetizing

[rigoletto]

Fixing pkg-plist again.

[rigoletto]

Fix empty line in a file.

[rigoletto]

This comment was removed by rigoletto.

[rigoletto]

More fixes.

[rigoletto]

Revert the distinfo file. There is not point to commit just timestamp changes.

[tobik]

x11/polybar/Makefile

51 ↗

(On Diff #40528)

This seems like a bogus change? Please keep the options helpers sorted.

x11/polybar/pkg-plist

2 ↗

(On Diff #40528)

polybar-msg is only built when BUILD_IPC_MSG is turned on, so this needs to either be added back in or we remove the IPC option entirely. It doesn't add any new dependencies AFAICT and we can just always build polybar-msg.

Build with IPC=off:

===> Checking for items in STAGEDIR missing from pkg-plist
===> Checking for items in pkg-plist which are not in STAGEDIR
Error: Missing: bin/polybar-msg
===> Error: Plist issues found.

[rigoletto]

Fixing and:

• add TESTS (test suite) OPTION
• I *temporarily* enabled some xcb-* stuff to see if they change anything related with some known broken modules.

Thanks!

[rigoletto]

Cleaning a GH_TAG what should not be in there.

[rigoletto]

• more modules tested and working: menu, and xworkspaces.

• for backlight and xbacklight I get this message:

error: Disabling module "backlight" (reason: Not supported for "DVI-I-0")

I do not know if those were supposed to be working or the message is just right about not being supported.

• the GitHub module show up but I had no opportunity to actually know if it is working or not.

• the battery module I do not have how to test, and I will try to proper test the ipc one due tomorrow.

• The commit 0edaead fix the very annoying bug of tray overlapping other modules. Do you think it worth to bring the port to that commit? See HERE.

Thanks!

[rigoletto]

Btw, upstream just merged all patches. \o/

x11/polybar/pkg-plist

2 ↗

(On Diff #40528)

Weird. Now it is being installed in here with %%IPC%% in there. I guess I've missed something while testing it.

Thanks!

[rigoletto]

Unless I understood wrong how that works (HERE and HERE), the IPC module unfortunately seems to be broken.

Adding "enable-ipc = true" to config and creating a simple custom module:

[module/demo]
type = custom/ipc
hook-0 = date
click-left = polybar-msg -p %pid% hook demo 1

and then starting polybar. The FIFO file is created in /tmp and an empty area appear on the specified part of the bar.

The simplest test would be running something like "polybar-msg cmd hide" to hide the bar, but that return "polybar-msg: No process with pid ${pid}".

The second one was a tentative to trigger the hook-0: "echo 'hook:demo/ipc1' >>/tmp/polybar_mqueue.${pid}" and ... notthing happened.

I tried several ways to make it work but the only one that appear to triggered the hook was setting "initial = 1" to make the hook-0 be triggered at the polybar start.

So I guess something is broken between the FIFO and polybar. For course, mouse clicking also does not work.

[rigoletto]

Last fixes.

Thanks.

[rigoletto]

More broken modules: cpu, ram.

Disabling IPC as it seem to be partially broken. Does not work when using the 'polybar-msg' command.

[rigoletto]

Re-enable IPC and update pkg-message to notice the polybar-msg facility does not
seem to work.

[rigoletto]

Removing GH_TAGNAME, what was not supposed to be in here.

Thanks.

[tobik]

x11/polybar/Makefile

59 ↗

(On Diff #42025)

How are the tests run? Does it happen automatically during the build or are you missing TESTS_TEST_TARGET?

[rigoletto]

x11/polybar/Makefile

59 ↗

(On Diff #42025)

They are automatic, the instructions are set on the CMakeLists.txt file. Here is another CMakeLists.txt with the option to enable it.