Skip to content

Commit

Permalink
Fix duplicate mount sources
Browse files Browse the repository at this point in the history
Fixes #9426
  • Loading branch information
bep committed Jan 26, 2022
1 parent 2655739 commit 7a080b6
Show file tree
Hide file tree
Showing 4 changed files with 39 additions and 8 deletions.
2 changes: 1 addition & 1 deletion hugofs/walk.go
Expand Up @@ -270,7 +270,7 @@ func (w *Walkway) walk(path string, info FileMetaInfo, dirEntries []FileMetaInfo
meta.Path = normalizeFilename(pathMeta)
meta.PathWalk = pathn

if fim.IsDir() && w.isSeen(meta.Filename) {
if fim.IsDir() && meta.IsSymlink && w.isSeen(meta.Filename) {
// Prevent infinite recursion
// Possible cyclic reference
meta.SkipDir = true
Expand Down
6 changes: 1 addition & 5 deletions hugofs/walk_test.go
Expand Up @@ -81,14 +81,12 @@ func TestWalkRootMappingFs(t *testing.T) {
}

c.Run("Basic", func(c *qt.C) {

bfs := prepare(c)

names, err := collectFilenames(bfs, "", "")

c.Assert(err, qt.IsNil)
c.Assert(names, qt.DeepEquals, []string{"a/test.txt", "b/test.txt", "c/test.txt"})

})

c.Run("Para", func(c *qt.C) {
Expand All @@ -112,12 +110,10 @@ func TestWalkRootMappingFs(t *testing.T) {
return errors.New("fail")
}
return nil

})
}

c.Assert(r.Wait(), qt.IsNil)

})
}

Expand Down Expand Up @@ -169,7 +165,7 @@ func TestWalkSymbolicLink(t *testing.T) {
names, err := collectFilenames(fs, workDir, workDir)
c.Assert(err, qt.IsNil)

c.Assert(names, qt.DeepEquals, []string{"blog/real/sub/a.txt", "docs/b.txt"})
c.Assert(names, qt.DeepEquals, []string{"blog/real/sub/a.txt", "blog/symlinked/sub/a.txt", "docs/b.txt"})
})

t.Run("BasePath Fs", func(t *testing.T) {
Expand Down
3 changes: 1 addition & 2 deletions hugolib/filesystems/basefs_test.go
Expand Up @@ -33,7 +33,6 @@ import (
"github.com/gohugoio/hugo/hugofs"
"github.com/gohugoio/hugo/hugolib/paths"
"github.com/gohugoio/hugo/modules"

)

func initConfig(fs afero.Fs, cfg config.Provider) error {
Expand Down Expand Up @@ -152,7 +151,7 @@ theme = ["atheme"]
checkFileCount(bfs.Data.Fs, "", c, 11) // 7 + 4 themes
checkFileCount(bfs.Archetypes.Fs, "", c, 10) // 8 + 2 themes
checkFileCount(bfs.Assets.Fs, "", c, 9)
checkFileCount(bfs.Work, "", c, 82)
checkFileCount(bfs.Work, "", c, 90)

c.Assert(bfs.IsData(filepath.Join(workingDir, "mydata", "file1.txt")), qt.Equals, true)
c.Assert(bfs.IsI18n(filepath.Join(workingDir, "myi18n", "file1.txt")), qt.Equals, true)
Expand Down
36 changes: 36 additions & 0 deletions hugolib/hugo_modules_test.go
Expand Up @@ -1134,3 +1134,39 @@ P1: {{ $p1.Title }}|{{ $p1.RelPermalink }}|Filename: {{ $p1.File.Filename }}

b.AssertFileContent("public/index.html", "P1: Abs|/p1/", "Filename: "+contentFilename)
}

// Issue 9426
func TestMountSameSource(t *testing.T) {
config := `baseURL = 'https://example.org/'
languageCode = 'en-us'
title = 'Hugo GitHub Issue #9426'
disableKinds = ['RSS','sitemap','taxonomy','term']
[[module.mounts]]
source = "content"
target = "content"
[[module.mounts]]
source = "extra-content"
target = "content/resources-a"
[[module.mounts]]
source = "extra-content"
target = "content/resources-b"
`
b := newTestSitesBuilder(t).WithConfigFile("toml", config)

b.WithContent("p1.md", "")

b.WithSourceFile(
"extra-content/_index.md", "",
"extra-content/subdir/_index.md", "",
"extra-content/subdir/about.md", "",
)

b.Build(BuildCfg{})

b.AssertFileContent("public/resources-a/subdir/about/index.html", "Single")
b.AssertFileContent("public/resources-b/subdir/about/index.html", "Single")
}

0 comments on commit 7a080b6

Please sign in to comment.