Skip to content

Commit 1662ca0

Browse files
committed
update-deps: skip bundled-gem object files in read_cc_deps
Mirrors the existing `.bundle/` filter in read_make_deps (line 330). After ordering `ext/configure-ext.mk` after prepare-gems, bundled-gem C extensions get compiled during the main build, so `Pathname.glob('**/*.o')` picks up things like `.bundle/gems/bigdecimal-4.1.2/ext/bigdecimal/bigdecimal.o`. Those targets then reach `in_makefile`, which has no case for `.bundle/` paths and raises "unexpected target". Bundled gems own their depend files, so they should be skipped here. https://claude.ai/code/session_01YRoRyZPew2LtN6u6BX8bzx
1 parent aed1130 commit 1662ca0

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

tool/update-deps

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -401,6 +401,7 @@ end
401401
def read_cc_deps(cwd)
402402
deps = {}
403403
Pathname.glob('**/*.o').sort.each {|fn_o|
404+
next if /\.bundle\// =~ fn_o.to_s # skip bundled gem extensions; they manage their own depend files
404405
fn_i = fn_o.sub_ext('.i')
405406
if !fn_i.exist?
406407
next if fn_o.sub_ext('.S').exist?

0 commit comments

Comments
 (0)