pkgsrc-Bugs archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: pkg/52922: devel/gmake: enable guile integration
The following reply was made to PR pkg/52922; it has been noted by GNATS.
From: Thomas Klausner <wiz%NetBSD.org@localhost>
To: NetBSD bugtracking <gnats-bugs%NetBSD.org@localhost>
Cc:
Subject: Re: pkg/52922: devel/gmake: enable guile integration
Date: Sat, 13 Jan 2018 01:15:43 +0100
On Fri, Jan 12, 2018 at 11:50:00PM +0000, vmagerya%gmail.com@localhost wrote:
> >Number: 52922
> >Category: pkg
> >Synopsis: devel/gmake: enable guile integration
> >Confidential: no
> >Severity: non-critical
> >Priority: medium
> >Responsible: pkg-manager
> >State: open
> >Class: change-request
> >Submitter-Id: net
> >Arrival-Date: Fri Jan 12 23:50:00 +0000 2018
> >Originator: Vitaly Magerya
> >Release:
> >Organization:
> >Environment:
> >Description:
> Recent releases of GNU make (since 4.0 I think) support Guile integration [1], but it's not enabled in devel/gmake. I'm attaching a patch that adds this support. Maybe it would be more appropriate to hide it under an option, but I don't have experience with pkgsrc to do that.
>
> [1] https://www.gnu.org/software/make/manual/html_node/Guile-Integration.html
> >How-To-Repeat:
>
> >Fix:
> RCS file: /cvsroot/pkgsrc/devel/gmake/Makefile,v
> retrieving revision 1.103
> diff -u -r1.103 Makefile
> --- Makefile 17 Dec 2016 14:14:06 -0000 1.103
> +++ Makefile 12 Jan 2018 23:34:21 -0000
> @@ -13,6 +13,7 @@
> LICENSE= gnu-gpl-v3
>
> USE_FEATURES= snprintf
> +USE_TOOLS= pkg-config
>
> GNU_CONFIGURE= yes
> TEXINFO_REQD+= 4.0
> @@ -40,4 +41,5 @@
> ${LN} -sf ${PREFIX}/bin/gmake ${DESTDIR}${PREFIX}/${PKGGNUDIR}/bin/make
> ${LN} -sf ${PREFIX}/${PKGMANDIR}/man1/gmake.1 ${DESTDIR}${PREFIX}/${PKGGNUDIR}${PKGMANDIR}/man1/make.1
>
> +.include "../../lang/guile20/buildlink3.mk"
> .include "../../mk/bsd.pkg.mk"
>
We can't enable that. guile20 itself uses gmake to build, so this
would introduce a cyclic dependency.
I'm not sure what the guile support does, but you'll need to create a
separate package for that.
Thomas
Home |
Main Index |
Thread Index |
Old Index