all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* ncurses conflicts between master and core-updates
@ 2017-01-06  5:19 Leo Famulari
  2017-01-06 13:18 ` Ludovic Courtès
  0 siblings, 1 reply; 4+ messages in thread
From: Leo Famulari @ 2017-01-06  5:19 UTC (permalink / raw)
  To: guix-devel

[-- Attachment #1: Type: text/plain, Size: 569 bytes --]

There's a tricky conflict in the ncurses package definition between
master and core-updates.

On core-updates, there is 57742b35dc026 (gnu: ncurses: Install
pkg-config ‘.pc’ files.), and on master, it's d230cf61f6a (gnu: ncurses:
support mingw.).

The mingw commit added some conditionals relating to mingw, and I'm not
sure if the pkg-config change should be applied for mingw builds or not.

Can you give me advice about how to resolve this, or merge master into
core-updates yourself?  There are a few other conflicts, but they are
easy to resolve.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: ncurses conflicts between master and core-updates
  2017-01-06  5:19 ncurses conflicts between master and core-updates Leo Famulari
@ 2017-01-06 13:18 ` Ludovic Courtès
  2017-01-06 20:33   ` Jan Nieuwenhuizen
  0 siblings, 1 reply; 4+ messages in thread
From: Ludovic Courtès @ 2017-01-06 13:18 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

Hello!

(Cc: Jan.)

Leo Famulari <leo@famulari.name> skribis:

> There's a tricky conflict in the ncurses package definition between
> master and core-updates.
>
> On core-updates, there is 57742b35dc026 (gnu: ncurses: Install
> pkg-config ‘.pc’ files.), and on master, it's d230cf61f6a (gnu: ncurses:
> support mingw.).
>
> The mingw commit added some conditionals relating to mingw, and I'm not
> sure if the pkg-config change should be applied for mingw builds or not.

I think it can’t hurt MinGW cross-builds to keep the pkg-config change.

> Can you give me advice about how to resolve this, or merge master into
> core-updates yourself?  There are a few other conflicts, but they are
> easy to resolve.

Your question was probably for Jan, no?

Ludo’.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: ncurses conflicts between master and core-updates
  2017-01-06 13:18 ` Ludovic Courtès
@ 2017-01-06 20:33   ` Jan Nieuwenhuizen
  2017-01-06 22:22     ` Leo Famulari
  0 siblings, 1 reply; 4+ messages in thread
From: Jan Nieuwenhuizen @ 2017-01-06 20:33 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

Ludovic Courtès writes:

>> There's a tricky conflict in the ncurses package definition between
>> master and core-updates.
>>
>> On core-updates, there is 57742b35dc026 (gnu: ncurses: Install
>> pkg-config ‘.pc’ files.), and on master, it's d230cf61f6a (gnu: ncurses:
>> support mingw.).
>>
>> The mingw commit added some conditionals relating to mingw, and I'm not
>> sure if the pkg-config change should be applied for mingw builds or not.

> I think it can’t hurt MinGW cross-builds to keep the pkg-config change.

Yes.

> Your question was probably for Jan, no?

I started doing a manual merge this morning and then forgot.  I pushed
a manual merge of both commits to

    f55d1eff65ea -- https://gitlab.com/janneke/guix/tree/master

and tested it builds mingw ncurses and mingw guile

    ./pre-inst-env guix build --keep-failed --target=i686-w64-mingw32 ncurses
    ./pre-inst-env guix build --keep-failed --target=i686-w64-mingw32 guile

HTH -- Greetings,
Jan

-- 
Jan Nieuwenhuizen <janneke@gnu.org> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar®  http://AvatarAcademy.nl  

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: ncurses conflicts between master and core-updates
  2017-01-06 20:33   ` Jan Nieuwenhuizen
@ 2017-01-06 22:22     ` Leo Famulari
  0 siblings, 0 replies; 4+ messages in thread
From: Leo Famulari @ 2017-01-06 22:22 UTC (permalink / raw)
  To: Jan Nieuwenhuizen; +Cc: guix-devel

On Fri, Jan 06, 2017 at 09:33:50PM +0100, Jan Nieuwenhuizen wrote:
> Ludovic Courtès writes:
> 
> >> There's a tricky conflict in the ncurses package definition between
> >> master and core-updates.
> >>
> >> On core-updates, there is 57742b35dc026 (gnu: ncurses: Install
> >> pkg-config ‘.pc’ files.), and on master, it's d230cf61f6a (gnu: ncurses:
> >> support mingw.).
> >>
> >> The mingw commit added some conditionals relating to mingw, and I'm not
> >> sure if the pkg-config change should be applied for mingw builds or not.
> 
> > I think it can’t hurt MinGW cross-builds to keep the pkg-config change.
> 
> Yes.
> 
> > Your question was probably for Jan, no?
> 
> I started doing a manual merge this morning and then forgot.  I pushed
> a manual merge of both commits to
> 
>     f55d1eff65ea -- https://gitlab.com/janneke/guix/tree/master

Thanks Jan! I used this resolve the conflict, and I updated the
core-updates branch.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-01-06 22:23 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-01-06  5:19 ncurses conflicts between master and core-updates Leo Famulari
2017-01-06 13:18 ` Ludovic Courtès
2017-01-06 20:33   ` Jan Nieuwenhuizen
2017-01-06 22:22     ` Leo Famulari

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.