Bug 271406 - lang/squeak: unbreak build on current
Summary: lang/squeak: unbreak build on current
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: Fernando Apesteguía
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-05-14 07:24 UTC by Trond Endrestøl
Modified: 2023-06-02 06:55 UTC (History)
3 users (show)

See Also:
bugzilla: maintainer-feedback? (takeshi.mutoh)


Attachments
Patch for lang/squeak disabling the warning on incompatible pointer to integer conversion (2.43 KB, patch)
2023-05-14 07:24 UTC, Trond Endrestøl
no flags Details | Diff
Update patch to build in my environment. (508 bytes, patch)
2023-05-15 01:30 UTC, takeshi.mutoh
takeshi.mutoh: maintainer-approval-
Details | Diff
Patch for lang/squeak disabling the warning on incompatible pointer to integer conversion (2.62 KB, patch)
2023-05-15 06:35 UTC, Trond Endrestøl
no flags Details | Diff
Patch for lang/squeak disabling the warning on incompatible pointer to integer conversion (2.68 KB, patch)
2023-05-18 16:29 UTC, Trond Endrestøl
Trond.Endrestol: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Trond Endrestøl 2023-05-14 07:24:00 UTC
Created attachment 242159 [details]
Patch for lang/squeak disabling the warning on incompatible pointer to integer conversion
Comment 1 Mina Galić freebsd_triage 2023-05-14 10:09:26 UTC
did you submit this issue upstream?
Comment 2 Trond Endrestøl 2023-05-14 15:50:37 UTC
(In reply to Mina Galić from comment #1)
No, I haven't. I saw a bunch of similar recent commits to the ports tree and figured one more wouldn't hurt.
Comment 3 takeshi.mutoh 2023-05-15 01:30:23 UTC
Created attachment 242177 [details]
Update patch to build in my  environment.

Updated the patch because of following error in my environment.

make: "/usr/home/mutoh/work/ports/lang/squeak/Makefile" line 67: Malformed conditional (${OPSYS} == FreeBSD && ${OSVERSION} >= 1400079)
make: Fatal errors encountered -- cannot continue
make: stopped in /usr/home/mutoh/work/ports/lang/squeak

Upstream has already finished developing this version of the Squeak VM, and I don't think they will incorporate this patch.
Comment 4 Trond Endrestøl 2023-05-15 06:35:55 UTC
Created attachment 242181 [details]
Patch for lang/squeak disabling the warning on incompatible pointer to integer conversion

It's better to include bsd.port.pre.mk and bsd.port.post.mk at the appropriate spots.
Comment 5 takeshi.mutoh 2023-05-16 05:57:06 UTC
(In reply to Trond.Endrestol from comment #4)

I try this patch work fine.
So please commit it.
Comment 6 Fernando Apesteguía freebsd_committer freebsd_triage 2023-05-18 14:20:22 UTC
(In reply to takeshi.mutoh from comment #5)
To avoid confusion, can you approve the patch you want to commit? (I think it is Trond's, but you approved your own instead).
Comment 7 Trond Endrestøl 2023-05-18 16:29:33 UTC
Created attachment 242255 [details]
Patch for lang/squeak disabling the warning on incompatible pointer to integer conversion

Inspired by commit 9b178ffd7e2fd77fecb8c80d22f7c80bc9e93977, take 13.2-STABLE >= 1302505 into account.
Comment 8 Fernando Apesteguía freebsd_committer freebsd_triage 2023-05-25 12:32:25 UTC
Ping.
Comment 9 Trond Endrestøl 2023-05-25 13:35:34 UTC
Comment on attachment 242177 [details]
Update patch to build in my  environment.

The other patch takes 13.2-STABLE into consideration.
Comment 10 Fernando Apesteguía freebsd_committer freebsd_triage 2023-05-31 09:44:11 UTC
Last ping.
Comment 11 takeshi.mutoh 2023-05-31 10:06:43 UTC
(In reply to Trond.Endrestol from comment #7)

Is this ping for me (maintainer) at comment #10?

I think there is no problem this patch.
So please commit.
Comment 12 Fernando Apesteguía freebsd_committer freebsd_triage 2023-05-31 12:05:13 UTC
(In reply to takeshi.mutoh from comment #11)
Again, there are *two patches* in this PR. One by you, one by Trond. Both are maintainer-approved.

Please: mark as the one that you *DON'T WANT* to be committed as obsolete :-)

From your last comment it is still not clear what patch you want to be committed.
Comment 13 takeshi.mutoh 2023-06-01 00:55:13 UTC
Comment on attachment 242177 [details]
Update patch to build in my  environment.

This patch don't care above 13.2-stable case.
So this patch becomes obsolete.
Comment 14 takeshi.mutoh 2023-06-01 01:02:05 UTC
(In reply to Fernando Apesteguía from comment #12)

I'm sorry.
I delete obsolete the patch (attachment 242181 [details]) and set maintainer-approved - at it.
Please commit attachment 242255 [details].
Comment 15 Fernando Apesteguía freebsd_committer freebsd_triage 2023-06-02 06:54:25 UTC
Committed,

Thanks!
Comment 16 commit-hook freebsd_committer freebsd_triage 2023-06-02 06:55:16 UTC
A commit in branch main references this bug:

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

commit 89ea113f0776c599e0f7c490bcfba9d7101c3dc8
Author:     Fernando Apesteguía <fernape@FreeBSD.org>
AuthorDate: 2023-06-01 06:35:26 +0000
Commit:     Fernando Apesteguía <fernape@FreeBSD.org>
CommitDate: 2023-06-02 06:49:10 +0000

    lang/squeak: unbreak build on current

    PR:             271406
    Reported by:    Trond.Endrestol@ximalas.info
    Approved by:    takeshi.mutoh@gmail.com (maintainer)

 lang/squeak/Makefile | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)