Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • cerbero cerbero
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 101
    • Issues 101
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 35
    • Merge requests 35
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GStreamerGStreamer
  • cerberocerbero
  • Merge requests
  • !31

ndk-build: don't advertise symbols from non-explicitly linked libraries

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Matthew Waters requested to merge ystreet/cerbero:android-exclude-libs into master Nov 29, 2018
  • Overview 3
  • Commits 1
  • Pipelines 0
  • Changes 2

Instead pass -Wl,exclude-libs,$lib for each library NOT mentioned explicitly on the command line so those symbols aren't placed in exported symbol tables. Fixes numerous linking errors in dependant libraries that are extremely likely to be unused directly.

This does however mean that any symbols and libraries that are used need to be present in the command line explicitly using either GSTREAMER_EXTRA_DEPS (for pkg-config) or -l$lib arguments.

Fixes #52 (closed) Fixes gst-libav#37 (closed)

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: android-exclude-libs