#30 Merged
Repository
ariscop ariscop
Branch
default
Repository
jacmoe jacmoe
Branch
default

fix build on archlinux

Author
  1. Andrew Cook avatarAndrew Cook
Reviewers
Description

${CMAKE_EXE_LINKER_FLAGS} doesn't have a trailing space on my install, not sure if its supposed to or not. either way, this fixes

Comments (6)

    1. Andrew Fenn

      Didn't we remove the spaces before because of some cmake linker error? I'm sure I committed a fix for that a few months back however I've forgotten everything about it. I'm not at a computer to test this right now but I just wanted to make you guys aware of this.

      1. Jacob Moen repo owner

        I believe you're right - it's a mystery why it fails on Arch Linux.. If the problem is specific to the setup Andrew Cook is using, then we need to pinpoint the problem so that we can either work around the issue, or at least be able to solve it somehow.

      2. Jacob Moen repo owner

        I've just had a run through the history of the main CMakeLists.txt, and can't find any fix related to this. Except that you added the '-Wl' flag without initial space. Let's see what our compiler says..

Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for ProjectModifiedEvent.java.
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.