Created attachment 254157 [details]
www/tgpt.patch

Changelog:
https://github.com/aandrew-me/tgpt/releases/tag/v2.8.2

Thanks.

Comment 3 Vladimir Druzenko 2024-10-11 16:18:20 UTC

make makesum generate 2 similar records in distinfo:
SHA256 (charmbracelet-x-v0.1.0_GH0.tar.gz) = f876110994066651d62c6e5ea1f94534a6e58f456bdc22849f3cecfa8836d0ed
SIZE (charmbracelet-x-v0.1.0_GH0.tar.gz) = 10317
SHA256 (charmbracelet-x-v0.1.0_GH0.tar.gz) = f876110994066651d62c6e5ea1f94534a6e58f456bdc22849f3cecfa8836d0ed
SIZE (charmbracelet-x-v0.1.0_GH0.tar.gz) = 10317

Perhaps the reason is this:
charmbracelet:x:v0.1.0:charmbracelet_x_input/vendor/github.com/charmbracelet/x/input \
charmbracelet:x:v0.1.0:charmbracelet_x_windows/vendor/github.com/charmbracelet/x/windows \

(In reply to Vladimir Druzenko from comment #3)
Yes, thanks for reporting what's likely causing it.

I ran portlint and it said me there are duplicate entries in distinfo so i manually removed one of them in distinfo and this was my workaround.

How should we handle this?

I already updated distinfo in the patch, do you need to run make makesum again?

Thanks.

Comment 5 Vladimir Druzenko 2024-10-11 17:07:30 UTC

(In reply to Yusuf Yaman from comment #4)
> I already updated distinfo in the patch, do you need to run make makesum again?
I must test this before commit too.

Tested this change:
charmbracelet:x:input/v0.1.0:charmbracelet_x_input/vendor/github.com/charmbracelet/x/charmbracelet:x:windows/v0.1.0:charmbracelet_x_windows/vendor/github.com/charmbracelet/x/windows \
Build fine.

Do we need "charmbracelet windows"?
Looks like this windows only module:
https://github.com/charmbracelet/x/blob/main/windows/types_windows.go
Build fine without it.

(In reply to Vladimir Druzenko from comment #5)
I couldn't imagine what changes you've made. It seems to be a module for Windows API. I think we can remove it if it built fine without it.

Comment 7 Vladimir Druzenko 2024-10-11 17:21:14 UTC

(In reply to Yusuf Yaman from comment #6)
Can you test runtime after remove "charmbracelet:x:v0.1.0:charmbracelet_x_windows/vendor/github.com/charmbracelet/x/windows" and rebuild?

Comment 8 Vladimir Druzenko 2024-10-11 17:24:43 UTC

Created attachment 254160 [details]
v1 (broken whitespaces)

Diff for review.

Created attachment 254172 [details]
www/tgpt.patch

Thanks for sending diff.

I tested runtime and it works fine.

I'm attaching the recent patch.

Comment 10 commit-hook 2024-10-12 20:42:32 UTC

A commit in branch main references this bug:

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

commit a1b339df9c5c6a27906c94fb671b2890eae50d70
Author:     Yusuf Yaman <nxjoseph@protonmail.com>
AuthorDate: 2024-10-12 20:38:45 +0000
Commit:     Vladimir Druzenko <vvd@FreeBSD.org>
CommitDate: 2024-10-12 20:38:45 +0000

    www/tgpt: Update 2.8.1 → 2.8.2

    Changelog:
    https://github.com/aandrew-me/tgpt/releases/tag/v2.8.2

    Remove windows-only module github.com/charmbracelet/x/windows.

    PR:     282011

 www/tgpt/Makefile |  6 ++----
 www/tgpt/distinfo | 10 +++++-----
 2 files changed, 7 insertions(+), 9 deletions(-)

Comment 11 Vladimir Druzenko 2024-10-12 20:45:56 UTC

Thanks.

(In reply to Vladimir Druzenko from comment #11)
You are welcome and thanks for giving a look.