Re: [new] glow - Render markdown on the CLI

[Juan Picca]

Fixed the `WANTLIB` line.
Due an error I used `WANTLIB = ` instead of `WANTLIB += `.

Regards,
JMPC


glow.tgz
Description: application/tar-gz

Re: [new] glow - Render markdown on the CLI

[Juan Picca]

Hi!

On 24/02/13 11:41AM, Stuart Henderson wrote:
> On 2024/02/12 12:46, juan.picca wrote:
> > Hi Stuart.
> >
> > Thanks for your tweaks!
> >
> > > - missing WANTLIB
> >
> > Can you tell me how to determine the `WANTLIB` value for the next time?
>
> make port-lib-depends-check (sometimes it needs adjustments but is often
> ok to just directly copy-and-paste).

Thanks for the help.

I attach the port with the suggested changes.

Regards,
JMPC


glow.tgz
Description: application/tar-gz

Re: [new] glow - Render markdown on the CLI

[Stuart Henderson]

On 2024/02/12 12:46, juan.picca wrote:
> Hi Stuart.
> 
> Thanks for your tweaks!
> 
> > - missing WANTLIB
> 
> Can you tell me how to determine the `WANTLIB` value for the next time?

make port-lib-depends-check (sometimes it needs adjustments but is often
ok to just directly copy-and-paste).

Re: [new] glow - Render markdown on the CLI

[juan.picca]

Hi Stuart.

Thanks for your tweaks!

> - missing WANTLIB

Can you tell me how to determine the `WANTLIB` value for the next time?

Regards,
JMPC

Re: [new] glow - Render markdown on the CLI

[Stuart Henderson]

On 2024/02/12 01:44, juan.picca wrote:
> Hi.
> 
> First port.
> 
> A markdown render for the terminal.
> 
> https://github.com/charmbracelet/glow
> 
> Written in go, the port is trivial.
> The site already generate a release for openbsd, 

I don't see that in assets on their releases page (and the last release
was long enough ago that it probably won't work on -current anyway).

> but I think is better having it in the ports tree.
> 
> Regards,
> JMPC

a few tweaks,

- textproc seems a better category?
- use tabs for whitespace
- adjust COMMENT; partly because the main part of it seems to be the TUI
not the CLI tools, partly to make it clear it has cloud stuff in it
(even if encrypted I imagine many people will stay away from it because
of that)
- missing WANTLIB

I haven't tested runtime as I'm not particularly interested in some
Markdown cloud storage tool :)


diff --git a/textproc/glow/Makefile b/textproc/glow/Makefile
index a528e9b..bd6f720 100644
--- a/textproc/glow/Makefile
+++ b/textproc/glow/Makefile
@@ -1,19 +1,21 @@
-COMMENT = render markdown on the CLI
-CATEGORIES = misc
+COMMENT =  terminal based markdown reader with cloud storage
+CATEGORIES =   textproc
 
-MODGO_MODNAME = github.com/charmbracelet/glow
-MODGO_VERSION = v1.5.1
+MODGO_MODNAME =github.com/charmbracelet/glow
+MODGO_VERSION =v1.5.1
 
-DISTNAME = glow-${MODGO_VERSION}
+DISTNAME = glow-${MODGO_VERSION}
 
-HOMEPAGE = https://github.com/charmbracelet/glow
+HOMEPAGE = https://github.com/charmbracelet/glow
 
 # MIT
 PERMIT_PACKAGE = Yes
 
-MODULES = lang/go
+WANTLIB += c pthread
 
-NO_TEST = Yes
+MODULES =  lang/go
+
+NO_TEST =  Yes
 
 .include "modules.inc"