Re: git: 560772623792 - main - net/freerdp3: adjust missed reference to www/webkit2-gtk
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Tue, 21 Jan 2025 14:23:30 UTC
Hi.
> Vladimir Druzenko wrote:
>> 21.01.2025 16:43, Charlie Li пишет:
>>> -WEBKIT_LIB_DEPENDS_= libsoup-2.4.so:devel/libsoup \
>>> +WEBKIT_LIB_DEPENDS= libsoup-2.4.so:devel/libsoup \
>>
>> This is not a typo. Please revert this line back.
>>
> Is there a reason for the trailing underscore? It causes tooling and
> regexes that look for *_DEPENDS to miss lines like this, so ports like
> this get missed when dependencies need their consumers bumped.
Found logics using it in line 216 through line 229 of
net/freerdp3/Makefile as below.
Possibly I'm overlooking something, but something like
"WEBENGINE_USE_GNOME= " are not implemented, while "*_USES=" and
"*_*DEPENDS" are implemented. Vladimir, is this the reason you've used
variable with trailing underscore?
===== Quote =====
.if ${PORT_OPTIONS:MAAD}
. if ${PORT_OPTIONS:MWEBENGINE}
USES+= ${WEBENGINE_USES_}
USE_QT+= ${WEBENGINE_USE_QT}
. elif ${PORT_OPTIONS:MWEBKIT}
LIB_DEPENDS+= ${WEBKIT_LIB_DEPENDS_}
USES+= ${WEBKIT_USES_}
USE_GNOME+= ${WEBKIT_USE_GNOME}
. else
CMAKE_OFF+= WITH_WEBVIEW
. endif
.else
CMAKE_OFF+= WITH_WEBVIEW
.endif
===== End quote =====
Regards.
--
Charlie Li
...nope, still don't have an exit line.
--
Tomoaki AOKI <junchoon@dec.sakura.ne.jp>