Skip to content

Commit

Permalink
Merge pull request #2166 from actonlang/pkg-upgrade-disable-gc
Browse files Browse the repository at this point in the history
Disable GC for acton pkg upgrade
  • Loading branch information
plajjan authored Feb 13, 2025
2 parents 02b8721 + cd10017 commit 003cb97
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions cli/src/acton.act
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
import acton.rts
import argparse
import file
import json
Expand Down Expand Up @@ -1033,6 +1034,14 @@ actor CmdPkgUpgrade(env, args):

zig = file.join_path([base_path(file.FileCap(env.cap)), "zig", "zig"])

# GAAH, the TLS lib does DNS lookups which interferes with GC as the malloc
# / free is asymmetric, we malloc (using GC malloc) while the free happens
# deeper inside and then incorrectly tries to free a GC malloced chunk
# We should replace with our own DNS lookups, but until then, disable GC for
# pkg upgrade. Should not be a practical problem to run without GC.
# TODO: remove this!!!!
acton.rts.disable_gc(env.syscap)

# What arguments do we need?
# - the name of the dependency (e.g. "foo", used in the build.act.json config file)
# - the URL to fetch it from
Expand Down

0 comments on commit 003cb97

Please sign in to comment.