Skip to content

make: made consistent use of spaces in Makefile.dep#2626

Merged
haukepetersen merged 1 commit intoRIOT-OS:masterfrom
haukepetersen:fix_makedep_tabs
Mar 18, 2015
Merged

make: made consistent use of spaces in Makefile.dep#2626
haukepetersen merged 1 commit intoRIOT-OS:masterfrom
haukepetersen:fix_makedep_tabs

Conversation

@haukepetersen
Copy link
Copy Markdown
Contributor

seems like some spaces slipped into the Makefile.dep which contains only tabs for indention otherwise...

@haukepetersen haukepetersen added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation labels Mar 17, 2015
@miri64
Copy link
Copy Markdown
Member

miri64 commented Mar 17, 2015

This is on purpose (at least for my part). Tabs and spaces are interpreted differently by make: tabs are interpreted as “the following line contains a command for a target” (which these lines don't carry) [Source], while spaces don't have any syntactic connotations. Though the manual states that it is in these cases arbitrary, using tabs for this kind of indentations already lead to problems in the past which is why I try to not use them for simple eye-candy indentatitions between ifneq() macros e.g. anymore.

@miri64
Copy link
Copy Markdown
Member

miri64 commented Mar 17, 2015

Never had the time to fix this for all lines though :-/

@haukepetersen
Copy link
Copy Markdown
Contributor Author

ok, then let me change the (all) tabs to spaces in this file...

@haukepetersen haukepetersen force-pushed the fix_makedep_tabs branch 2 times, most recently from 95cbf67 to 8a8999f Compare March 17, 2015 16:53
@haukepetersen haukepetersen changed the title make: some spaces are now tabs in Makefile.dep make: made consistent use spaces in Makefile.dep Mar 17, 2015
@haukepetersen haukepetersen changed the title make: made consistent use spaces in Makefile.dep make: made consistent use of spaces in Makefile.dep Mar 17, 2015
@haukepetersen
Copy link
Copy Markdown
Contributor Author

now there is a consistent use of 2 space indention in Makefile.dep. Also changed the PR title accordingly.

@miri64
Copy link
Copy Markdown
Member

miri64 commented Mar 17, 2015

ACK when travis is happy.

@jnohlgard
Copy link
Copy Markdown
Member

The commit message is missing a letter. s/ake/make/

@haukepetersen
Copy link
Copy Markdown
Contributor Author

ups, fixed it.

@haukepetersen
Copy link
Copy Markdown
Contributor Author

and go.

haukepetersen added a commit that referenced this pull request Mar 18, 2015
make: made consistent use of spaces in Makefile.dep
@haukepetersen haukepetersen merged commit 0e8df14 into RIOT-OS:master Mar 18, 2015
@haukepetersen haukepetersen deleted the fix_makedep_tabs branch March 18, 2015 14:19
haukepetersen added a commit to haukepetersen/RIOT that referenced this pull request Mar 25, 2015
Though technically it does not make a difference here, tabs have
a different semantic meaning than spaces in Makefiles. To be consistent
in RIOT we agreed to use two spaces for if conditions -> see RIOT-OS#2626.
muthmano-dev pushed a commit to muthmano-dev/RIOT that referenced this pull request Mar 29, 2015
Though technically it does not make a difference here, tabs have
a different semantic meaning than spaces in Makefiles. To be consistent
in RIOT we agreed to use two spaces for if conditions -> see RIOT-OS#2626.
muthmano-dev pushed a commit to muthmano-dev/RIOT that referenced this pull request Mar 30, 2015
Though technically it does not make a difference here, tabs have
a different semantic meaning than spaces in Makefiles. To be consistent
in RIOT we agreed to use two spaces for if conditions -> see RIOT-OS#2626.
haukepetersen added a commit that referenced this pull request Mar 31, 2015
Though technically it does not make a difference here, tabs have
a different semantic meaning than spaces in Makefiles. To be consistent
in RIOT we agreed to use two spaces for if conditions -> see #2626.
kushalsingh007 pushed a commit to kushalsingh007/RIOT that referenced this pull request Apr 12, 2015
Though technically it does not make a difference here, tabs have
a different semantic meaning than spaces in Makefiles. To be consistent
in RIOT we agreed to use two spaces for if conditions -> see RIOT-OS#2626.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants