Skip to content

[libheif] Fix usage#22818

Merged
ras0219-msft merged 4 commits intomicrosoft:masterfrom
JackBoosY:dev/jack/13354
Jan 28, 2022
Merged

[libheif] Fix usage#22818
ras0219-msft merged 4 commits intomicrosoft:masterfrom
JackBoosY:dev/jack/13354

Conversation

@JackBoosY
Copy link
Copy Markdown
Contributor

@JackBoosY JackBoosY commented Jan 27, 2022 •
edited
Loading

Fix definiton LIBHEIF_API value.

#if defined(_MSC_VER) && !defined(LIBHEIF_STATIC_BUILD)
#ifdef LIBHEIF_EXPORTS
#define LIBHEIF_API __declspec(dllexport)
#else
#define LIBHEIF_API __declspec(dllimport)
#endif
#elif defined(HAVE_VISIBILITY) && HAVE_VISIBILITY
#ifdef LIBHEIF_EXPORTS
#define LIBHEIF_API __attribute__((__visibility__("default")))
#else
#define LIBHEIF_API
#endif
#else
#define LIBHEIF_API
#endif

Fixes #13354 #18107.

@JackBoosY JackBoosY added category:port-bug The issue is with a library, which is something the port should already support info:internal labels Jan 27, 2022
@JackBoosY JackBoosY mentioned this pull request Jan 27, 2022
Closed
github-actions[bot]
github-actions bot approved these changes Jan 27, 2022
View reviewed changes
Copy link
Copy Markdown

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You have modified or added at least one vcpkg.json where a "license" field is missing.
If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/libheif/vcpkg.json

Valid values for the license field are listed at https://spdx.org/licenses/

@JackBoosY JackBoosY mentioned this pull request Jan 27, 2022
Closed
@JackBoosY JackBoosY linked an issue Jan 27, 2022 that may be closed by this pull request
[libheif] Fix static linking #18107
Closed
@LilyWangLL
Copy link
Copy Markdown
Contributor

LilyWangLL commented Jan 27, 2022 •
edited
Loading

Could you add license in vcpkg.json for libheif?

@JackBoosY JackBoosY requested a review from LilyWangLL January 27, 2022 16:23
@LilyWangLL LilyWangLL added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Jan 28, 2022
@ras0219-msft ras0219-msft merged commit 8e7e20c into microsoft:master Jan 28, 2022
@ras0219-msft
Copy link
Copy Markdown
Contributor

ras0219-msft commented Jan 28, 2022

This LGTM, thanks!

@JackBoosY JackBoosY deleted the dev/jack/13354 branch January 29, 2022 04:40
@saucecontrol saucecontrol mentioned this pull request Mar 10, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@github-actions github-actions[bot] github-actions[bot] approved these changes

+1 more reviewer

@LilyWangLL LilyWangLL LilyWangLL approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@LilyWangLL LilyWangLL

Labels

category:port-bug The issue is with a library, which is something the port should already support info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[libheif] Fix static linking [openimageio[opencolorio] / libheif] Linking against OpenImageIO failed

3 participants

@JackBoosY @LilyWangLL @ras0219-msft