[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#14068: [PATCH] Officially support DragonFly BSD
From: |
John Marino |
Subject: |
bug#14068: [PATCH] Officially support DragonFly BSD |
Date: |
Thu, 18 Apr 2013 09:59:47 +0200 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:10.0) Gecko/20120129 Thunderbird/10.0 |
On 4/18/2013 07:46, Glenn Morris wrote:
John Marino wrote:
The previously submitted patch was for release 24.3.
The patch to add DragonFly support to version 24.3.50.112178 is much
smaller:
https://raw.github.com/jrmarino/DPorts/staged/editors/emacs-devel/dragonfly/patch-configure.ac
+ case "${canonical}" in
+ i[3456]86-*-dragonfly*) machine=intel386 ;;
+ amd64-*-dragonfly*|x86_64-*-dragonfly*) machine=amdx86-64 ;;
+ esac
machine is no longer used, so these lines are unnecessary.
Okay, good to know!
- freebsd) LIBS_SYSTEM="-lutil" ;;
+ freebsd|netbsd|openbsd|dragonfly) LIBS_SYSTEM="-lutil" ;;
Why are you adding netbsd and openbsd as well?
It was likely left over from years. The submitted patch came from
pkgsrc and freebsd ports. It is appropriate to only add DragonFly (as
you have done) and let representatives from the other BSDs speak for
themselves.
Apart from that, it seems fine to me.
It's probably just small enough not to need a copyright assignment; but
for the ChangeLog, can you tell me who the author is?
I think it's reasonable to call me (John Marino) the primary author. I
basically had to create it from scratch -- most of this information was
in those "S" files and had to be moved to the configure script, so it's
basically new.
Revised patch against current trunk:
With a quick glance, it looks good to me.
Thanks!
John