[Lldb-commits] [PATCH] D54476: [CMake] Streamline code signing for debugserver

Jonas Devlieghere via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Nov 13 12:39:05 PST 2018

JDevlieghere added inline comments.

Comment at: tools/debugserver/source/CMakeLists.txt:118
+      OUTPUT_VARIABLE xcode_dev_dir)
+    string(STRIP ${xcode_dev_dir} xcode_dev_dir)
sgraenitz wrote:
> JDevlieghere wrote:
> > Why did you make this variable name lowercase? Does that have any semantic meaning?
> LLVM tends to use lowercase names for local/non-cached variables, e.g. `update_src_props` in:
> https://github.com/llvm-mirror/llvm/blob/master/cmake/modules/AddLLVM.cmake
> I didn't find any guidelines on conventions for CMake, but I thought it's a good idea. It's a little surprising to mix in such changes to patches that also have semantic changes. Maybe I should continue to use uppercase names for now and add a subsequent patch (with a proper title) that does the change for the whole file. What do you think?
Cool, I wasn't aware of the convention. I think either is fine, whatever you prefer. 


More information about the lldb-commits mailing list