New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #28114 (closed update: fixed)

Opened 2 years ago

Last modified 2 years ago

rxvt-unicode: update to 9.10

Reported by: evangoldenberg@… Owned by: ryandesign@…
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch maintainer Cc: north@…
Port: rxvt-unicode

Description

Among the changes:

  • 256 color support is built in, so a patch is no longer necessary
  • Explicitly depends on xrender and fontconfig (fixes #28110)

Attachments

Portfile-rxvt-unicode.diff (1.3 KB) - added by evangoldenberg@… 2 years ago.
Portfile-rxvt-unicode-256color.diff (403 bytes) - added by evangoldenberg@… 2 years ago.

Change History

Changed 2 years ago by evangoldenberg@…

comment:1 Changed 2 years ago by macsforever2000@…

  • Keywords maintainer added
  • Version 1.9.2 deleted

comment:2 Changed 2 years ago by ryandesign@…

  • Owner changed from macports-tickets@… to ryandesign@…
  • Cc north@… added
  • Status changed from new to assigned

Thanks.

comment:3 Changed 2 years ago by ryandesign@…

  • Status changed from assigned to closed
  • Resolution set to fixed

comment:4 Changed 2 years ago by jmr@…

There doesn't seem to be anything related to xrender and fontconfig in the patch, did you forget to include it?

comment:5 Changed 2 years ago by evangoldenberg@…

As of 9.10, urxvt configure/make explicitly depends on xrender and fontconfig (via pkg-config) when Xft support is on (which it is in macports). It presumably never used to, and that is likely what caused #28110.

I haven't done too much digging to determine why linking against xrender and fontconfig broke in the first place, but the fact that the Urxvt 9.10 release notes mention it suggests it was an upstream issue.

In any case, urxvt seems to build without issue after this update.

comment:6 Changed 2 years ago by evangoldenberg@…

What I do appear to have forgotten, however, is to enable 256 colors. Apparently it is not enabled by the --enable-everything flag, despite having built in support.

New patch attached, which applies over the already committed one.

Changed 2 years ago by evangoldenberg@…

comment:7 Changed 2 years ago by evangoldenberg@…

  • Status changed from closed to reopened
  • Resolution fixed deleted

comment:8 Changed 2 years ago by ryandesign@…

  • Status changed from reopened to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.