unofficial mirror of bug-guile@gnu.org 
 help / color / mirror / Atom feed
From: Thomas Klausner <tk@giga.or.at>
To: Andy Wingo <wingo@pobox.com>
Cc: 25921-done@debbugs.gnu.org
Subject: bug#25921: pkgsrc patches: Dragonfly support
Date: Wed, 15 Mar 2017 19:04:45 +0100	[thread overview]
Message-ID: <20170315180445.vl4dktxx6mtvpbjt@danbala> (raw)
In-Reply-To: <87pohk85ex.fsf@pobox.com>

On Tue, Mar 14, 2017 at 12:27:18PM +0100, Andy Wingo wrote:
> On Wed 01 Mar 2017 21:27, Thomas Klausner <tk@giga.or.at> writes:
> 
> > $NetBSD: patch-lib_signal.in.h,v 1.1 2016/12/03 03:15:33 marino Exp $
> >
> > DragonFly support.
> >
> > --- lib/signal.in.h.orig	2016-06-29 09:12:27 UTC
> > +++ lib/signal.in.h
> > @@ -61,6 +61,7 @@
> >  #if (@GNULIB_PTHREAD_SIGMASK@ || defined GNULIB_POSIXCHECK) \
> >      && ((defined __APPLE__ && defined __MACH__) \
> >          || defined __FreeBSD__ || defined __OpenBSD__ || defined __osf__ \
> > +        || defined __DragonFly__ \
> >          || defined __sun || defined __ANDROID__) \
> >      && ! defined __GLIBC__
> >  # include <pthread.h>
> >
> 
> This file comes from gnulib.  Can you send this patch there?
> bug-gnulib@gnu.org.  That way you will fix it once and reap the benefits
> many places.

I sent this upstream and learned that it's probably unnecessary, so
I've removed it from pkgsrc.

> > $NetBSD: patch-libguile_threads.c,v 1.1 2016/12/03 03:15:33 marino Exp $
> >
> > DragonFly support.
> >
> > --- libguile/threads.c.orig	2016-06-20 20:35:06 UTC
> > +++ libguile/threads.c
> > @@ -2252,7 +2252,7 @@ scm_ia64_ar_bsp (const void *opaque)
> >    return (void *) ctx->uc_mcontext.sc_ar_bsp;
> >  }
> >  # endif /* linux */
> > -# ifdef __FreeBSD__
> > +# if defined __FreeBSD__ || defined __DragonFly__
> >  #  include <ucontext.h>
> >  void *
> 
> This is for ia64 support which AFAIU DragonFly doesn't do; I think you
> can drop this one.

Ok, removed too.

Thanks for the feedback!
 Thomas





      reply	other threads:[~2017-03-15 18:04 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-01 20:27 bug#25921: pkgsrc patches: Dragonfly support Thomas Klausner
2017-03-14 11:27 ` Andy Wingo
2017-03-15 18:04   ` Thomas Klausner [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/guile/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170315180445.vl4dktxx6mtvpbjt@danbala \
    --to=tk@giga.or.at \
    --cc=25921-done@debbugs.gnu.org \
    --cc=wingo@pobox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).