Skip to content
This repository was archived by the owner on Feb 13, 2023. It is now read-only.

Commit 9a8b183

Browse files
committed
Revert previous commit and fix the loading of Drupal VM ruby lib
1 parent 0da90e2 commit 9a8b183

File tree

2 files changed

+8
-23
lines changed

2 files changed

+8
-23
lines changed

Vagrantfile

+8-3
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,21 @@
11
# -*- mode: ruby -*-
22
# vi: set ft=ruby :
33

4-
require_relative 'lib/drupalvm/vagrant'
5-
64
drupalvm_env = ENV['DRUPALVM_ENV'] || 'vagrant'
75

86
# Default paths when the project is based on Drupal VM.
97
host_project_dir = host_drupalvm_dir = host_config_dir = __dir__
108
guest_project_dir = guest_drupalvm_dir = guest_config_dir = '/vagrant'
119

1210
if File.exist?("#{host_project_dir}/composer.json")
13-
cconfig = load_composer_config("#{host_project_dir}/composer.json")
11+
cconfig = {}
12+
composer_conf = JSON.parse(File.read("#{host_project_dir}/composer.json"))
13+
if composer_conf['extra'] && composer_conf['extra']['drupalvm']
14+
cconfig = composer_conf['extra']['drupalvm']
15+
end
1416

1517
# If Drupal VM is a Composer dependency set the correct path.
18+
vendor_dir = composer_conf.fetch('extra', {}).fetch('vendor-dir', 'vendor')
1619
drupalvm_path = "#{vendor_dir}/geerlingguy/drupal-vm"
1720
if Dir.exist?("#{host_project_dir}/#{drupalvm_path}")
1821
host_drupalvm_dir = "#{host_project_dir}/#{drupalvm_path}"
@@ -26,6 +29,8 @@ if File.exist?("#{host_project_dir}/composer.json")
2629
end
2730
end
2831

32+
require "#{host_drupalvm_dir}/lib/drupalvm/vagrant"
33+
2934
default_config_file = "#{host_drupalvm_dir}/default.config.yml"
3035
unless File.exist?(default_config_file)
3136
raise_message "Configuration file not found! Expected in #{default_config_file}"

lib/drupalvm/vagrant.rb

-20
Original file line numberDiff line numberDiff line change
@@ -34,16 +34,6 @@ def resolve_jinja_variables(vconfig)
3434
end
3535
end
3636

37-
# Return the drupalvm extra-field from the passed composer.json
38-
def load_composer_config(path)
39-
cconfig = {}
40-
composer_conf = JSON.parse(File.read(path))
41-
if composer_conf['extra'] && composer_conf['extra']['drupalvm']
42-
cconfig = composer_conf['extra']['drupalvm']
43-
end
44-
cconfig
45-
end
46-
4737
# Return the combined configuration content all files provided.
4838
def load_config(files)
4939
vconfig = {}
@@ -66,16 +56,6 @@ def ansible_version
6656
/^[^\s]+ (.+)$/.match(`#{ansible_bin} --version`) { |match| return match[1] }
6757
end
6858

69-
# Return the path to the composer executable
70-
def composer_bin
71-
@composer_bin ||= which('composer')
72-
end
73-
74-
# Return Composer's vendor directory.
75-
def vendor_dir
76-
@vendor_dir ||= composer_bin ? `#{composer_bin} config vendor-dir`.strip : 'vendor'
77-
end
78-
7959
# Require that if installed, the ansible version meets the requirements.
8060
def require_ansible_version(requirement)
8161
return unless ansible_bin

0 commit comments

Comments
 (0)