schema/gen: Include .json instead of excluding .go

[wking]

To avoid accidentally including .swp, .un~, etc. Test with:

$ touch schema/.defs-image.json.swp
$ touch schema/.content-descriptor.json.un~
$ touch schema/foojson
$ make schema-fs
$ grep '"/' schema/fs.go
        "/config-schema.json": {
        "/content-descriptor.json": {
        "/defs-config.json": {
        "/defs-image.json": {
        "/defs.json": {
        "/image-layout-schema.json": {
        "/image-manifest-schema.json": {
        "/manifest-list-schema.json": {
        "/": {
                local: "/",

CC @vbatts.

[vbatts]

this didn't work for me ☹️
I had to just add to the ignores.

[wking]

On Mon, Jan 30, 2017 at 09:40:52AM -0800, Vincent Batts wrote: this didn't work for me :-\

The test I ran above [1] worked for me. Are you using a fresh esc install? Maybe it depends on Go version (I'm running 1.7.1)? [1]: #538 (comment)

[vbatts]

sorry, it works, but when i had tmp files (vim .*.swp files) the pattern still built them into the schema-fs.

[wking]

On Tue, Jan 31, 2017 at 10:40:23AM -0800, Vincent Batts wrote: sorry, it _works_, but when i had tmp files (vim `.*.swp` files) the pattern still built them into the schema-fs.

In my local testing, I added a schema/.defs-image.json.swp file, and it was not included in fs.go [1]. Can you give me steps to reproduce your inclusion? Maybe with a Dockerfile? Or by sharing your Go, esc, etc., versions? For example, here is it not happening in a Docker container where I've created a .defs-image.json.swp: $ docker run --rm golang:1.7.5 sh -c 'go get -d github.com/opencontainers/image-spec/schema && cd $GOPATH/src/github.com/opencontainers/image-spec && git config --add remote.origin.fetch "+refs/pull/*/head:refs/remotes/origin/pr/*" && git fetch -q origin && git checkout origin/pr/538 && go get -u github.com/mjibson/esc && touch schema/.defs-image.json.swp schema/defs.json && make schema-fs && grep "\"/" schema/fs.go' … HEAD is now at eab6872... schema/gen: Include .json instead of excluding .go generating schema fs "/config-schema.json": { "/content-descriptor.json": { "/defs-config.json": { "/defs-image.json": { "/defs.json": { "/image-layout-schema.json": { "/image-manifest-schema.json": { "/manifest-list-schema.json": { "/": { [1]: #538 (comment)

[vbatts]

rebase pls

[wking]

On Tue, Feb 14, 2017 at 01:42:06PM -0800, Vincent Batts wrote: rebase pls

Rebased with eab6872 → 6f633c2.

[vbatts]

LGTM

[jonboulle]

lgtm