Re: "firefox" fails packaging on 9.3_STABLE/amd64

[David Brownlee <abs%absd.org@localhost>]

On Sat, 24 Jun 2023 at 03:41, John D. Baker <jdbaker%consolidated.net@localhost> wrote:
>
> On Fri, 23 Jun 2023, John D. Baker wrote:
>
> > > Looks like firefox-114.0.1/widget/gtk/moz.build has
> > >
> > > if CONFIG["MOZ_WAYLAND"]:
> > >     DIRS += ["wayland", "mozwayland", "vaapitest"]
> > >
> > > So you may need to add a conditional to the PLIST for that to build :)
> >
> > There's no option for "wayland" in firefox yet there is a
> > "PLIST.wayland" conditional used for "lib/firefox/libmozwayland.so".
> > I've copied it to before "lib/firefox/vaapitest" and will try again
> > once my build host finishes other tasks.
>
> This seems to have worked:
>
> +Index: www/firefox/PLIST
> +===================================================================
> +RCS file: /cvsroot/pkgsrc/www/firefox/PLIST,v
> +retrieving revision 1.182
> +diff -u -p -r1.182 PLIST
> +--- www/firefox/PLIST  14 Jun 2023 16:22:18 -0000      1.182
> ++++ www/firefox/PLIST  24 Jun 2023 02:34:24 -0000
> +@@ -33,7 +33,7 @@ lib/firefox/pingsender
> + lib/firefox/platform.ini
> + lib/firefox/plugin-container
> + lib/firefox/removed-files
> +-lib/firefox/vaapitest
> ++${PLIST.wayland}lib/firefox/vaapitest
> + share/applications/firefox.desktop
> + share/icons/hicolor/128x128/apps/firefox.png
> + share/icons/hicolor/16x16/apps/firefox.png

Committed, thanks. Had a quick check and none of the earlier firefox
versions mention vaapitest, so they should be safe :)

David