Page MenuHomeFreeBSD

Add STATIC option to x11-fonts/fontconfig
ClosedPublic

Authored by cy on Apr 30 2016, 5:22 AM.
Tags
None
Referenced Files
F109684290: D6155.id15755.diff
Sat, Feb 8, 8:02 AM
Unknown Object (File)
Jan 4 2025, 9:32 AM
Unknown Object (File)
Nov 6 2024, 10:57 PM
Unknown Object (File)
Oct 9 2024, 9:52 PM
Unknown Object (File)
Sep 20 2024, 6:59 AM
Unknown Object (File)
Sep 5 2024, 2:33 PM
Unknown Object (File)
Sep 5 2024, 1:25 PM
Unknown Object (File)
Sep 4 2024, 11:44 AM

Details

Summary

I h ave a requirement to statically link x11/xforward. Unfortunately x11-fonts/fontconfig does not install static (.a) libraries. The patch in this differential revision adds the option to install a static libfontconfig.a library.

Test Plan

A new x11/xforward-static port has been created locally and tested using a locally updated x11/fontconfig port with ENABLE_STATIC on my firewall/gateway, which does not have X installed. The test has passed.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

cy retitled this revision from to Add ENABLE_STATIC option to x11-fonts/fontconfig.
cy updated this object.
cy edited the test plan for this revision. (Show Details)
cy added reviewers: marcus, cem, wblock.
cy set the repository for this revision to rP FreeBSD ports repository.
cy added a subscriber: gnome.
kwm added inline comments.
x11-fonts/fontconfig/Makefile
43 ↗(On Diff #15755)

Could you rename the option to STATIC? OPTIONS already implies enable/disable.

44 ↗(On Diff #15755)

When renamed this lines can be removed, as the STATIC define is in bsd.options.desc.mk.

45 ↗(On Diff #15755)

STATIC_CONFIGURE_ENABLE=static

cy retitled this revision from Add ENABLE_STATIC option to x11-fonts/fontconfig to Add STATIC option to x11-fonts/fontconfig.
cy marked an inline comment as done.

Changed ENABLE_STATIC to STATIC.

cy added a subscriber: x11.

STATIC_CONFIGURE_ENABLE= also set

Do we really need an option? Can't the port always install the static library?

Do we really need an option? Can't the port always install the static library?

/me 100% agrees

In D6155#131419, @bapt wrote:

Do we really need an option? Can't the port always install the static library?

/me 100% agrees

I also agree with this. I see no need to have a separate option for this.
Other than that, I see no problem in approving this.

If people are happy with this, I'll commit it and move onto xforward.

This revision is now accepted and ready to land.May 3 2016, 6:32 AM

Yeah this is fine too. Could you add a comment though why static is enabled? Otherwise someone might remove it in a future updated (probably me..).

This revision was automatically updated to reflect the committed changes.