github/hugo
1
0
Fork 0
mirror of https://github.com/gohugoio/hugo.git synced 2024-11-21 20:46:30 -05:00
Code Issues Projects Releases Packages Wiki Activity

Revert "postcss: Improve validation of option 'config'"

Browse source 
This reverts commit 9a0370e8eb.

Closes #10990
This commit is contained in:
Bjørn Erik Pedersen 2023-05-22 20:31:59 +02:00
parent 85b13c105a
commit bd38e35f97
No known key found for this signature in database
GPG key ID: 330E6E2BD4859D8F
3 changed files with 8 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
hugolib/filesystems/basefs.go
View file

@ -188,19 +188,19 @@ func (b *BaseFs) AbsProjectContentDir(filename string) (string, string, error) {
// ResolveJSConfigFile resolves the JS-related config file to a absolute // ResolveJSConfigFile resolves the JS-related config file to a absolute
// filename. One example of such would be postcss.config.js. // filename. One example of such would be postcss.config.js.
func (fs *BaseFs) ResolveJSConfigFile(name string) (string, bool) { func (fs *BaseFs) ResolveJSConfigFile(name string) string {
// First look in assets/_jsconfig // First look in assets/_jsconfig
fi, err := fs.Assets.Fs.Stat(filepath.Join(files.FolderJSConfig, name)) fi, err := fs.Assets.Fs.Stat(filepath.Join(files.FolderJSConfig, name))
if err == nil { if err == nil {
return fi.(hugofs.FileMetaInfo).Meta().Filename, fi.IsDir() return fi.(hugofs.FileMetaInfo).Meta().Filename
} }
// Fall back to the work dir. // Fall back to the work dir.
fi, err = fs.Work.Stat(name) fi, err = fs.Work.Stat(name)
if err == nil { if err == nil {
return fi.(hugofs.FileMetaInfo).Meta().Filename, fi.IsDir() return fi.(hugofs.FileMetaInfo).Meta().Filename
} }
return "", false return ""
} }
// MakePathRelative creates a relative path from the given filename. // MakePathRelative creates a relative path from the given filename.

8
resources/resource_transformers/babel/babel.go
View file

@ -134,17 +134,13 @@ func (t *babelTransformation) Transform(ctx *resources.ResourceTransformationCtx
} }
configFile = filepath.Clean(configFile) configFile = filepath.Clean(configFile)
isConfigFileDir := false
// We need an absolute filename to the config file. // We need an absolute filename to the config file.
if !filepath.IsAbs(configFile) { if !filepath.IsAbs(configFile) {
configFile, isConfigFileDir = t.rs.BaseFs.ResolveJSConfigFile(configFile) configFile = t.rs.BaseFs.ResolveJSConfigFile(configFile)
if isConfigFileDir {
logger.Warnf("babel config %q must be a file, not a directory", configFile)
}
if configFile == "" && t.options.Config != "" { if configFile == "" && t.options.Config != "" {
// Only fail if the user specified config file is not found. // Only fail if the user specified config file is not found.
return fmt.Errorf("babel config file %q not found", configFile) return fmt.Errorf("babel config %q not found:", configFile)
} }
} }

8
resources/resource_transformers/postcss/postcss.go
View file

@ -172,17 +172,13 @@ func (t *postcssTransformation) Transform(ctx *resources.ResourceTransformationC
} }
configFile = filepath.Clean(configFile) configFile = filepath.Clean(configFile)
isConfigFileDir := false
// We need an absolute filename to the config file. // We need an absolute filename to the config file.
if !filepath.IsAbs(configFile) { if !filepath.IsAbs(configFile) {
configFile, isConfigFileDir = t.rs.BaseFs.ResolveJSConfigFile(configFile) configFile = t.rs.BaseFs.ResolveJSConfigFile(configFile)
if configFile == "" && options.Config != "" { if configFile == "" && options.Config != "" {
// Only fail if the user specified config file is not found. // Only fail if the user specified config file is not found.
return fmt.Errorf("postcss config directory %q not found", options.Config) return fmt.Errorf("postcss config %q not found:", options.Config)
}
if !isConfigFileDir {
logger.Warnf("postcss config %q must be a directory", options.Config)
} }
} }