Bug 272831 - x11/polybar: Add patches for CPU and memory module
Summary: x11/polybar: Add patches for CPU and memory module
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Alexandre C. Guimarães
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-07-30 22:21 UTC by Joel Bodenmann
Modified: 2023-08-06 03:27 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (rigoletto)


Attachments
patch (6.07 KB, patch)
2023-07-30 22:21 UTC, Joel Bodenmann
jbo: maintainer-approval?
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Joel Bodenmann freebsd_committer freebsd_triage 2023-07-30 22:21:14 UTC
Created attachment 243720 [details]
patch

I've added patches to make the CPU & memory module work under FreeBSD.

Patches have been handed to upstream but it's questionable when and whether they will be accepted given that upstream is pretty Linux centric.
Comment 1 Alexandre C. Guimarães freebsd_committer freebsd_triage 2023-08-04 13:25:16 UTC
Hi.

I will try to commit it during the weekend.

Also, I'm with severe time issues and not using polybar anymore (what make harder to test).

Would you like to take the maintainership?

Thank you.
Comment 2 Joel Bodenmann freebsd_committer freebsd_triage 2023-08-04 15:23:17 UTC
Hi,

Yeah I can take over maintainership - no problem!
Would you like to handle this PR before transferring maintainership or shall we transfer immediately?

Thank you for your maintenance efforts so far! :)
Comment 3 Alexandre C. Guimarães freebsd_committer freebsd_triage 2023-08-04 17:15:43 UTC
Could you please re-create the patch against the lastest HEAD, it is failing to apply in here?

I change the maintainer when committing your patches.

Thank you.
Comment 4 Joel Bodenmann freebsd_committer freebsd_triage 2023-08-05 18:49:26 UTC
Can you provide some information regarding "it is failing to apply in here?".
I can apply it successfully to the current HEAD. Your comment made me doubt tho so I re-created the patch again (good exercise) and it's still applying just like before.

I am seeing trailing-whitespace warnings but I have no idea where they originate from. The upstream patches contained within the current version of the port patch have been created using `git diff --no-index`. The second time I created the port I used `make makepatch` and they both result in the same whitespace warnings when being applied.

Nevertheless, it applies correctly as far as I can tell.
Comment 5 Alexandre C. Guimarães freebsd_committer freebsd_triage 2023-08-06 01:42:16 UTC
(In reply to Joel Bodenmann from comment #4)

Those were at lines 28, 36, 41, 80, 104, 109, and 132.

I fixed them in here now. ;-)
Comment 6 commit-hook freebsd_committer freebsd_triage 2023-08-06 03:06:43 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=6cdddcb8fd5df5db70abb86c4abe56664b60c7f9

commit 6cdddcb8fd5df5db70abb86c4abe56664b60c7f9
Author:     Joel Bodenmann <jbo@insane.engineer>
AuthorDate: 2023-08-06 03:04:55 +0000
Commit:     Alexandre C. Guimarães <rigoletto@FreeBSD.org>
CommitDate: 2023-08-06 03:04:55 +0000

    x11/polybar: Add patches for CPU and memory modules

    - give it to Joel Bodenmann <jbo@insane.engineer>

    PR:             272831

 x11/polybar/Makefile                               |  4 +-
 x11/polybar/files/patch-src_modules_cpu.cpp (new)  | 66 ++++++++++++++++++++++
 .../files/patch-src_modules_memory.cpp (new)       | 49 ++++++++++++++++
 x11/polybar/pkg-message                            |  4 +-
 4 files changed, 119 insertions(+), 4 deletions(-)
Comment 7 Alexandre C. Guimarães freebsd_committer freebsd_triage 2023-08-06 03:27:01 UTC
Thank you.