Skip to content

Use set in dependency string in meta translator - #4325

Merged
vigsterkr merged 1 commit into
shogun-toolbox:developfrom
vinx13:meta/unique_include
Jun 4, 2018
Merged

vigsterkr merged 1 commit into
shogun-toolbox:developfrom
vinx13:meta/unique_include

Conversation

@vinx13

@vinx13 vinx13 commented Jun 3, 2018

Copy link
Copy Markdown
Member

fix #4308

@vigsterkr

Copy link
Copy Markdown
Member

👍

@vigsterkr
vigsterkr merged commit cd08c4e into shogun-toolbox:develop Jun 4, 2018
dependencies = list(dependencies)

translations = list(map(self.translateDependencyElement, dependencies))
translations = list(set(map(self.translateDependencyElement, dependencies)))

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

when generating includes/imports of modules use a set to do that

3 participants