Bug 272102 - www/deno: panics a lot in dprint-swc-ext (needs update)
Summary: www/deno: panics a lot in dprint-swc-ext (needs update)
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Mikael Urankar
URL: https://github.com/denoland/deno/rele...
Keywords: crash
Depends on:
Blocks:
 
Reported: 2023-06-20 18:57 UTC by Val Packett
Modified: 2023-06-21 11:37 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (mikael)


Attachments
0001-www-deno-update-to-1.34.3.patch (45.56 KB, patch)
2023-06-21 07:29 UTC, Val Packett
val: maintainer-approval?
Details | Diff
retrieve rss with sysctl (1.46 KB, text/plain)
2023-06-21 08:07 UTC, Mikael Urankar
no flags Details
disable upgrade (480 bytes, text/plain)
2023-06-21 08:09 UTC, Mikael Urankar
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Val Packett 2023-06-20 18:57:43 UTC
The currently packaged 1.34.1 is a really unfortunate version because very often  it crashes with this:

============================================================
Deno has panicked. This is a bug in Deno. Please report this
at https://github.com/denoland/deno/issues/new.
If you can reliably reproduce this panic, include the
reproduction steps and re-run with the RUST_BACKTRACE=1 env
var set and include the backtrace in your report.

Platform: freebsd x86_64
Version: 1.34.1
Args: ["deno", "run", "--allow-all", "build.ts"]

thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', /wrkdirs/usr/ports/www/deno/work/deno-1.34.1/cargo-crates/dprint-swc-ext-0.9.0/src/view
/generated.rs:12430:22
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace


In the current v1.34.3 that dependency has been updated to dprint-swc-ext-0.10.0 which should fix that..
Comment 2 Val Packett 2023-06-21 07:29:18 UTC
Created attachment 242913 [details]
0001-www-deno-update-to-1.34.3.patch

Here's a patch!
Comment 3 Mikael Urankar freebsd_committer freebsd_triage 2023-06-21 08:07:41 UTC
Created attachment 242914 [details]
retrieve rss with sysctl

(In reply to Val Packett from comment #2)
Thanks, I was updating the port on my side.

Do you have time to review the attached file? it's based on OpenBSD's code.
Comment 4 Mikael Urankar freebsd_committer freebsd_triage 2023-06-21 08:09:28 UTC
Created attachment 242915 [details]
disable upgrade

and this one too.
Comment 5 Val Packett 2023-06-21 08:11:53 UTC
both LGTM
Comment 6 commit-hook freebsd_committer freebsd_triage 2023-06-21 11:36:38 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=44fc0f1cf985d7982ff549f26cb5a8b98fc1120f

commit 44fc0f1cf985d7982ff549f26cb5a8b98fc1120f
Author:     Val Packett <val@packett.cool>
AuthorDate: 2023-06-21 07:12:38 +0000
Commit:     Mikael Urankar <mikael@FreeBSD.org>
CommitDate: 2023-06-21 11:35:21 +0000

    www/deno: update to 1.34.3

    Important update due to dprint-swc-ext dependency bump fixing a panic.
    While here, avoid bringing in the rayon dependency via sysinfo.

    Changelog:      https://github.com/denoland/deno/compare/v1.34.1...v1.34.3
    PR:             272102
    Sponsored by:   https://www.patreon.com/valpackett

 www/deno/Makefile                                  | 109 ++++++-----
 www/deno/distinfo                                  | 214 +++++++++++----------
 www/deno/files/patch-Cargo.lock                    |  98 +---------
 ...rgo-crates_libffi-sys-2.2.1_build_not__msvc.rs} |   4 +-
 www/deno/files/patch-ext_fs_30__fs.js (gone)       |  20 --
 www/deno/files/patch-ext_fs_std__fs.rs (gone)      |  11 --
 www/deno/files/patch-runtime_Cargo.toml            |   2 +-
 www/deno/files/patch-tools_wpt_utils.ts (gone)     |  10 -
 8 files changed, 177 insertions(+), 291 deletions(-)