-
Bug
-
Resolution: Fixed
-
P3
-
None
-
b112
It would be great if we could try to give a consistent formatting before we make this the new build system that should live for another 20 years. :-)
Since there are no well-known and accepted formatting rules for makefiles and configure script ("m4sh"), Erik and I suggested the following rules:
* In makefiles, use four spaces for indentation. Use tabs *ONLY* were required by Make for rule interpretation.
* In autoconf (*.m4) files, use two spaces for indentation.
The rationale is this:
* Four spaces matches the Java indentation level, and it felt like a good amount.
* Two spaces is more suitable in autoconf files, since while you can typically get some editor assistance for makefile indentation, no known editor understands "m4sh", the weird "language" that configure scipts are written in, so all indentaion has to be done manually (and thus two spaces is simpler than four).
Also, when Erik, Jonas and I discussed formatting in May, we decided that we should try to keep to this rule:
* In the Setup*Compilation macro, there should never be any spaces before or after the = in the named argument lists.
We should try to make this consistent as well.
Since there are no well-known and accepted formatting rules for makefiles and configure script ("m4sh"), Erik and I suggested the following rules:
* In makefiles, use four spaces for indentation. Use tabs *ONLY* were required by Make for rule interpretation.
* In autoconf (*.m4) files, use two spaces for indentation.
The rationale is this:
* Four spaces matches the Java indentation level, and it felt like a good amount.
* Two spaces is more suitable in autoconf files, since while you can typically get some editor assistance for makefile indentation, no known editor understands "m4sh", the weird "language" that configure scipts are written in, so all indentaion has to be done manually (and thus two spaces is simpler than four).
Also, when Erik, Jonas and I discussed formatting in May, we decided that we should try to keep to this rule:
* In the Setup*Compilation macro, there should never be any spaces before or after the = in the named argument lists.
We should try to make this consistent as well.
- duplicates
-
JDK-8005352 build-infra: Make sure all new files follows recommended indentation and formatting
- Closed
- relates to
-
JDK-8027117 adapt JDK-7165611 to new build-infra whitespace/indent policy
- Resolved