Re: web-eid-app-chrome: support iridium

On Tue, Jul 25, 2023 at 04:42:10PM +0000, Klemens Nanni wrote:
> After one of the recent www/iridum updates, the extension/native host now
> works there just like it does with www/chromium.
>
> Is this the right way to depend on either chromium or iridium?
> The way I understand this: use either iridium-* or chromium-* if installed,
> otherwise install www/chromium to satisfy the runtime dependency.

Not in practise, as I can't uninstall chromium when iridium is
installed:

$ pkg_info -S
Information for inst:web-eid-chrome-2.3.1p0

Signature:
web-eid-chrome-2.3.1p0,10,@chromium-115.0.5790.102,@web-eid-native-2.3.1p0

Mark, I checked packages-specs(7) and looked for other ports with
variant dependencies, but they use the same stemA-*|stemB-*:pkgpathA
pattern... can you tell me how this is supposed to work?

>
> diff --git a/security/web-eid-app/Makefile b/security/web-eid-app/Makefile
> index e0802355391..0f689e15ae4 100644
> --- a/security/web-eid-app/Makefile
> +++ b/security/web-eid-app/Makefile
> @@ -1,5 +1,5 @@
> COMMENT-main = native messaging host for Web eID browser extension
> -COMMENT-chrome = chromium extension
> +COMMENT-chrome = Chromium extension
> CATEGORIES = security www
>
> # open "release tarballs with bundled git submodules"
> @@ -11,6 +11,7 @@ DISTFILES = ${GH_DISTFILE}
> PKGNAME-main = ${PKGNAME:S/app/native/}
> REVISION-main = 0
> PKGNAME-chrome = ${PKGNAME:S/app/chrome/}
> +REVISION-chrome = 0
>
> # TODO: firefox
> MULTI_PACKAGES = -main -chrome
> @@ -69,7 +70,7 @@ RUN_DEPENDS-main = devel/desktop-file-utils \
>
> # TODO support www/iridium and www/ungoogled-chromium as well
> RUN_DEPENDS-chrome = ${FULLPKGPATH-main} \
> - www/chromium
> + chromium-*|iridium-*:www/chromium
> LIB_DEPENDS-chrome = # empty
>
> # XXX pcsclite is detected with pkg-config, but its -I and -L are ignored
> diff --git a/security/web-eid-app/pkg/DESCR-chrome b/security/web-eid-app/pkg/DESCR-chrome
> index 5e214c12bc5..3233d177e58 100644
> --- a/security/web-eid-app/pkg/DESCR-chrome
> +++ b/security/web-eid-app/pkg/DESCR-chrome
> @@ -1 +1 @@
> -Configuration for the official Web eID extension for Chromium.
> +Configuration for the official Web eID extension for Chromium and Iridium.
> diff --git a/security/web-eid-app/pkg/PLIST-chrome b/security/web-eid-app/pkg/PLIST-chrome
> index b0465b00f2d..1f277f9fb0c 100644
> --- a/security/web-eid-app/pkg/PLIST-chrome
> +++ b/security/web-eid-app/pkg/PLIST-chrome
> @@ -1,10 +1,14 @@
> -share/doc/pkg-readmes/web-eid-chrome
> +share/doc/pkg-readmes/${PKGSTEM}
> share/google-chrome/
> share/google-chrome/extensions/
> @sample ${SYSCONFDIR}/chromium/extensions/
> +@sample ${SYSCONFDIR}/iridium/extensions/
> share/google-chrome/extensions/ncibgoaomkmdpilpocfeponihegamlic.json
> @sample ${SYSCONFDIR}/chromium/extensions/ncibgoaomkmdpilpocfeponihegamlic.json
> +@sample ${SYSCONFDIR}/iridium/extensions/ncibgoaomkmdpilpocfeponihegamlic.json
> share/web-eid/
> @sample ${SYSCONFDIR}/chromium/native-messaging-hosts/
> +@sample ${SYSCONFDIR}/iridium/native-messaging-hosts/
> share/web-eid/eu.webeid.json
> @sample ${SYSCONFDIR}/chromium/native-messaging-hosts/eu.webeid.json
> +@sample ${SYSCONFDIR}/iridium/native-messaging-hosts/eu.webeid.json
> diff --git a/security/web-eid-app/pkg/README-chrome b/security/web-eid-app/pkg/README-chrome
> index 16010e12dd3..2abe2f39f1b 100644
> --- a/security/web-eid-app/pkg/README-chrome
> +++ b/security/web-eid-app/pkg/README-chrome
> @@ -4,7 +4,8 @@
>
> Install the official Web eID extension
> https://chrome.google.com/webstore/detail/ncibgoaomkmdpilpocfeponihegamlic
> -and add the following unveil rules to ${SYSCONFDIR}/chromium/unveil.main:
> +and add the following unveil rules to either of
> + ${SYSCONFDIR}/{chromium,iridium}/unveil.main:
>
> # native messaging host
> /usr/local/bin r
>