Tuesday, May 31, 2022

Re: UPDATE: emulators/retroarch

Ryan Freeman writes:
> On Fri, May 13, 2022 at 06:43:18PM -0700, Ryan Freeman wrote:
> > - RUN_DEPENDS on p5-Net-DBus and p5-X11-Protocol added, solves:
> > Can't locate X11/Protocol.pm in @INC (you may need to install the X11::Prot
> ocol module) (@INC contains: /usr/local/libdata/perl5/site_perl/amd64-openbsd
> /usr/local/libdata/perl5/site_perl /usr/libdata/perl5/amd64-openbsd /usr/lib
> data/perl5) at -e line 6.
> > BEGIN failed--compilation aborted at -e line 6.

I couldn't reproduce these errors, and as far as I know retroarch doesn't
use Perl at all during build or runtime, so I removed these. (If they
turn out to be necessary we can add them back.)

> > - CONFIGURE_ENV relied on {INCLUDE,LIBRARY}_DIRS, upstream now hard-sets these
> > to ''. Switching CONFIGURE_ENV to CFLAGS/CXXFLAGS/LDFLAGS worked.

I also modified these to append the user's CFLAGS/CXXFLAGS.

Everything else looked good to me. It's committed now. Thanks for
working on this!

--
Anthony J. Bentley

No comments:

Post a Comment