Re: svn commit: r366634 - head/contrib/nvi/common

2020-10-12 Thread Alexander Richardson
On Mon, 12 Oct 2020 at 14:55, Jessica Clarke  wrote:
>
> On 12 Oct 2020, at 11:42, Alex Richardson  wrote:
> > --- head/contrib/nvi/common/common.h  Mon Oct 12 10:42:19 2020
> > (r366633)
> > +++ head/contrib/nvi/common/common.h  Mon Oct 12 10:42:24 2020
> > (r366634)
> > @@ -14,7 +14,7 @@
> > #ifdef __linux__
> > #include "/usr/include/db1/db.h"  /* Only include db1. */
> > #else
> > -#include "/usr/include/db.h" /* Only include db1. */
> > +#include   /* Only include db1. */
> > #endif
>
> Can this not be expressed more nicely as the following?
>
> /* Only include db1 */
> #if __has_include()
> #include 
> #else
> #include 
> #endif
>
> Jess
>

Yes it could, but I'd prefer it if upstream fixes it instead so we can
drop this diff. See also https://github.com/lichray/nvi2/issues/69.
___
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"


Re: svn commit: r366634 - head/contrib/nvi/common

2020-10-12 Thread Jessica Clarke
On 12 Oct 2020, at 11:42, Alex Richardson  wrote:
> --- head/contrib/nvi/common/common.h  Mon Oct 12 10:42:19 2020
> (r366633)
> +++ head/contrib/nvi/common/common.h  Mon Oct 12 10:42:24 2020
> (r366634)
> @@ -14,7 +14,7 @@
> #ifdef __linux__
> #include "/usr/include/db1/db.h"  /* Only include db1. */
> #else
> -#include "/usr/include/db.h" /* Only include db1. */
> +#include   /* Only include db1. */
> #endif

Can this not be expressed more nicely as the following?

/* Only include db1 */
#if __has_include()
#include 
#else
#include 
#endif

Jess

___
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"


svn commit: r366634 - head/contrib/nvi/common

2020-10-12 Thread Alex Richardson
Author: arichardson
Date: Mon Oct 12 10:42:24 2020
New Revision: 366634
URL: https://svnweb.freebsd.org/changeset/base/366634

Log:
  Fix buildworld on Linux/macOS after nvi update
  
  This re-applies r365941 which was lost in the nvi update.

Modified:
  head/contrib/nvi/common/common.h

Modified: head/contrib/nvi/common/common.h
==
--- head/contrib/nvi/common/common.hMon Oct 12 10:42:19 2020
(r366633)
+++ head/contrib/nvi/common/common.hMon Oct 12 10:42:24 2020
(r366634)
@@ -14,7 +14,7 @@
 #ifdef __linux__
 #include "/usr/include/db1/db.h"   /* Only include db1. */
 #else
-#include "/usr/include/db.h"   /* Only include db1. */
+#include /* Only include db1. */
 #endif
 #include  /* May refer to the bundled regex. */
 
___
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"