aboutsummaryrefslogtreecommitdiffhomepage
path: root/modules
diff options
context:
space:
mode:
authorBjørn Erik Pedersen <[email protected]>2023-08-23 12:39:24 +0200
committerBjørn Erik Pedersen <[email protected]>2023-08-23 18:05:18 +0200
commitdcf425c846dcd6fbb0c05e87342077ab870eb7e1 (patch)
tree603f96feb9bb7a7503d86a4788dc5a6918ba4045 /modules
parent9a8c84d6000fab3ec6847a330c45bfe24b76265d (diff)
downloadhugo-dcf425c846dcd6fbb0c05e87342077ab870eb7e1.tar.gz
hugo-dcf425c846dcd6fbb0c05e87342077ab870eb7e1.zip
Fix it so disable a module does not disable transitive dependency required by others
The motivation behind the original implementation was probably to show disabled modules when running `hugo mod graph`. Fixes #11376
Diffstat (limited to 'modules')
-rw-r--r--modules/client.go6
-rw-r--r--modules/collect.go37
-rw-r--r--modules/module.go8
3 files changed, 15 insertions, 36 deletions
diff --git a/modules/client.go b/modules/client.go
index 7108c9bef..b41ca142a 100644
--- a/modules/client.go
+++ b/modules/client.go
@@ -153,10 +153,6 @@ func (c *Client) Graph(w io.Writer) error {
continue
}
- prefix := ""
- if module.Disabled() {
- prefix = "DISABLED "
- }
dep := pathVersion(module.Owner()) + " " + pathVersion(module)
if replace := module.Replace(); replace != nil {
if replace.Version() != "" {
@@ -166,7 +162,7 @@ func (c *Client) Graph(w io.Writer) error {
dep += " => " + replace.Dir()
}
}
- fmt.Fprintln(w, prefix+dep)
+ fmt.Fprintln(w, dep)
}
return nil
diff --git a/modules/collect.go b/modules/collect.go
index e47563ab7..5b5418bcd 100644
--- a/modules/collect.go
+++ b/modules/collect.go
@@ -109,11 +109,8 @@ func (h *Client) collect(tidy bool) (ModulesConfig, *collector) {
}
type ModulesConfig struct {
- // All modules, including any disabled.
- AllModules Modules
-
// All active modules.
- ActiveModules Modules
+ AllModules Modules
// Set if this is a Go modules enabled project.
GoModulesFilename string
@@ -123,7 +120,7 @@ type ModulesConfig struct {
}
func (m ModulesConfig) HasConfigFile() bool {
- for _, mod := range m.ActiveModules {
+ for _, mod := range m.AllModules {
if len(mod.ConfigFilenames()) > 0 {
return true
}
@@ -133,18 +130,12 @@ func (m ModulesConfig) HasConfigFile() bool {
}
func (m *ModulesConfig) setActiveMods(logger loggers.Logger) error {
- var activeMods Modules
for _, mod := range m.AllModules {
if !mod.Config().HugoVersion.IsValid() {
logger.Warnf(`Module %q is not compatible with this Hugo version; run "hugo mod graph" for more information.`, mod.Path())
}
- if !mod.Disabled() {
- activeMods = append(activeMods, mod)
- }
}
- m.ActiveModules = activeMods
-
return nil
}
@@ -228,7 +219,7 @@ func (c *collector) getVendoredDir(path string) (vendoredModule, bool) {
return v, found
}
-func (c *collector) add(owner *moduleAdapter, moduleImport Import, disabled bool) (*moduleAdapter, error) {
+func (c *collector) add(owner *moduleAdapter, moduleImport Import) (*moduleAdapter, error) {
var (
mod *goModule
@@ -316,11 +307,10 @@ func (c *collector) add(owner *moduleAdapter, moduleImport Import, disabled bool
}
ma := &moduleAdapter{
- dir: moduleDir,
- vendor: vendored,
- disabled: disabled,
- gomod: mod,
- version: version,
+ dir: moduleDir,
+ vendor: vendored,
+ gomod: mod,
+ version: version,
// This may be the owner of the _vendor dir
owner: realOwner,
}
@@ -343,7 +333,7 @@ func (c *collector) add(owner *moduleAdapter, moduleImport Import, disabled bool
return ma, nil
}
-func (c *collector) addAndRecurse(owner *moduleAdapter, disabled bool) error {
+func (c *collector) addAndRecurse(owner *moduleAdapter) error {
moduleConfig := owner.Config()
if owner.projectMod {
if err := c.applyMounts(Import{}, owner); err != nil {
@@ -352,17 +342,18 @@ func (c *collector) addAndRecurse(owner *moduleAdapter, disabled bool) error {
}
for _, moduleImport := range moduleConfig.Imports {
- disabled := disabled || moduleImport.Disable
-
+ if moduleImport.Disable {
+ continue
+ }
if !c.isSeen(moduleImport.Path) {
- tc, err := c.add(owner, moduleImport, disabled)
+ tc, err := c.add(owner, moduleImport)
if err != nil {
return err
}
if tc == nil || moduleImport.IgnoreImports {
continue
}
- if err := c.addAndRecurse(tc, disabled); err != nil {
+ if err := c.addAndRecurse(tc); err != nil {
return err
}
}
@@ -531,7 +522,7 @@ func (c *collector) collect() {
projectMod := createProjectModule(c.gomods.GetMain(), c.ccfg.WorkingDir, c.moduleConfig)
- if err := c.addAndRecurse(projectMod, false); err != nil {
+ if err := c.addAndRecurse(projectMod); err != nil {
c.err = err
return
}
diff --git a/modules/module.go b/modules/module.go
index 42bd94e7b..da963de58 100644
--- a/modules/module.go
+++ b/modules/module.go
@@ -40,9 +40,6 @@ type Module interface {
// Directory holding files for this module.
Dir() string
- // This module is disabled.
- Disabled() bool
-
// Returns whether this is a Go Module.
IsGoMod() bool
@@ -81,7 +78,6 @@ type moduleAdapter struct {
dir string
version string
vendor bool
- disabled bool
projectMod bool
owner Module
@@ -115,10 +111,6 @@ func (m *moduleAdapter) Dir() string {
return m.gomod.Dir
}
-func (m *moduleAdapter) Disabled() bool {
- return m.disabled
-}
-
func (m *moduleAdapter) IsGoMod() bool {
return m.gomod != nil
}