Error in Docker Plugin Force Latest Parameter Sequence?

[AshleyAitken]

Howdy All,

I'm pretty much a beginner at all of this so please understand if this is incorrect.
I am trying to force the "latest" tag in docker native packager and it seems to fail all the time.

I believe this line in DockerPlugin.scala Line 288

val latestCmd = Seq("docker", "tag", "-f", tag, name)

should be

val latestCmd = Seq("docker", "tag", "-f", name, tag)

i.e. sequence of name and tag is incorrect.
If this is an error is it possible to fix please (or do I need to submit a pull request etc).
Thanks,
Ashley.

[muuki88]

docker tag --help says (and docs)

Usage: docker tag [OPTIONS] IMAGE[:TAG] [REGISTRYHOST/][USERNAME/]NAME[:TAG]

Examples show that id comes first and then the tag:

$ sudo docker tag 5db5f8471261 ouruser/sinatra:devel

So I think this should work as we have tests for this as well.

[AshleyAitken]

Sorry @muuki88 I was confused by that, you are right.

[muuki88]

No problem :)