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

Support for special file names, CMake, Makefiles and Rakefiles #1141

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

osimola
Copy link

@osimola osimola commented Aug 15, 2017

This PR adds special filenames to go with extensions, to add support for CMakeLists.txt, Makefile, Rakefile and possible other special files.

A second commit reduces copying and fixes potential buffer overrun if a single filename/extension takes over SINGLE_EXT_LEN or more or a single language has more than MAX_EXTENSIONS extensions, which would overwrite the zero byte or null pointer at end of array.

@aswild
Copy link
Contributor

aswild commented Aug 26, 2017

Hey, thanks for this patch! I've been wanting ag --make to work properly for a while but never got around to fixing it myself.

@osimola
Copy link
Author

osimola commented Aug 26, 2017

Hi. Looks like there was a release recently and this PR was not either merged or explicitly rejected. Is there any way I could help this getting merged?
There is also a similar older pull request, #981 that has also been ignored. Merging either one of these would be great help to CMake users.

@aswild
Copy link
Contributor

aswild commented Aug 26, 2017

Yeah, this project has quite a few stale/ignored PRs, but such is life in the open source world.

If it makes you feel any better, I merged this into my personal fork. 😃 I hope it goes upstream still because resolving filetype merge conflicts will be a bitch until then.

@osimola osimola mentioned this pull request May 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants