Bug 265405 - graphics/bonzomatic: fix build on armv6 armv7 arm64
Summary: graphics/bonzomatic: fix build on armv6 armv7 arm64
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: Dmitry Marakasov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-07-23 17:10 UTC by Robert Clausecker
Modified: 2022-08-30 11:37 UTC (History)
1 user (show)

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


Attachments
graphics/bonzomatic: fix build on armv6 armv7 arm64 (1.27 KB, patch)
2022-07-23 17:10 UTC, Robert Clausecker
fuz: maintainer-approval? (amdmi3)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Robert Clausecker freebsd_committer freebsd_triage 2022-07-23 17:10:53 UTC
Created attachment 235440 [details]
graphics/bonzomatic: fix build on armv6 armv7 arm64

Add a missing cast the compiler was complaining about.

Tested with Poudriere on armv7 arm64 FreeBSD 13.
Please MFH if possible.
Comment 1 Dmitry Marakasov freebsd_committer freebsd_triage 2022-07-26 12:50:19 UTC
Have you submitted this upstream?
Comment 2 Robert Clausecker freebsd_committer freebsd_triage 2022-07-26 14:00:42 UTC
(In reply to Dmitry Marakasov from comment #1)

No.
Comment 3 Dmitry Marakasov freebsd_committer freebsd_triage 2022-07-26 15:28:01 UTC
Please do. This fix does not seem to be FreeBSD specific and should be upstreamed.
Comment 4 Robert Clausecker freebsd_committer freebsd_triage 2022-07-26 15:55:21 UTC
(In reply to Dmitry Marakasov from comment #3)

Sure, I'll do that after you have committed the patch I sent you.  I do not like build fixes being hung on an upstream that may or may not respond.
Comment 5 Dmitry Marakasov freebsd_committer freebsd_triage 2022-07-26 16:44:25 UTC
Sorry, I'm not going to accept this until upstream PR is created.
Comment 6 Robert Clausecker freebsd_committer freebsd_triage 2022-07-26 19:21:17 UTC
Okay.
Comment 7 Dmitry Marakasov freebsd_committer freebsd_triage 2022-07-26 21:12:49 UTC
This is pending upstream PR submission.
Comment 8 Robert Clausecker freebsd_committer freebsd_triage 2022-07-26 21:18:41 UTC
(In reply to Dmitry Marakasov from comment #7)

Which I am not going to do before the patch is committed.  Hence “not accepted.”
Comment 9 Dmitry Marakasov freebsd_committer freebsd_triage 2022-07-26 21:48:37 UTC
I can do it myself if you are not interested in doing it right, and doing it right implies making upstream aware of the problem. It's pretty simple, and I don't see any reason for making a scene out of it.
Comment 10 Robert Clausecker freebsd_committer freebsd_triage 2022-07-26 21:59:17 UTC
(In reply to Dmitry Marakasov from comment #9)

My current objective is to work through the ports collection and fix build issues on armv7 as they crop up.  I've had bad experiences with other ports were a build fix was delayed indefinitely and I had lots of busywork to do because the maintainer wanted upstream to fix this instead of committing a patch locally.

I am once again happy to file a bug report with upstream, but I do not accept that as a condition to have the patch applied.  After the patch is applied, I can of course file a bug report so upstream can fix the issue and take all the time in the world to do it while the FreeBSD ports is in working order.  However, I do not want to be part of that.

I do not like to be sent on errands with the vacuous promise of having the patch applied maybe some day when the stars are right and upstream decides to respond.  Have played this game too often and it just stops me from doing productive work.
Comment 11 Robert Clausecker freebsd_committer freebsd_triage 2022-08-12 03:04:52 UTC
Hi there,

Any progress with this one?
Comment 12 commit-hook freebsd_committer freebsd_triage 2022-08-12 12:29:46 UTC
A commit in branch main references this bug:

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

commit a811d270e8259c51e78878a2f04f962555f9cf90
Author:     Dmitry Marakasov <amdmi3@FreeBSD.org>
AuthorDate: 2022-08-12 12:23:12 +0000
Commit:     Dmitry Marakasov <amdmi3@FreeBSD.org>
CommitDate: 2022-08-12 12:23:50 +0000

    graphics/bonzomatic: fix build on arm*

    PR:             265405
    Submitted by:   fuz@fuz.su

 graphics/bonzomatic/files/patch-src_main.cpp (new) | 11 +++++++++++
 1 file changed, 11 insertions(+)
Comment 13 Robert Clausecker freebsd_committer freebsd_triage 2022-08-12 15:18:57 UTC
Issue reported upstream.

https://github.com/Gargaj/Bonzomatic/issues/172
Comment 14 Dmitry Marakasov freebsd_committer freebsd_triage 2022-08-12 16:59:48 UTC
It's quite sudden to ask this. Unfortunately you have one-sided position, and as a result you're making _me_ do _your_ errands for the sake of your task, and it as well stops me from doing productive work. Next time please be sure to submit upstream PR first before submitting a patch to my ports. Note that there's absolutely no requirement to get the fix committed upstream and I'm absolutely fine with just issue or PR submitted. The main goal is to make problem globally visible instead of pulluting the tree with patches

Upstream PR: https://github.com/Gargaj/Bonzomatic/pull/171
Comment 15 Robert Clausecker freebsd_committer freebsd_triage 2022-08-12 18:36:33 UTC
Hi,

You can be sure that if requested, I'll submit a PR as soon as the patch is committed to ports.

As a compromise, I will try to no longer submit patches for any of your ports.  This way, your preferred process will no longer be interfered with.
Comment 16 Robert Clausecker freebsd_committer freebsd_triage 2022-08-23 17:45:03 UTC
Seems like upstream committed the patch (yay!).  Can this be closed now?