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

[3.6] make tags: index also Modules/_ctypes/ (#4648) #4659

Merged
merged 2 commits into from
Nov 30, 2017
Merged

[3.6] make tags: index also Modules/_ctypes/ (#4648) #4659

merged 2 commits into from
Nov 30, 2017

Conversation

vstinner
Copy link
Member

Avoid also "cd $(srcdir)" to not change the current directory.

(cherry picked from commit 3be3b97)

@vstinner vstinner changed the title [2.7] make tags: index also Modules/_ctypes/ (#4648) [3.6] make tags: index also Modules/_ctypes/ (#4648) Nov 30, 2017
dzeban and others added 2 commits November 30, 2017 22:57
* Fix `make tags` warnings

`make tags` target tries to find C sources and headers in "Grammar" and
"Mac" folders and generates these warnings:

    ctags: Warning: cannot open source file "Grammar/*.[ch]" : No such file or directory
    ctags: Warning: cannot open source file "Mac/*.[ch]" : No such file or directory

This commit changes $SRCDIRS variable in configure.ac to remote these
directories. This variable is used only for tags generation.

Also, "configure" was regenerated with `autoreconf`.

* Fix `make tags` fail on non-default tag names

When ctags overrides default tags filename (e.g. `-f .tags`) `make tags`
is failed because it assumes to see default `tags` filename:

    sort: cannot read: tags: No such file or directory

This commit explicitly specifies "tags" filename for tags generation.

(cherry picked from commit 8a543c0)
Avoid also "cd $(srcdir)" to not change the current directory.

(cherry picked from commit 3be3b97)
@vstinner vstinner merged commit 29cb50b into python:3.6 Nov 30, 2017
@vstinner vstinner deleted the make_tags_ctypes36 branch November 30, 2017 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants