From nobody Mon Jan 30 09:56:26 2023 X-Original-To: ports-bugs@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4P53VL5r8yz3c62F for ; Mon, 30 Jan 2023 09:56:26 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4P53VL4nq1z3pw4 for ; Mon, 30 Jan 2023 09:56:26 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1675072586; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=rFTm7H62ExcCc2uBt3qiCdenk0dfmEktOypfphkIU/w=; b=tuJyo7mxUQ79K0tgs0/NDVb5/VvZGCxw5Nop6xb1SEn+xTVEybAKCf6pqxOaQixiDxtLeO FAXbprUHwLgx+gwlmO6fyZvF9Lx0krBMngo8tItNc225PHnNFrjnn22nwZK/RrD4yViMHX H4Yl65EzUzYUCbMm4OXMEuesQw1pFJe55MT92WpOO+6/XeZNdiksmRyfpdUzP2uQ197WPE 9IrdQa/cs/15quWgScHtAaxXWQXwQY5WyOpSrdovL/vyHcUGQVG6sBaZ9+rD9m/UJ5NYIR 6vD6KLdgEu9rR5PWaOOGCVEHeHF38XGoojzX7fVbciVcA/VudyILrk4cZld83w== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1675072586; a=rsa-sha256; cv=none; b=p4/kpJs5lbTJamDUvOMnYQ1hkGNqmVoR5tSRMv6fh+Ew6mihqDARvdAX1xjnuj9HnrVN6/ xOJRuHkF5l/Xnn9SJbVWOSPv8dy9/xDlh+ojMKaSSs1pAsYhltrlx3KoOCdrFXzN1bNMn5 XGhCqg2D0iQJd3/3AotAraRMqUpJo3gjC8CcM4NBcNzgEbEJUocfGcnbzWuTd+2ADXBCzb QJkfAiX5eMnIqzolWDAQUCm/xX9Dkr3Bc+xcoBn+XeccS20UGIbkKO0Z20A12lTnOPGwic GZT0b9ZykqaGsxlKy5y3Bqaj3McIvi2+ewfYcQJl/arHym5en5BjXv3gQ96NLA== Received: from kenobi.freebsd.org (kenobi.freebsd.org [IPv6:2610:1c1:1:606c::50:1d]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 4P53VL3nqcz13vp for ; Mon, 30 Jan 2023 09:56:26 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: from kenobi.freebsd.org ([127.0.1.5]) by kenobi.freebsd.org (8.15.2/8.15.2) with ESMTP id 30U9uQVg070123 for ; Mon, 30 Jan 2023 09:56:26 GMT (envelope-from bugzilla-noreply@freebsd.org) Received: (from www@localhost) by kenobi.freebsd.org (8.15.2/8.15.2/Submit) id 30U9uQsQ070122 for ports-bugs@FreeBSD.org; Mon, 30 Jan 2023 09:56:26 GMT (envelope-from bugzilla-noreply@freebsd.org) X-Authentication-Warning: kenobi.freebsd.org: www set sender to bugzilla-noreply@freebsd.org using -f From: bugzilla-noreply@freebsd.org To: ports-bugs@FreeBSD.org Subject: [Bug 269241] lang/spidermonkey102: builds fine with TRYBROKEN=1 on 12.4 (does not fail the readelf check) Date: Mon, 30 Jan 2023 09:56:26 +0000 X-Bugzilla-Reason: AssignedTo X-Bugzilla-Type: new X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: Ports & Packages X-Bugzilla-Component: Individual Port(s) X-Bugzilla-Version: Latest X-Bugzilla-Keywords: X-Bugzilla-Severity: Affects Some People X-Bugzilla-Who: jcfyecrayz@liamekaens.com X-Bugzilla-Status: New X-Bugzilla-Resolution: X-Bugzilla-Priority: --- X-Bugzilla-Assigned-To: nc@FreeBSD.org X-Bugzilla-Flags: maintainer-feedback? X-Bugzilla-Changed-Fields: bug_id short_desc product version rep_platform op_sys bug_status bug_severity priority component assigned_to reporter flagtypes.name Message-ID: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: https://bugs.freebsd.org/bugzilla/ Auto-Submitted: auto-generated List-Id: Ports bug reports List-Archive: https://lists.freebsd.org/archives/freebsd-ports-bugs List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-freebsd-ports-bugs@freebsd.org X-BeenThere: freebsd-ports-bugs@freebsd.org MIME-Version: 1.0 X-ThisMailContainsUnwantedMimeParts: N https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D269241 Bug ID: 269241 Summary: lang/spidermonkey102: builds fine with TRYBROKEN=3D1 on 12.4 (does not fail the readelf check) Product: Ports & Packages Version: Latest Hardware: Any OS: Any Status: New Severity: Affects Some People Priority: --- Component: Individual Port(s) Assignee: nc@FreeBSD.org Reporter: jcfyecrayz@liamekaens.com Assignee: nc@FreeBSD.org Flags: maintainer-feedback?(nc@FreeBSD.org) Building lang/spidermonkey102 works fine on FreeBSD 12.4 with TRYBROKEN=3D1= (or with poudriere, testport or bulk -T). It was marked broken with scant details (a brief comment about not finding readelf), so it's not clear what may have "fixed" it (nor how to reproduce = the reported problem), and no reference to a problem report with any further information. Digging further, it turns out that moz.configure has the following check: def validate_readelf(path): . . retcode, stdout, stderr =3D get_cmd_output(path, "--help") return retcode =3D=3D 0 and any(l.startswith(" -d ") for l in stdout.splitlines()) It attempts to check the output of 'readelf --help' to see if -d is support= ed.=20 But /usr/bin/readelf (from the BSD licensed elftoolchain project) sends the output of --help to stderr rather than stdout. This is why the check failed for /usr/bin/readelf. As of FreeBSD 12.4, however, there is also /usr/bin/llvm-readelf. The configure script checks for that flavor of readelf as well, and llvm-readelf DOES send --help output to stdout. In fact, the configure check tries llvm-readelf first before trying to fall back to readelf. So, on 12.4, the configure check for readelf does not fail as it did for 12= .3.=20 Here is the snippet showing how 'make configure' fails in 12.3: . . checking for readelf... not found DEBUG: readelf: Looking for llvm-readelf DEBUG: readelf: Looking for x86_64-portbld-freebsd12.3-readelf DEBUG: readelf: Looking for x86_64-freebsd12.3-readelf DEBUG: readelf: Looking for readelf DEBUG: Executing: `/usr/bin/readelf --help` DEBUG: readelf: /usr/bin/readelf found but didn't work . . Now on 12.4, the configure check is happy: . . checking for readelf... /usr/bin/llvm-readelf checking for objcopy... /usr/bin/llvm-objcopy . . [Incidentally, the GNU readelf also sends --help output to stdout as well. = So this problem never occurred on most or all Linux distributions.] One COULD fix the configure check to also look at stderr like so: --- work/firefox-102.4.0/moz.configure.orig 2022-10-10 09:55:56.0000000= 00 -0600 +++ work/firefox-102.4.0/moz.configure 2023-01-29 21:22:35.614382000 -0700 @@ -812,7 +812,8 @@ def validate_readelf(path): # option in the `--help` output, which fortunately, s compatible betwe= en # llvm-readelf and readelf. retcode, stdout, stderr =3D get_cmd_output(path, "--help") - return retcode =3D=3D 0 and any(l.startswith(" -d ") for l in stdout.splitlines()) + return retcode =3D=3D 0 and (any(l.startswith(" -d ") for l in stdout.splitlines()) \ + or any(l.startswith(" -d ") for l in stderr.splitlines())) @depends("--enable-compile-environment", target, host) But now that 12.4 has llvm-readelf, that patch is not needed unless we want= to support 12.3, which goes EOL in March, 2023. If so, adding the above patch does the job. Given that 12.3 is shortly going to be EOL and the project package builders= are using 12.4, I think we should just remove the BROKEN_FreeBSD_12 from lang/spidermonkey102/Makefile: diff --git a/lang/spidermonkey102/Makefile b/lang/spidermonkey102/Makefile index 219c5543fe83..3f32420c85c6 100644 --- a/lang/spidermonkey102/Makefile +++ b/lang/spidermonkey102/Makefile @@ -13,8 +13,6 @@ WWW=3D https://spidermonkey.dev/ LICENSE=3D MPL20 LICENSE_FILE=3D ${WRKSRC}/LICENSE -BROKEN_FreeBSD_12=3D ERROR: Cannot find readelf - BUILD_DEPENDS=3D ${LOCALBASE}/bin/python${PYTHON3_DEFAULT}:lang/python${PYTHON3_DEFAULT:S/./= /g} \ ${RUST_DEFAULT}>=3D1.35:lang/${RUST_DEFAULT} \ autoconf2.13:devel/autoconf2.13 \ --=20 You are receiving this mail because: You are the assignee for the bug.=