[Lldb-commits] [lldb] Fix MAKE_DWP on Linux: set DSYM so the DWP packaging step runs (PR #198240)

[Greg Clayton via lldb-commits]

================

----------------
clayborg wrote:

>From AI about Makefile indentation:

You should not indent ifeq, ifneq, ifdef, or ifndef statements with a tab. 

In a Makefile, lines starting with a tab are interpreted as shell commands (part of a recipe). Since these conditional directives are part of the Make syntax itself and not shell commands, indenting them with a tab will cause Make to pass them to the shell, resulting in a syntax error. 

Rules for Indenting Conditionals
- Avoid Tabs: Never start a Make conditional directive (ifeq, else, endif) with a tab.
- Optional Spaces: You may use spaces for logical indentation to make nested conditionals more readable.
- Global Context: Conditionals are often written starting at the very beginning of the line (column zero) to avoid confusion.
- Inside Recipes: If you use a conditional inside a target's recipe, it must still follow Make's syntax. Most developers avoid this by using shell-level if statements instead, or by ensuring the Make conditional is not tab-indented. 


https://github.com/llvm/llvm-project/pull/198240