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.

Sign up for GitHub

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

Jump to bottom

CMakeParseArguments patch not accepted in CMake #36

Closed
drdanz opened this issue May 21, 2014 · 1 comment
Closed

CMakeParseArguments patch not accepted in CMake #36

drdanz opened this issue May 21, 2014 · 1 comment
Assignees
@drdanz
Labels
Component: CMake Legacy Component: CMake Next Module: CMakeParseArguments Severity: Minor Status: Backlog
Milestone
Version 0.4

Comments

@drdanz
Copy link
Member

drdanz commented May 21, 2014

The patch for handling empty arguments in CMakeParseArguments was not accepted, because this module is going to be changed in a command, and this feature should be included see http://thread.gmane.org/gmane.comp.programming.tools.cmake.devel/8546

Nonetheless we require this patch in YCMEPHelper, therefore we cannot drop it. Also when the command will be implemented in cmake, it will still be required for legacy versions.

Nonetheless I'm not sure if it makes sense to keep it in "cmake-next", maybe a "cmake-legacy" folder would make more sense.
@drdanz drdanz added this to the Version 0.2 milestone May 26, 2014
@drdanz drdanz self-assigned this Nov 17, 2016
@drdanz drdanz modified the milestones: Version 0.4, Version 0.2 Nov 17, 2016
@traversaro
Copy link
Member

Issue superseded by #454 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@drdanz drdanz

Labels
Component: CMake Legacy Component: CMake Next Module: CMakeParseArguments Severity: Minor Status: Backlog
Projects
None yet
Milestone
Version 0.4
Development

No branches or pull requests
2 participants
@drdanz @traversaro