Skip to content

fix: CMake example commands in README.md - #29

Merged
lava merged 1 commit into
lava:masterfrom
amintahmasbi:patch-1
May 29, 2017
Merged

fix: CMake example commands in README.md#29
lava merged 1 commit into
lava:masterfrom
amintahmasbi:patch-1

Conversation

@amintahmasbi

Copy link
Copy Markdown

The format of target_include_directories has changed in the recent CMake versions (3.5 and higher)
'target_link_libraries' had a typo
Minor fix: "apt-get" is generally preferred over aptitude for Ubuntu

Thank you!

The format of `target_include_directories` has changed in the recent CMake versions (3.5 and higher)
'target_link_libraries' had a typo
Minor fix: "apt-get" is generally preferred over aptitude for Ubuntu
 
Thank you!
@lava

lava commented May 29, 2017

Copy link
Copy Markdown
Owner

I think nowadays, "apt" is actually the preferred version ;)

Anyways, thanks for the contribution!

@lava
lava merged commit c9b777a into lava:master May 29, 2017
@amintahmasbi
amintahmasbi deleted the patch-1 branch May 30, 2017 15:11
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.

2 participants