Skip to content

fix: remove symbol visibility settings#37

Merged
deepin-bot[bot] merged 1 commit intolinuxdeepin:masterfrom
Johnson-zs:master
Jan 15, 2025
Merged

fix: remove symbol visibility settings#37
deepin-bot[bot] merged 1 commit intolinuxdeepin:masterfrom
Johnson-zs:master

Conversation

@Johnson-zs
Copy link
Contributor

  • Remove CMAKE_CXX_VISIBILITY_PRESET hidden
  • Remove CMAKE_VISIBILITY_INLINES_HIDDEN
  • Remove -fvisibility=hidden compile option

This change allows symbols to be visible by default in the shared library.

Log: tag 1.0.16

- Remove CMAKE_CXX_VISIBILITY_PRESET hidden
- Remove CMAKE_VISIBILITY_INLINES_HIDDEN
- Remove -fvisibility=hidden compile option

This change allows symbols to be visible by default in the shared library.

Log: tag 1.0.16
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Johnson-zs

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@github-actions
Copy link

TAG Bot

TAG: 1.0.16
EXISTED: no
DISTRIBUTION: unstable

@Johnson-zs
Copy link
Contributor Author

/forcemerge

@deepin-bot
Copy link
Contributor

deepin-bot bot commented Jan 15, 2025

This pr force merged! (status: blocked)

@deepin-bot deepin-bot bot merged commit 01c9aa3 into linuxdeepin:master Jan 15, 2025
14 checks passed
@Zeno-sole
Copy link
Contributor

/topic cd-20250211

@deepin-ci-robot
Copy link

Add topic: cd-20250211 successed.

@Zeno-sole
Copy link
Contributor

/integr-topic cd-20250211

@deepin-ci-robot
Copy link

Integrated with pr deepin-community/Repository-Integration#2569

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

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants