Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] docker commands in README don't work #6845

Closed
4 of 5 tasks
mdavis-xyz opened this issue Jul 2, 2020 · 1 comment · Fixed by #6846
Closed
4 of 5 tasks

[BUG] docker commands in README don't work #6845

mdavis-xyz opened this issue Jul 2, 2020 · 1 comment · Fixed by #6846

Comments

@mdavis-xyz
Copy link
Contributor

mdavis-xyz commented Jul 2, 2020

Bug Report Checklist

  • Have you provided a full/minimal spec to reproduce the issue?
  • [n/a] Have you validated the input using an OpenAPI validator (example)?
  • What's the version of OpenAPI Generator used?
  • Have you search for related issues/PRs?
  • What's the actual output vs expected output?
  • [Optional] Bounty to sponsor the fix (example)
Description
openapi-generator version
openapi-generator-cli 5.0.0-SNAPSHOT
  commit : 068ad02
  built  : 2020-07-02T17:04:18Z
  source : https://github.com/openapitools/openapi-generator
  docs   : https://openapi-generator.tech/
OpenAPI declaration file content or url

see below

Command line used for generation

see below

Steps to reproduce
  1. Go to the development-in-docker section of the readme.
  2. Follow the instructions, copy-pasting in each command

expected result

The commands run sucessfully and generate code.

actual result

$ ./run-in-docker.sh generate -i modules/openapi-generator/src/test/resources/2_0/petstore.yaml     -g go -o /gen/out/go-petstore -DpackageName=petstore
++ dirname ./run-in-docker.sh
+ cd .
+ maven_cache_repo=/home/ec2-user/.m2/repository
+ mkdir -p /home/ec2-user/.m2/repository
++ id -u
++ id -g
+ docker run --rm -it -w /gen -e GEN_DIR=/gen -e MAVEN_CONFIG=/var/maven/.m2 -e 'MAVEN_OPTS=-Dhttps.protocols=TLSv1.2 -Dmaven.repo.local=/var/maven/.m2/repository -Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=WARN -Dorg.slf4j.simpleLogger.showDateTime=true -Djava.awt.headless=true -Djacoco.skip=true' -u 501:501 -v /home/ec2-user/environment/dispatch_swagger/docker/openapi-generator:/gen -v /home/ec2-user/environment/dispatch_swagger/docker/openapi-generator/CI/run-in-docker-settings.xml:/var/maven/.m2/settings.xml -v /home/ec2-user/.m2/repository:/var/maven/.m2/repository --entrypoint /gen/docker-entrypoint.sh maven:3-jdk-8 generate -i modules/openapi-generator/src/test/resources/2_0/petstore.yaml -g go -o /gen/out/go-petstore -DpackageName=petstore
[error] Found unexpected parameters: [-DpackageName=petstore]

See 'openapi-generator-cli help' for usage.
Related issues/PRs

N/A

Suggest a fix

The documentation should clarify how to pass in options like packageName in the command line.
Including on pages like this

@auto-labeler
Copy link

auto-labeler bot commented Jul 2, 2020

👍 Thanks for opening this issue!
🏷 I have applied any labels matching special text in your issue.

The team will review the labels and make any necessary changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant