summaryrefslogtreecommitdiffstats
path: root/commands
diff options
context:
space:
mode:
authorBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>2022-04-17 10:35:01 +0200
committerBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>2022-04-17 10:35:01 +0200
commit6c35a1a9eacf2aa86a11ecd31c4022ce330b2f16 (patch)
treec390750370d67e646446bdc64c22b430aeb1292c /commands
parent363bc907c0ab4b6989bada2e1ffca4ef17d7b8a4 (diff)
Revert "Fix PostProcess regression for hugo server"
Diffstat (limited to 'commands')
-rw-r--r--commands/commandeer.go28
-rw-r--r--commands/commands_test.go4
-rw-r--r--commands/server_test.go46
3 files changed, 28 insertions, 50 deletions
diff --git a/commands/commandeer.go b/commands/commandeer.go
index c42de5d11..1162a4b70 100644
--- a/commands/commandeer.go
+++ b/commands/commandeer.go
@@ -395,23 +395,23 @@ func (c *commandeer) loadConfig() error {
}
c.fsCreate.Do(func() {
- // Assume both source and destination are using same filesystem.
- fs := hugofs.NewFromSourceAndDestination(sourceFs, sourceFs, config)
+ fs := hugofs.NewFrom(sourceFs, config)
if c.publishDirFs != nil {
// Need to reuse the destination on server rebuilds.
fs.PublishDir = c.publishDirFs
fs.PublishDirServer = c.publishDirServerFs
} else {
- if c.renderStaticToDisk {
- publishDirStatic := config.GetString("publishDirStatic")
- workingDir := config.GetString("workingDir")
- absPublishDirStatic := paths.AbsPathify(workingDir, publishDirStatic)
+ publishDir := config.GetString("publishDir")
+ publishDirStatic := config.GetString("publishDirStatic")
+ workingDir := config.GetString("workingDir")
+ absPublishDir := paths.AbsPathify(workingDir, publishDir)
+ absPublishDirStatic := paths.AbsPathify(workingDir, publishDirStatic)
- fs = hugofs.NewFromSourceAndDestination(sourceFs, afero.NewMemMapFs(), config)
- // Writes the dynamic output to memory,
+ if c.renderStaticToDisk {
+ // Writes the dynamic output oton memory,
// while serve others directly from /public on disk.
- dynamicFs := fs.PublishDir
+ dynamicFs := afero.NewMemMapFs()
staticFs := afero.NewBasePathFs(afero.NewOsFs(), absPublishDirStatic)
// Serve from both the static and dynamic fs,
@@ -427,10 +427,18 @@ func (c *commandeer) loadConfig() error {
},
},
)
+ fs.PublishDir = dynamicFs
fs.PublishDirStatic = staticFs
} else if createMemFs {
// Hugo writes the output to memory instead of the disk.
- fs = hugofs.NewFromSourceAndDestination(sourceFs, afero.NewMemMapFs(), config)
+ fs.PublishDir = new(afero.MemMapFs)
+ fs.PublishDirServer = fs.PublishDir
+ fs.PublishDirStatic = fs.PublishDir
+ } else {
+ // Write everything to disk.
+ fs.PublishDir = afero.NewBasePathFs(afero.NewOsFs(), absPublishDir)
+ fs.PublishDirServer = fs.PublishDir
+ fs.PublishDirStatic = fs.PublishDir
}
}
diff --git a/commands/commands_test.go b/commands/commands_test.go
index 97d81ec6e..e3ec7bd99 100644
--- a/commands/commands_test.go
+++ b/commands/commands_test.go
@@ -375,10 +375,6 @@ Single: {{ .Title }}
List: {{ .Title }}
Environment: {{ hugo.Environment }}
-For issue 9788:
-{{ $foo :="abc" | resources.FromString "foo.css" | minify | resources.PostProcess }}
-PostProcess: {{ $foo.RelPermalink }}
-
`)
return dir
diff --git a/commands/server_test.go b/commands/server_test.go
index c2aa0dfd5..ea50afd94 100644
--- a/commands/server_test.go
+++ b/commands/server_test.go
@@ -31,6 +31,16 @@ import (
qt "github.com/frankban/quicktest"
)
+func TestServer(t *testing.T) {
+ c := qt.New(t)
+
+ r := runServerTest(c, true, "")
+
+ c.Assert(r.err, qt.IsNil)
+ c.Assert(r.homeContent, qt.Contains, "List: Hugo Commands")
+ c.Assert(r.homeContent, qt.Contains, "Environment: development")
+}
+
// Issue 9518
func TestServerPanicOnConfigError(t *testing.T) {
c := qt.New(t)
@@ -91,42 +101,6 @@ baseURL="https://example.org"
}
-func TestServerBugs(t *testing.T) {
- c := qt.New(t)
-
- for _, test := range []struct {
- name string
- flag string
- assert func(c *qt.C, r serverTestResult)
- }{
- // Issue 9788
- {"PostProcess, memory", "", func(c *qt.C, r serverTestResult) {
- c.Assert(r.err, qt.IsNil)
- c.Assert(r.homeContent, qt.Contains, "PostProcess: /foo.min.css")
- }},
- {"PostProcess, disk", "--renderToDisk", func(c *qt.C, r serverTestResult) {
- c.Assert(r.err, qt.IsNil)
- c.Assert(r.homeContent, qt.Contains, "PostProcess: /foo.min.css")
- }},
- } {
- c.Run(test.name, func(c *qt.C) {
- config := `
-baseURL="https://example.org"
-`
-
- var args []string
- if test.flag != "" {
- args = strings.Split(test.flag, "=")
- }
- r := runServerTest(c, true, config, args...)
- test.assert(c, r)
-
- })
-
- }
-
-}
-
type serverTestResult struct {
err error
homeContent string