Index | Thread | Search

From:
Theo Buehler <tb@theobuehler.org>
Subject:
Re: fix lumina build on sparc64
To:
ports@openbsd.org
Date:
Tue, 27 Jan 2026 14:26:21 +0100

Download raw body.

Thread
On Tue, Jan 27, 2026 at 02:24:03PM +0100, Claudio Jeker wrote:
> On Tue, Jan 27, 2026 at 02:16:16PM +0100, Landry Breuil wrote:
> > Le Tue, Jan 27, 2026 at 02:02:54PM +0100, Theo Buehler a écrit :
> > > On Tue, Jan 27, 2026 at 01:51:55PM +0100, Claudio Jeker wrote:
> > > > x11/lumina does not build on sparc64 because the configure scripts use the
> > > > wrong compiler and fail. The problem is that the PATH variable does not
> > > > include the ports bin dir with the compiler links.
> > > > 
> > > > I came up with this diff which works for me.
> > > > I guess the pre-configure hunk is not needed since it only uses sed.
> > > > I'm a bit surprised that the PATH needs to be manually set like this.
> > > 
> > > I think it's because the custom do-configure doesn't include the
> > > default environment. See below.
> > > 
> > > > 
> > > > -- 
> > > > :wq Claudio
> > > > 
> > > > Index: Makefile
> > > > ===================================================================
> > > > RCS file: /cvs/ports/x11/lumina/Makefile,v
> > > > diff -u -p -r1.41 Makefile
> > > > --- Makefile	14 Mar 2025 15:31:28 -0000	1.41
> > > > +++ Makefile	27 Jan 2026 12:47:38 -0000
> > > > @@ -42,11 +42,13 @@ FAKE_FLAGS=		INSTALL_ROOT=${DESTDIR}/
> > > >  NO_TEST=		Yes
> > > >  
> > > >  pre-configure:
> > > > +	@PATH=${PORTPATH};			\
> > > >  	sed -i -e "s,!!LOCALBASE!!,${LOCALBASE}," \
> > > >  		-e "s,!!SYSCONFDIR!!,${SYSCONFDIR}," \
> > > >  		${WRKSRC}/src-qt5/OS-detect.pri
> > > >  
> > > >  do-configure:
> > > > +	@PATH=${PORTPATH};			\
> > > >  	cd ${WRKSRC} && ${LOCALBASE}/bin/qmake-qt5 \
> > > 
> > > It'll take some time for me to test this, but I'd expect this to work:
> > > 
> > >   	cd ${WRKSRC} && env -i ${MAKE_ENV} ${LOCALBASE}/bin/qmake-qt5 \
> > 
> > yeah that's the more correct variant, or even ${SETENV} ${MAKE_ENV}
>  
> So the diff below also works for me.

Great. I would bump revision to be on the safe side.

ok tb

> 
> -- 
> :wq Claudio
> 
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/x11/lumina/Makefile,v
> diff -u -p -r1.41 Makefile
> --- Makefile	14 Mar 2025 15:31:28 -0000	1.41
> +++ Makefile	27 Jan 2026 13:22:20 -0000
> @@ -47,7 +47,7 @@ pre-configure:
>  		${WRKSRC}/src-qt5/OS-detect.pri
>  
>  do-configure:
> -	cd ${WRKSRC} && ${LOCALBASE}/bin/qmake-qt5 \
> +	cd ${WRKSRC} && ${SETENV} ${MAKE_ENV} ${LOCALBASE}/bin/qmake-qt5 \
>  		CONFIG+="configure WITH_I18N NO_XINPUT"
>  
>  post-install:
>