clang-tidy and 3.7 release notes

Hi,

It was pointed out to me that there's been a bunch of work on new
checkers, but there's no mention of this in the 3.7 release notes.

Would someone be willing to write something about this?

Cheers,
Hans

I’ll try to come up with a short summary. Where exactly should the text go?

– Alex

I figured in Clang's release notes, docs/ReleaseNotes.rst on the 3.7
branch: http://llvm.org/viewvc/llvm-project/cfe/branches/release_37/docs/ReleaseNotes.rst?view=log

If you don't want to check out the branch, you can also just send me
some text and I'll commit it.

Thanks,
Hans

In article <CAOweq9+cUTyTKJd-+ex5wnfW4q+jZ8hHjfaztxzOSz879C=siA@mail.gmail.com>,
    Alexander Kornienko via cfe-dev <cfe-dev@lists.llvm.org> writes:

OK, clang-tidy summary of changes from 3.6.2 final:

Nice!

Is clang-tidy, clang-modernize, etc., getting included in the built
distributions yet?
<https://llvm.org/bugs/show_bug.cgi?id=22765&gt;

Many thanks! r245328.

- Hans

I hadn't seen this bug :frowning:

Unfortunately it seems the release script isn't building the code in
clang-tools-extra. This is easy to fix, but I'm unsure whether doing
so at this point in the release is a good idea.

- Hans

>
> In article <CAOweq9+cUTyTKJd-+ex5wnfW4q+jZ8hHjfaztxzOSz879C=
siA@mail.gmail.com>,
> Alexander Kornienko via cfe-dev <cfe-dev@lists.llvm.org> writes:
>
>> OK, clang-tidy summary of changes from 3.6.2 final:
>
> Nice!
>
> Is clang-tidy, clang-modernize, etc., getting included in the built
> distributions yet?
> <https://llvm.org/bugs/show_bug.cgi?id=22765&gt;

I hadn't seen this bug :frowning:

Unfortunately it seems the release script isn't building the code in
clang-tools-extra. This is easy to fix, but I'm unsure whether doing
so at this point in the release is a good idea.

I see you've decided to try
<https://llvm.org/bugs/show_bug.cgi?id=22765#c3&gt; including
clang-tools-extra tools to the release. As it turns out to be the first
release containing clang-tidy, it may make sense to describe either all
existing checks or none of them, and additionally add a top-level
description of clang-tidy itself (can send you tomorrow). WDYT?

-- Alex

They were part of the previous releases too, just not in the pre-built
binaries. As most end-users get clang via their linux distro or
similar, I don't think this really changes much.

We kind of have a high-level doc for these tools:
docs/ClangTools.html. That mentions clang-check, clang-format and
clang-modernize. Maybe we should add some documentation of clang-tidy
there, unless it's already documented somewhere else?

- Hans

It’s documented in http://clang.llvm.org/extra/clang-tidy.html, but it may make sense to add a reference to the document you mentioned.

In any case, if the previous release contained the tool, the release notes should be fine then.