[Einstein Toolkit] #1842: clang-format options not valid for versions <=3.5

List overview All Threads
Download

newer

older

[Einstein Toolkit] #1887: Error...

[Einstein Toolkit] #1883:...

Einstein Toolkit

24 Nov 2015 24 Nov '15

10:42 a.m.

#1842: clang-format options not valid for versions <=3.5 --------------------+------------------------------------------------------- Reporter: knarf | Owner: Type: defect | Status: new Priority: unset | Milestone: Component: Cactus | Version: development version Keywords: | --------------------+------------------------------------------------------- The current .clang-format file is not valid for clang-format versions 3.5 and below. The attached patch would 'fix' this by removing key words not known in version 3.5, but still wouldn't work for versions below that. I do not suggest to apply the patch - it should just show that this is quite some number.

Looking at the .clang-format file it seems that it does not use one of the pre-defined styles, but rather creates it's own (even if it is based on one given the comment in the file). I wonder if all of this is really necessary, or if we could not simply use one of the pre-defined styles, possibly with a few changes that work across clang-format versions.

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/1842 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Show replies by date

Einstein Toolkit

24 Nov 24 Nov

10:55 a.m.

#1842: clang-format options not valid for versions <=3.5 ---------------------+------------------------------------------------------ Reporter: knarf | Owner: Type: defect | Status: new Priority: unset | Milestone: Component: Cactus | Version: development version Resolution: | Keywords: ---------------------+------------------------------------------------------

Comment (by eschnett):

The .clang-format file is exactly the (default) LLVM style, with the exception of setting "Standard" to "Cpp03".

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/1842#comment:1 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

2:20 p.m.

Comment (by knarf):

Replying to [comment:1 eschnett]:

...

The .clang-format file is exactly the (default) LLVM style, with the

exception of setting "Standard" to "Cpp03".

That may be true for version 3.6. These don't work for earlier versions. The output of {{{ clang-format-XXX -style=llvm -dump-config }}} is attached to the ticket - and they are different from the current state in the flesh, and are different from each other.

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/1842#comment:2 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

2:45 p.m.

Comment (by knarf):

I tried the following: {{{ --- BasedOnStyle: LLVM Standard: Cpp03 ... }}} If I read the documentation correctly, this should result in a style based on LLVM, but with the standard being changed - which I also understood you wanted?

While a quick check using this and version 3.5 works, it reveals still differences to what is currently in Carpet:

$ git diff | wc -l 178

The same using version 3.4 also works, but shows more changes:

$ git diff | wc -l 687

The same happens without .clang-format for {{{ clang-format-XXX -style="{Standard: Cpp03}" -i *.cc *.hh }}}

I don't have version 3.6 or newer.

This to me indicates that different versions of clang-format format 'llvm- style' differently. But we also cannot (easily) specify our own style, since the options necessary to do so don't seem to work across versions.

Unless I misunderstood something (perfectly possible), we either have to dictate a certain version of clang-format (difficult to enforce and might still break existing editor-build-setups), or we would have to come up with a configuration that works at least across some commonly installed versions of clang-format in a consistent manner.

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/1842#comment:3 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

2:49 p.m.

Comment (by knarf):

I also tried using the config from version 3.4 with version 3.5 - to no avail (Standard it set to Cpp03 there by default): {{{ $ clang-format-3.4 -style=llvm -dump-config > .clang-format $ clang-format-3.4 -style="{Standard: Cpp03}" -i *.cc *.hh $ git diff | wc -l 687$ clang-format-3.4 -style=file -i *.cc *.hh $ git diff | wc -l 687 $ clang-format-3.5 -style="{Standard: Cpp03}" -i *.cc *.hh $ git diff | wc -l 178 $ clang-format-3.5 -style=file -i *.cc *.hh $ git diff | wc -l 178

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/1842#comment:4 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

25 Nov 25 Nov

8:44 p.m.

Comment (by eschnett):

Since the global .clang-format file doesn't work, the option to use seems to be: {{{ clang-format -style="{BasedOnStyle: LLVM, Standard: Cpp03}" }}}

Unfortunately, it seems that different clang-format versions make slightly different decisions regarding how to break lines or how to interpret ampersand characters. (The latter are difficult to interpret if one doesn't have full type information.) It thus seems that re-formatting everything with different versions of clang-format leads to changes.

I'm using clang-format 3.7, the latest released version. Clang is relatively easy to install (easier than GCC), and we should be able to decide on a single version for the Einstein Toolkit. For example, Ubuntu LTS currently provides version 3.6.

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/1842#comment:5 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

26 Nov 26 Nov

1:21 a.m.

Comment (by rhaas):

Which version was used for the already formated thorns? We should use that one otherwise we have to re-format everything if the chosen version produces a different formatting.

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/1842#comment:6 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

2:03 a.m.

Comment (by knarf):

Replying to [comment:5 eschnett]:

...

I'm using clang-format 3.7, the latest released version. Clang is

relatively easy to install (easier than GCC), and we should be able to decide on a single version for the Einstein Toolkit. For example, Ubuntu LTS currently provides version 3.6.

Debian (stable) has version 3.5 (and 3.4), not 3.6 or newer.

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/1842#comment:7 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

8 May 8 May

4:41 a.m.

Comment (by hinder):

3.9, the latest version available in MacPorts, also produces changes on current master. But do we know that current master is "clean" with respect to 3.7, which Erik was using 5 months ago (comment:5)?

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/1842#comment:8 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

4:52 a.m.

Comment (by hinder):

Installing using MacPorts pulls in many gigabytes of files, including all of clang and all of llvm. It is much easier to install using homebrew;

{{{ $ brew install clang-format ==> Downloading https://homebrew.bintray.com/bottles/clang- format-2015-04-21.mavericks.bottle.tar.gz ######################################################################## 100.0% ==> Pouring clang-format-2015-04-21.mavericks.bottle.tar.gz /usr/local/Cellar/clang-format/2015-04-21: 5 files, 1.4M }}}

This gives {{{ clang-format version 3.7.0 (tags/google/testing/2015-04-21) }}} which is the version Erik mentioned using. However, this version also produces changes in current master, similar to the ones I got in 3.9. I suspect that current master is not "clean". Erik, please can you check?

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/1842#comment:9 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

3805

Age (days ago)

3971

Last active (days ago)

trac@lists.einsteintoolkit.org

9 comments

1 participants

tags (0)

participants (1)

Einstein Toolkit