Consider changing --from-bundles (bool) to --bundle-file (string)

[janetkuo]

How about we make the flag for specifying bundles to something similar to --file (e.g. --bundle-file)? It'll be easier to specify than --from-bundles + --file=xxx. We can validate the flags so that either --file or --bundle-file can be specified at a time.

[ngtuna]

+1. when adding --from-bundles flag I was just thinking about reusing --file flag. Could you make a commit for this?

[janetkuo]

Sure
On Thu, Jul 21, 2016 at 7:30 PM Tuna notifications@github.com wrote:

+1. when adding --from-bundles flag I was just thinking about reusing
--file flag. Could you make a commit for this?

—
You are receiving this because you authored the thread.

Reply to this email directly, view it on GitHub
#53 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEpqQ0kV4Xg4hzgoc7scb2OhlRBBDzGFks5qYCtKgaJpZM4JSVqW
.

[sebgoa]

so +1 I confirm, this is not very intuitive:

$ ./kompose convert --from-bundles ./docker-compose-bundle.dsb 
FATA[0000] Failed to parse bundles file: %!(EXTRA *errors.errorString=JSON syntax error at byte 1: invalid character 'm' looking for beginning of value) 
$ ./kompose convert --from-bundles --file docker-compose-bundle.dsb 
file "web-svc.json" created
file "redis-svc.json" created
file "web-deployment.json" created
file "redis-deployment.json" created

[ngtuna]

@janetkuo: after you finish the patch to introduce --bundle-file flag, I will update README for both bundlefile and compose v2 guidances

#54