Skip to content
This repository was archived by the owner on May 3, 2019. It is now read-only.

Link everything statically#1

Merged
ColinDuquesnoy merged 8 commits intomasterfrom
static_linking
Jan 14, 2015
Merged

Link everything statically#1
ColinDuquesnoy merged 8 commits intomasterfrom
static_linking

Conversation

@ColinDuquesnoy
Copy link
Owner

This PR change the way the application links with the plugins and the intermediary library (libmellowplayer). Everything will now get linked statically

There are a few advantages to use static linking over dynamic linking:

  • it's infinitely easier to deploy (especially on mac and windows)
  • it's also easier to document, there is one single procedure for adding a plugin and contributing to the project.
  • there is no need to package libraries, headers, pkgconfigs,...
  • this ensure a review of the plugin by the upstream team before a plugin is released in the wild,...

Basically the idea is to keep the system as simple as possible for the developers (following the KISS principles).

@ColinDuquesnoy ColinDuquesnoy added this to the 1.0 milestone Jan 14, 2015
@ColinDuquesnoy ColinDuquesnoy self-assigned this Jan 14, 2015
ColinDuquesnoy added a commit that referenced this pull request Jan 14, 2015
@ColinDuquesnoy ColinDuquesnoy merged commit 3997d19 into master Jan 14, 2015
@ColinDuquesnoy ColinDuquesnoy deleted the static_linking branch January 14, 2015 23:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant