Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 27 Nov 2024 13:28:27 +0000
From:      bugzilla-noreply@freebsd.org
To:        ports-bugs@FreeBSD.org
Subject:   [Bug 282733] [NEW PORT] textproc/dmenu-translate: Quick text translation with dmenu
Message-ID:  <bug-282733-7788-jA9Cs9OJwR@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-282733-7788@https.bugs.freebsd.org/bugzilla/>
References:  <bug-282733-7788@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D282733

Yusuf Yaman <nxjoseph@protonmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #255475|0                           |1
        is obsolete|                            |

--- Comment #6 from Yusuf Yaman <nxjoseph@protonmail.com> ---
Created attachment 255487
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=3D255487&action=
=3Dedit
textproc/dmenu-translate.patch

You are welcome and thanks for explaining things to fix with references and=
 in
a detailed way.

> do not invent a version number when upstream doesn't have one, instead fo=
llow
> the example given in =C2=A7 5.4.3 "USE_GITHUB" Example 17.

Thanks, done.

> use the full git hash for GH_TAGNAME so it's unambiguous in any case

done.

> prefer DISTVERSION over PORTVERSION if possible

done.

> please check the grammar of your option descriptions; perhaps something l=
ike
> this would work?

i just used yours.

> our mktemp does not support --tempdir.  Please check the manual.  Perhaps=
 you
> want "mktemp -d dmenu-translate.XXXXXX'

Hmm, it seem to work for me, man page of mktemp says:

     -p tmpdir, --tmpdir[=3Dtmpdir]
             Use tmpdir for the -t flag if the TMPDIR environment variable =
is
             not set.  Additionally, any provided template arguments will be
             interpreted relative to the path specified as tmpdir.  If tmpd=
ir
             is either empty or omitted, then the TMPDIR environment variab=
le
             will be used.

> I'm sorry for not catching these issues earlier.

No problem.

--=20
You are receiving this mail because:
You are the assignee for the bug.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-282733-7788-jA9Cs9OJwR>