Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Search libgdiplus in macports install location #47373

Merged
merged 1 commit into from
Jan 26, 2021
Merged

Search libgdiplus in macports install location #47373

merged 1 commit into from
Jan 26, 2021

Conversation

am11
Copy link
Member

@am11 am11 commented Jan 23, 2021

When libgdiplus is installed via macports, System.TypeInitializationException exception is thrown.
This is because macports installs the library in /opt/local/lib.

$ port install libgdiplus
$ port contents libgdiplus
Port libgdiplus contains:
  /opt/local/lib/libgdiplus.0.dylib
  /opt/local/lib/libgdiplus.a
  /opt/local/lib/libgdiplus.dylib
  /opt/local/lib/pkgconfig/libgdiplus.pc
  /opt/local/share/doc/libgdiplus/AUTHORS
  /opt/local/share/doc/libgdiplus/COPYING
  /opt/local/share/doc/libgdiplus/ChangeLog
  /opt/local/share/doc/libgdiplus/LICENSE
  /opt/local/share/doc/libgdiplus/NEWS
  /opt/local/share/doc/libgdiplus/README.md
  /opt/local/share/doc/libgdiplus/TODO

PR adds /opt/local/lib/libgdiplus.dylib as a well-known probing path.

@ghost
Copy link

ghost commented Jan 23, 2021

Tagging subscribers to this area: @safern, @tannergooding
See info in area-owners.md if you want to be subscribed.

Issue Details

When libgdiplus is installed via macports, System.TypeInitializationException exception is thrown.
This is because macports installs the library in /opt/local/lib.

$ port install libgdiplus
$ port contents libgdiplus
Port libgdiplus contains:
  /opt/local/lib/libgdiplus.0.dylib
  /opt/local/lib/libgdiplus.a
  /opt/local/lib/libgdiplus.dylib
  /opt/local/lib/pkgconfig/libgdiplus.pc
  /opt/local/share/doc/libgdiplus/AUTHORS
  /opt/local/share/doc/libgdiplus/COPYING
  /opt/local/share/doc/libgdiplus/ChangeLog
  /opt/local/share/doc/libgdiplus/LICENSE
  /opt/local/share/doc/libgdiplus/NEWS
  /opt/local/share/doc/libgdiplus/README.md
  /opt/local/share/doc/libgdiplus/TODO

PR adds /opt/local/lib/libgdiplus.dylib as a well-known probing path.

Author: am11
Assignees: -
Labels:

area-System.Drawing

Milestone: -

@danmoseley
Copy link
Member

danmoseley commented Jan 23, 2021

@akoeplinger I'm not familiar with macports. Is this still a trusted libgdiplus channel? If so this change seems reasonable.

@akoeplinger
Copy link
Member

It's as trusted as homebrew so yeah this seems fine :)

@ghost
Copy link

ghost commented Jan 25, 2021

Hello @akoeplinger!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@am11
Copy link
Member Author

am11 commented Jan 26, 2021

x86 failure is #47374.

@akoeplinger akoeplinger merged commit 687b2a9 into dotnet:master Jan 26, 2021
@am11 am11 deleted the feature/libgdiplus-dllimport-search-path branch January 26, 2021 14:03
@ghost ghost locked as resolved and limited conversation to collaborators Feb 25, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants