Skip to content

Commit 656d8ab

Browse files
authored
Merge pull request #209 from applandinc/fix/builtin-path
fix: Use require_name as the default package 'path' for builtins
2 parents 0dc884f + bcb4367 commit 656d8ab

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

lib/appmap/config.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,7 @@ class << self
163163
def package_hooks(methods, path: nil, gem: nil, force: false, builtin: false, handler_class: nil, require_name: nil)
164164
Array(methods).map do |method|
165165
package = if builtin
166-
Package.build_from_builtin(path, require_name: require_name, labels: method.labels, shallow: false)
166+
Package.build_from_builtin(path || require_name, require_name: require_name, labels: method.labels, shallow: false)
167167
elsif gem
168168
Package.build_from_gem(gem, require_name: require_name, labels: method.labels, shallow: false, force: force, optional: true)
169169
elsif path

0 commit comments

Comments
 (0)