Skip to content

Fix Makefile syntax#130

Merged
chinmayshah99 merged 1 commit intoOpenMined:devfrom
replomancer:makefilefix
May 6, 2020
Merged

Fix Makefile syntax#130
chinmayshah99 merged 1 commit intoOpenMined:devfrom
replomancer:makefilefix

Conversation

@replomancer
Copy link
Copy Markdown
Member

Description

Here's a small fix of a syntax error in the Makefile.

Running make always produces this warning:

Makefile:20: extraneous text after 'ifeq' directive

Type of change

Please mark options that are relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Run make help without this fix to see:

Makefile:20: extraneous text after 'ifeq' directive

as the first line of output.

With this fix the warning goes away 🧹

@chinmayshah99 chinmayshah99 merged commit a065a8a into OpenMined:dev May 6, 2020
@chinmayshah99 chinmayshah99 added the Type: Bug 🐛 Some functionality not working in the codebase as intended label May 6, 2020
dvadym pushed a commit to dvadym/PyDP that referenced this pull request Jul 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Bug 🐛 Some functionality not working in the codebase as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants