Having invalid (but unused) go files anywhere in the project path fails packr build about packr HOT 13 CLOSED

I guess the obvious solution would be to include .idea into the list of directories skipped by builder.Builder's Run() method.

I can provide a PR if you like.

from packr.

A PR would be excellent!

from packr.

Sure, I've submitted the PR. Thanks!

from packr.

I have the same issue with my templating program, i have templates that are invalid go code and so it refuses to build

for example

// Copyright {{.Year}} {{.Author}}. All rights reserved.
// Use of this source code is governed by a BSD-style
// license that can be found in the LICENSE file.

package {{.Project}}

I could work around this by naming them something like .goo and having my program change the names, but i'd rather this was fixed

from packr.

from packr.

@markbates Maybe i'll do that, seems like a overhead when creating templates since you would need to name them foo.go.tmpl and that is kind of busywork

from packr.

from packr.

You misunderstand me, go build works fine packr build throws this error

➜  gocs git:(master) ✗ packr build
Error: go/src/github.com/UlisseMini/gocs/goc_default/templates/default/{{Project}}.go:5:9: expected 'IDENT', found '{'
Usage:
  packr build [flags]

Flags:
  -h, --help   help for build

Global Flags:
  -v, --verbose   print verbose logging information

I don't see why packr needs to scan all .go files including in subdirectories and make sure they are syntastically valid.

from packr.

from packr.

If packr follows go rules, why does packr build fail when go build is fine?

from packr.

I've fixed it by changing templates to _templates, thanks for the help :)

from packr.

Just to clarify what's happening. Packr scans your .go files looking for boxes, using go/package. Because those files aren't "proper" go files, the Go tools to parse the package error out. You need to tell Packr to skip those when parsing, to do that it follows the same conventions Go does when trying to figure out what it should/shouldn't build.

from packr.

Thanks for explaining, it makes sense now :)

from packr.