pkg/52668: firefox 56.x: extra full dependencies of clang and rust

[Izumi Tsutsui <tsutsui%ceres.dti.ne.jp@localhost>]

>Number:         52668
>Category:       pkg
>Synopsis:       firefox 56.x: extra full dependencies of clang and rust
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    pkg-manager
>State:          open
>Class:          change-request
>Submitter-Id:   net
>Arrival-Date:   Sun Oct 29 07:20:00 +0000 2017
>Originator:     Izumi Tsutsui
>Release:        NetBSD 7.1
>Organization:
>Environment:
System: NetBSD 7.1 GENERIC
Architecture: i386
Machine: i386
>Description:
pkgsrc/www/firefox 56 now requires clang and rust to build and
dependencies are pulled via buildlink3.mk in mozilla-common.mk:
> .include "../../lang/clang/buildlink3.mk"
> .include "../../lang/rust/buildlink3.mk"

It specifies "full dependency" for them so pkg_add(1) will also
install all binaries clang, rust, and their dependencies
(libssh2, libgit2, and llvm etc.), but looks rust and clang binaries
are not necessary at runtime for the firefox.

>How-To-Repeat:
cd pkgsrc/www/firefox
make package-install
pkg_add firefox
pkg_info firefox

>Fix:
Adding the following "BUILDLINK_DEPMETHOD" lines before buildlink3.mk
inclusion would work.
(I'm not sure if it should be default in buildlink3.mk as gcc49 etc.)

firefox 56.0.2 binaries seems working installed rust and clang packages
on NetBSD/i386 7.1 and NetBSD/amd64 7.1.

Index: mozilla-common.mk
===================================================================
RCS file: /cvsroot/pkgsrc/www/firefox/mozilla-common.mk,v
retrieving revision 1.99
diff -u -p -r1.99 mozilla-common.mk
--- mozilla-common.mk	3 Oct 2017 13:17:37 -0000	1.99
+++ mozilla-common.mk	29 Oct 2017 02:35:57 -0000
@@ -189,8 +189,10 @@ BUILDLINK_API_DEPENDS.nss+=	nss>=3.32.1
 .include "../../graphics/MesaLib/buildlink3.mk"
 BUILDLINK_API_DEPENDS.cairo+=	cairo>=1.10.2nb4
 .include "../../graphics/cairo/buildlink3.mk"
+BUILDLINK_DEPMETHOD.clang=	build
 .include "../../lang/clang/buildlink3.mk"
 BUILDLINK_API_DEPENDS.rust+=	rust>=1.20.0
+BUILDLINK_DEPMETHOD.rust=	build
 .include "../../lang/rust/buildlink3.mk"
 BUILDLINK_API_DEPENDS.libvpx+=	libvpx>=1.3.0
 .include "../../multimedia/libvpx/buildlink3.mk"