[Webkit-unassigned] [Bug 103605] [EFL] uniformed coding style in cmake files
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Nov 29 19:27:17 PST 2012
https://bugs.webkit.org/show_bug.cgi?id=103605
Laszlo Gombos <laszlo.gombos at webkit.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #176659|review? |review-
Flag| |
--- Comment #2 from Laszlo Gombos <laszlo.gombos at webkit.org> 2012-11-29 19:29:33 PST ---
(From update of attachment 176659)
View in context: https://bugs.webkit.org/attachment.cgi?id=176659&action=review
Thank you for the initiative. I do not think anyone would object trying to unify the style, but there needs to be an agreement on what the style is. I CCd a few more folks who might have input to the style.
r- for now as at least the ChangeLog needs to be fixed, but more importantly I'd like to hear from other stakeholders before this lands.
> ChangeLog:3
> + [EFL] uniformed coding style in cmake files
If the intent is to change all files that are input to CMake you should change the prefix of the bug to [CMake].
We usually start the title with a capital letter;
"Unify coding style for CMake files" sounds better to me for a title.
> ChangeLog:8
> + Uniform coding style for .cmake files with rules:
What about other files that CMake takes as input - e.g. CMakeLists.txt ?
> ChangeLog:12
> + - 4-space as indent, no TAB
> + - Uppercase and no space when call macros: MESSAGE("testing")
> + - 1-space after condition clause: IF (), ELSE (), ENDIf(), FOREACH ()
> + - 1-space after macro definition: MACRO (), ENDMACRO ()
Where are all these rules coming from ? Are these custom for other projects as well ? Have you looked at other styles, like http://techbase.kde.org/Policies/CMake_Coding_Style (i just picked one, it does not mean that that is the one we should take as a base).
> ChangeLog:19
> + * Source/cmake/WebKitPackaging.cmake:
Again, what about CMakeLists.txt ?
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list