[Bug 1047260] Don't delete jsoncheck from the package

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1047260



--- Comment #2 from Dridi Boukelmoune <dridi.boukelmoune at gmail.com> ---
Hi,

I'm sorry I've broken the syntastic integration (trust me I love syntastic) but
I have removed jsoncheck for a couple reasons:

1) it doesn't exist in the current master branch

It means that I could only upgrade shellcheck with a new fedora release,
because the removal of jsoncheck would be a breaking change.

With the next version you'll simply do:
shellckeck -f json some_script

And it will come with a man page :)

2) jsoncheck is misleading to me

It sounds more like a tool that would check some json, and it doesn't feel
legitimate in a "shell checking" package to me.

I see several solutions, assuming the next shellcheck will hopefully come
before f21:
- press upstream to get a release soon (there are other features I also want)
- backport the "-f json" behavior
- restore jsoncheck, and postpone the next update to f21
- restore jsoncheck, and maintain a substitute script with the next updates
(and remove it in f21)

In any case, I'd need some coordination with the syntastic maintainer.

What do you think ?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=wCujiNP3kO&a=cc_unsubscribe