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][C++][Pistache] Generated CMakeLists.txt sets compiler flag c++11 but should set c++14 #4378

Closed
4 of 6 tasks
cjuenger opened this issue Nov 5, 2019 · 1 comment · Fixed by #4386
Closed
4 of 6 tasks

Comments

@cjuenger
Copy link

cjuenger commented Nov 5, 2019

Bug Report Checklist

  • Have you provided a full/minimal spec to reproduce the issue?
  • 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

The cpp-pistache-server generator generates a CMakeLists.txt file that sets c++11 compiler flag. As pistache do not supports c++11 anymore it should be changed to c++14.

openapi-generator version

4.2.0

OpenAPI declaration file content or url
openapi: 3.0.0
info:
  title: Some title
  description: Some description
  version: 0.0.0.0
  
servers:
  - url: http://localhost:8080 
  
paths:
  /doc:
    get:
      operationId: getDoc
      tags:
        - TAG
      security: []
      responses:
        '200':
          description: Documentation of the API.
          content:
            text/html:
              schema:
                type: string
        '500':
          description: Internal Server Error
Command line used for generation

openapi-generator generate -g cpp-pistache-server -i API_SPEC.yaml -o .\src -Dapis=TAG -Dmodels -DsupportingFiles

Steps to reproduce

Generate the cpp-pistache-server sources by executing the command line mentioned above.

Related issues/PRs
Suggest a fix

Change compiler flag from c++11 to c++14.

@auto-labeler
Copy link

auto-labeler bot commented Nov 5, 2019

👍 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.

2 participants