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
prev parent 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).