devel/openmpi Can't link example program, symbols local in libmpi.so.0.1?

[Thomas L.]

Hi,

I tried compiling a MPI example program 
(https://raw.githubusercontent.com/wesleykendall/mpitutorial/gh-pages/tutorials/mpi-hello-world/code/mpi_hello_world.c)
 with openmpi on -current and it fails linking:
$ mpicc -v Downloads/mpi_hello_world.c 
OpenBSD clang version 6.0.0 (tags/RELEASE_600/final) (based on LLVM 6.0.0)
Target: amd64-unknown-openbsd6.4
Thread model: posix
InstalledDir: /usr/bin
 "/usr/bin/cc" -cc1 -triple amd64-unknown-openbsd6.4 -emit-obj -mrelax-all 
-disable-free -disable-llvm-verifier -discard-value-names -main-file-name 
mpi_hello_world.c -mrelocation-model pic -pic-level 1 -pic-is-pie 
-mthread-model posix -mdisable-fp-elim -relaxed-aliasing -masm-verbose 
-mconstructor-aliases -munwind-tables -target-cpu x86-64 -dwarf-column-info 
-debugger-tuning=gdb -v -resource-dir /usr/lib/clang/6.0.0 -I 
/usr/local/include -fdebug-compilation-dir /home/tom -ferror-limit 19 
-fmessage-length 224 -femulated-tls -fwrapv -pthread -stack-protector 2 
-D_RET_PROTECTOR -ret-protector -fobjc-runtime=gnustep 
-fdiagnostics-show-option -fno-builtin-malloc -fno-builtin-calloc 
-fno-builtin-realloc -fno-builtin-valloc -fno-builtin-free -fno-builtin-strdup 
-fno-builtin-strndup -o /tmp/mpi_hello_world-a9e376.o -x c 
Downloads/mpi_hello_world.c
clang -cc1 version 6.0.0 based upon LLVM 6.0.0 default target 
amd64-unknown-openbsd6.4
#include "..." search starts here:
#include <...> search starts here:
 /usr/local/include
 /usr/lib/clang/6.0.0/include
 /usr/include
End of search list.
 "/usr/bin/ld" -e __start --eh-frame-hdr -Bdynamic -dynamic-linker 
/usr/libexec/ld.so -o a.out /usr/bin/../lib/crt0.o /usr/bin/../lib/crtbegin.o 
-L/usr/local/lib -L/usr/bin/../lib -L/usr/lib /tmp/mpi_hello_world-a9e376.o 
-lmpi -lopen-rte -lopen-pal -E -lutil -lm -lcompiler_rt -lpthread -lc 
-lcompiler_rt /usr/bin/../lib/crtend.o
/usr/bin/ld: error: undefined symbol: MPI_Init
>>> referenced by mpi_hello_world.c
>>>   /tmp/mpi_hello_world-a9e376.o:(main)

/usr/bin/ld: error: undefined symbol: MPI_Comm_size
>>> referenced by mpi_hello_world.c
>>>   /tmp/mpi_hello_world-a9e376.o:(main)

/usr/bin/ld: error: undefined symbol: MPI_Comm_rank
>>> referenced by mpi_hello_world.c
>>>   /tmp/mpi_hello_world-a9e376.o:(main)

/usr/bin/ld: error: undefined symbol: MPI_Get_processor_name
>>> referenced by mpi_hello_world.c
>>>   /tmp/mpi_hello_world-a9e376.o:(main)

/usr/bin/ld: error: undefined symbol: MPI_Finalize
>>> referenced by mpi_hello_world.c
>>>   /tmp/mpi_hello_world-a9e376.o:(main)
cc: error: linker command failed with exit code 1 (use -v to see invocation)

The reason seems to be, that the relevant symbols in libmpi.so.0.1 are local:
$ nm /usr/local/lib/libmpi.so.0.1  | grep MPI_Init
00070330 t MPI_Init
000704f0 t MPI_Init_thread
000706b0 t MPI_Initialized
00070330 T PMPI_Init
000704f0 T PMPI_Init_thread
000706b0 T PMPI_Initialized

Unfortunately, I have no idea why this happens.

Kind regards,

Thomas

Re: py-wxPython extras

[Jeremie Courreges-Anglas]

On Thu, May 24 2018, "Elias M. Mariani"  wrote:
> Hi,
> I was looking for Editra (text editor) and I found that there is not
> port of Editra by itself, It is contained within the py-wxPython port.
> https://github.com/openbsd/ports/blob/439514e8c82c269624be233d533b016d3a63a201/x11/py-wxPython/pkg/PLIST
>
> Shouldn't be better to remove the extras from py-wxPython and having
> those apart ?
> I mean, if you want wxPython maybe you do not want Editra.

ritchie~$ du -sh /usr/local/lib/python2.7/site-packages/wx/tools/Editra 
/usr/local/lib/python2.7/site-packages/wx/tools/XRCed
8.6M/usr/local/lib/python2.7/site-packages/wx/tools/Editra
1.3M/usr/local/lib/python2.7/site-packages/wx/tools/XRCed
ritchie ~$ pkg_info -s py-wxPython
Information for inst:py-wxPython-3.0.2.0

Size: 64856495

Gaining 10M may be worth it, but...

> And looking for standalone applications inside other ports doesn't
> seem right, I might as well thought that there was no port of Editra.

making those programs more visible is what matters most IMHO.  I would
support such a move.

However... my tries to update wxPython to the latest 4.x upstream
release were painful (we're using the last "classic", 3.0.x release).
I'm not sure it's a good idea to make the port more complicated *now*. :)

-- 
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE

Using clang from base install instead of from ports on loongson-current

[manphiz]

Hi,

Since loongson on current now provides llvm/clang, is it possible to
prefer this over devel/llvm from ports when building ports?



signature.asc
Description: OpenPGP digital signature

Re: Ruby 2.6.0

[George Koehler]

On Sat, 12 Jan 2019 21:33:53 -0800
Jeremy Evans  wrote:

> I think was able to fix both JIT issues.

Your new tarball from Sun, 13 Jan, fixes the JIT problems on my
powerpc machine.  "MJIT: CC defaults to /usr/bin/cc"

Old result, broken JIT:
  Finished tests in 6145.605596s, 3.2864 tests/s, 371.8192 assertions/s.
  20197 tests, 2285054 assertions, 64 failures, 16 errors, 112 skips
New result, fixed JIT:
  Finished tests in 6827.563384s, 2.9582 tests/s, 334.5004 assertions/s.
  20197 tests, 2283823 assertions, 14 failures, 15 errors, 111 skips

(I failed to clean my old ruby26 binary and the TEST_DEPENDS
reinstalled the old one.  The JIT tests passed, so they must have been
testing the new binary, not the old one.)

I saw 2 failures that weren't DES crypt or SSL:

  8) Failure:  
Fiddle::TestImport#test_gettimeofday [/usr/ports/pobj/ruby-2.6.0/ruby-2.6.0/test
/fiddle/test_import.rb:126]:   
Failed assertion, no message given.

I looked at test/fiddle/test_import.rb and it declares Timeval with
"long tv_sec", which has the correct 64 bits on amd64 and the wrong
32 bits on powerpc.  I typically run the tests on amd64, so I didn't
see this failure until now.  The test is wrong so we can ignore it.

 12) Failure:  
TestEnumerable#test_sum [/usr/ports/pobj/ruby-2.6.0/ruby-2.6.0/test/ruby/test_en
um.rb:1065]:   
 expected but was
. 

test/ruby/test_enum.rb does `k.new(+Float::INFINITY, 0.0).sum` where
k#each yields Float::INFINITY then 0.0.  I can reproduce the wrong NaN
in both irb25 and irb26 on powerpc:

irb(main):045:0> Enumerator.new{|y| y << Float::INFINITY << 0.0}.sum
=> NaN
irb(main):046:0> Enumerator.new{|y| y << Float::INFINITY << 0.0}.reduce(:+)
=> Infinity
irb(main):047:0> Enumerator.new{|y| y << Float::INFINITY << 0.0}.to_a.sum
=> Infinity

Trying the same 3 lines on amd64: irb25 gives NaN, Infinity, NaN;
irb from ruby 2.7.0dev (2019-01-13 trunk 66808) gives the correct
Infinity, Infinity, Infinity.  Therefore, Enumerable#sum has some
Float problem on powerpc, but Ruby 2.6 isn't worse than 2.5.

-- 
George Koehler 

Re: update sysutils/faubackup

[Solene Rapenne]

Solene Rapenne  wrote:
> hello
> 
> following this thread about faubackup
> https://marc.info/?l=openbsd-ports=154046081401380=2
> 
> I have been able to contact the authors, after some digging they were able to
> find a backup of the CVS repository (not including all history). I rehosted
> that history and last versions and did some changes to faubackup, publishing a
> new release.
> 
> New release includes some changes like "removing sourceforge links/mails", and
> I upstreamed some of our patches, like replacing rsh by ssh.
> 

Hi, slightly different diff from input with jca@, the license is a GPLv2+

Since last time, tarball checksum is still the same. I don't know much about
framagit gitlab tarball generation. If it ends up that the checksum is not
reliable I will host it somewhere else.

Index: Makefile
===
RCS file: /data/cvs/ports/sysutils/faubackup/Makefile,v
retrieving revision 1.9
diff -u -p -r1.9 Makefile
--- Makefile15 Nov 2018 20:15:09 -  1.9
+++ Makefile14 Jan 2019 05:45:45 -
@@ -1,19 +1,20 @@
 # $OpenBSD: Makefile,v 1.9 2018/11/15 20:15:09 solene Exp $
 
 COMMENT =  full and incremental backups on filesystem
-DISTNAME = faubackup-0.5.9
-REVISION = 1
+V =v0.6.1
+DISTNAME = faubackup-${V}
+PKGNAME =   ${DISTNAME:S/-v/-/}
 
 CATEGORIES =   sysutils
 
-HOMEPAGE = http://freshmeat.sourceforge.net/projects/faubackup
+HOMEPAGE = https://framagit.org/soleneBSD/faubackup/
 
-# GPLv2
+# GPLv2+
 PERMIT_PACKAGE_CDROM = Yes
 
 WANTLIB =  c popt>=0.3
 
-MASTER_SITES = ${MASTER_SITE_SOURCEFORGE:=faubackup/}
+MASTER_SITES = https://framagit.org/soleneBSD/faubackup/-/archive/${V}/
 
 LIB_DEPENDS =  devel/popt
 TEST_DEPENDS = ${MODGNU_AUTOCONF_DEPENDS}
Index: distinfo
===
RCS file: /data/cvs/ports/sysutils/faubackup/distinfo,v
retrieving revision 1.2
diff -u -p -r1.2 distinfo
--- distinfo18 Jan 2015 03:15:09 -  1.2
+++ distinfo19 Dec 2018 08:20:40 -
@@ -1,2 +1,2 @@
-SHA256 (faubackup-0.5.9.tar.gz) = D2uLJI7FXvqBs68p6bvYa2yrM2IovPJx7gHqMw00fLk=
-SIZE (faubackup-0.5.9.tar.gz) = 134982
+SHA256 (faubackup-v0.6.0.tar.gz) = 04c8Rx2hj0PRcKS3et/+E4jXKr+pJRKYlHNSFF257dQ=
+SIZE (faubackup-v0.6.0.tar.gz) = 135778
Index: patches/patch-faubackup_conf
===
RCS file: patches/patch-faubackup_conf
diff -N patches/patch-faubackup_conf
--- patches/patch-faubackup_conf29 Dec 2008 19:51:09 -  1.1.1.1
+++ /dev/null   1 Jan 1970 00:00:00 -
@@ -1,12 +0,0 @@
-$OpenBSD: patch-faubackup_conf,v 1.1.1.1 2008/12/29 19:51:09 sturm Exp $
 faubackup.conf.origSat Nov 29 13:13:59 2008
-+++ faubackup.conf Sat Nov 29 13:14:22 2008
-@@ -23,7 +23,7 @@ $keepdays= 7;
- #@ignore = qw( **.o **/cache **/Cache **/nobackup );
- 
- # use this command for remote commands
--$rsh  = "rsh";
-+$rsh  = "ssh";
- 
- 
- # make perl happy
Index: patches/patch-src_faubackup-gather_c
===
RCS file: patches/patch-src_faubackup-gather_c
diff -N patches/patch-src_faubackup-gather_c
--- patches/patch-src_faubackup-gather_c29 Dec 2008 19:51:09 -  
1.1.1.1
+++ /dev/null   1 Jan 1970 00:00:00 -
@@ -1,14 +0,0 @@
-$OpenBSD: patch-src_faubackup-gather_c,v 1.1.1.1 2008/12/29 19:51:09 sturm Exp 
$
 src/faubackup-gather.c.origSun Dec 28 13:00:40 2008
-+++ src/faubackup-gather.c Sun Dec 28 13:02:58 2008
-@@ -56,8 +56,8 @@ send_buf(void *_buf, size_t len)
-   while( len > 0 ) {
-   ret = fwrite( buf, 1, len, stdout );
-   if( ferror(stdout) ) {
--  fprintf( stderr, "%s: stdout fwrite %d bytes: %s\n",
--  progname, len, strerror(errno));
-+  fprintf( stderr, "%s: stdout fwrite %lu bytes: %s\n",
-+  progname, (unsigned long) len, 
strerror(errno));
-   exit(1);
-   }
-   buf += ret;
Index: patches/patch-src_faubackup-scatter_c
===
RCS file: patches/patch-src_faubackup-scatter_c
diff -N patches/patch-src_faubackup-scatter_c
--- patches/patch-src_faubackup-scatter_c   29 Dec 2008 19:51:09 -  
1.1.1.1
+++ /dev/null   1 Jan 1970 00:00:00 -
@@ -1,14 +0,0 @@
-$OpenBSD: patch-src_faubackup-scatter_c,v 1.1.1.1 2008/12/29 19:51:09 sturm 
Exp $
 src/faubackup-scatter.c.orig   Sun Dec 28 13:00:52 2008
-+++ src/faubackup-scatter.cSun Dec 28 13:02:24 2008
-@@ -69,8 +69,8 @@ recv_buf(void *buf, size_t len, int mayfail)
-   }
- 
-   if( len!=0 && ! mayfail ) {
--  fprintf(stderr, "%s: protocol error (%d bytes missing)\n",
--  

Re: devel/openmpi Can't link example program, symbols local in libmpi.so.0.1?

[Andreas Kusalananda Kähäri]

On Sun, Jan 13, 2019 at 04:37:23PM +0100, Thomas L. wrote:
> Hi,
> 
> I tried compiling a MPI example program 
> (https://raw.githubusercontent.com/wesleykendall/mpitutorial/gh-pages/tutorials/mpi-hello-world/code/mpi_hello_world.c)
>  with openmpi on -current and it fails linking:
> $ mpicc -v Downloads/mpi_hello_world.c 
> OpenBSD clang version 6.0.0 (tags/RELEASE_600/final) (based on LLVM 6.0.0)
> Target: amd64-unknown-openbsd6.4
> Thread model: posix
> InstalledDir: /usr/bin
>  "/usr/bin/cc" -cc1 -triple amd64-unknown-openbsd6.4 -emit-obj -mrelax-all 
> -disable-free -disable-llvm-verifier -discard-value-names -main-file-name 
> mpi_hello_world.c -mrelocation-model pic -pic-level 1 -pic-is-pie 
> -mthread-model posix -mdisable-fp-elim -relaxed-aliasing -masm-verbose 
> -mconstructor-aliases -munwind-tables -target-cpu x86-64 -dwarf-column-info 
> -debugger-tuning=gdb -v -resource-dir /usr/lib/clang/6.0.0 -I 
> /usr/local/include -fdebug-compilation-dir /home/tom -ferror-limit 19 
> -fmessage-length 224 -femulated-tls -fwrapv -pthread -stack-protector 2 
> -D_RET_PROTECTOR -ret-protector -fobjc-runtime=gnustep 
> -fdiagnostics-show-option -fno-builtin-malloc -fno-builtin-calloc 
> -fno-builtin-realloc -fno-builtin-valloc -fno-builtin-free 
> -fno-builtin-strdup -fno-builtin-strndup -o /tmp/mpi_hello_world-a9e376.o -x 
> c Downloads/mpi_hello_world.c
> clang -cc1 version 6.0.0 based upon LLVM 6.0.0 default target 
> amd64-unknown-openbsd6.4
> #include "..." search starts here:
> #include <...> search starts here:
>  /usr/local/include
>  /usr/lib/clang/6.0.0/include
>  /usr/include
> End of search list.
>  "/usr/bin/ld" -e __start --eh-frame-hdr -Bdynamic -dynamic-linker 
> /usr/libexec/ld.so -o a.out /usr/bin/../lib/crt0.o /usr/bin/../lib/crtbegin.o 
> -L/usr/local/lib -L/usr/bin/../lib -L/usr/lib /tmp/mpi_hello_world-a9e376.o 
> -lmpi -lopen-rte -lopen-pal -E -lutil -lm -lcompiler_rt -lpthread -lc 
> -lcompiler_rt /usr/bin/../lib/crtend.o
> /usr/bin/ld: error: undefined symbol: MPI_Init
> >>> referenced by mpi_hello_world.c
> >>>   /tmp/mpi_hello_world-a9e376.o:(main)
> 
> /usr/bin/ld: error: undefined symbol: MPI_Comm_size
> >>> referenced by mpi_hello_world.c
> >>>   /tmp/mpi_hello_world-a9e376.o:(main)
> 
> /usr/bin/ld: error: undefined symbol: MPI_Comm_rank
> >>> referenced by mpi_hello_world.c
> >>>   /tmp/mpi_hello_world-a9e376.o:(main)
> 
> /usr/bin/ld: error: undefined symbol: MPI_Get_processor_name
> >>> referenced by mpi_hello_world.c
> >>>   /tmp/mpi_hello_world-a9e376.o:(main)
> 
> /usr/bin/ld: error: undefined symbol: MPI_Finalize
> >>> referenced by mpi_hello_world.c
> >>>   /tmp/mpi_hello_world-a9e376.o:(main)
> cc: error: linker command failed with exit code 1 (use -v to see invocation)
> 
> The reason seems to be, that the relevant symbols in libmpi.so.0.1 are local:
> $ nm /usr/local/lib/libmpi.so.0.1  | grep MPI_Init
> 00070330 t MPI_Init
> 000704f0 t MPI_Init_thread
> 000706b0 t MPI_Initialized
> 00070330 T PMPI_Init
> 000704f0 T PMPI_Init_thread
> 000706b0 T PMPI_Initialized
> 
> Unfortunately, I have no idea why this happens.
> 
> Kind regards,
> 
> Thomas

Yes, I also noticed this in June last year [1].  Only the profiling
version of each API function is non-local.  Unfortunately, I was in a
hurry to get something working so I simply moved my development and
testing to another operating system rather than digging into why this
was so.  I might try to look into this later if time permits.

[1] https://marc.info/?l=openbsd-ports=152929789302360=2


Regards,

-- 
Andreas Kusalananda Kähäri,
National Bioinformatics Infrastructure Sweden (NBIS),
Uppsala University, Sweden.

Re: UPDATE: editors/kile (Replacement for editors/kile-kde4)

[Rafael Sadowski]

On Fri Jan 04, 2019 at 09:24:54PM +0100, Rafael Sadowski wrote:
> Information for inst:kile-2.9.92
> 
> Comment:
> user-friendly TeX/LaTex editor for KDE
> 
> Description:
> Kile is a user-friendly TeX/LaTeX editor for KDE. The main features are:
> 
>   * Auto-completion of (La)TeX commands.
>   * Templates and wizards make starting a new document very little work.
>   * Easy insertion of many standard tags and symbols and the option to
> define (an arbitrary number of) user defined tags.
>   * Easy finding chapter or sections.
>   * Collecting documents that belong together into a project.
>   * Easy insertion of citations and references when using projects.
>   * Flexible and smart build system to compile your LaTeX documents.
>   * QuickPreview, preview a selected part of your document.
>   * Easy access to various help sources.
>   * Advanced editing commands.
> 
> Maintainer: Rafael Sadowski 
> 
> WWW: https://kile.sourceforge.io
> 
> OK to replace / update KDE4 kile by KDE5? Feedback welcome.
> 
> RS

ping

Re: fix parallel build of automake-1.16

[Rafael Sadowski]

On Sat Jan 12, 2019 at 11:09:30PM +0300, Kirill Bychkov wrote:
> Hi!
> automake-1.16 fails to build with MAKE_JOBS set in mk.conf
> with the following error:
> 
> :  && mkdir -p doc  && ./pre-inst-env /usr/bin/perl ./doc/help2man
> --output=doc/aclocal-1.16.1 aclocal-1.16
> :  && mkdir -p doc  && ./pre-inst-env /usr/bin/perl ./doc/help2man
> --output=doc/automake-1.16.1 automake-1.16
> help2man: can't get `--help' info from aclocal-1.16
> Try `--no-discard-stderr' if option outputs to stderr
> help2man: can't get `--help' info from automake-1.16
> Try `--no-discard-stderr' if option outputs to stderr
> 
> The patch below fixes build. Tested with MAKE_JOBS=2 on amd64
> and i386.

Hmm, no issues with MAKE_JOBS=4 and without your patch on amd64.

> 
> OK to commit?
> 
> Index: patches/patch-Makefile_in
> ===
> RCS file: patches/patch-Makefile_in
> diff -N patches/patch-Makefile_in
> --- /dev/null 1 Jan 1970 00:00:00 -
> +++ patches/patch-Makefile_in 12 Jan 2019 20:02:58 -
> @@ -0,0 +1,20 @@
> +$OpenBSD$
> +
> +Fix parallel build.Taken from FreeBSD.
> +https://svnweb.freebsd.org/ports/head/devel/automake/files/patch-Makefile.in?revision=474011=markup

I think the link is uesless because no more additional informations.

> +
> +Index: Makefile.in
> +--- Makefile.in.orig
>  Makefile.in
> +@@ -3690,9 +3690,9 @@ doc/aclocal.1 doc/automake.1:
> +   && echo ".so man1/$$f-$(APIVERSION).1" > $@
> +
> + doc/aclocal-$(APIVERSION).1: $(aclocal_script) lib/Automake/Config.pm
> +-$(update_mans) aclocal-$(APIVERSION)
> ++$(update_mans) $(aclocal_script)
> + doc/automake-$(APIVERSION).1: $(automake_script) lib/Automake/Config.pm
> +-$(update_mans) automake-$(APIVERSION)
> ++$(update_mans) $(automake_script)
> +
> + # We depend on configure.ac so that we regenerate the tarball
> + # whenever the Automake version changes.
> 
> 

Re: fix parallel build of automake-1.16

[Kirill Bychkov]

On Sun, January 13, 2019 15:57, Rafael Sadowski wrote:
> On Sat Jan 12, 2019 at 11:09:30PM +0300, Kirill Bychkov wrote:
>> Hi!
>> automake-1.16 fails to build with MAKE_JOBS set in mk.conf
>> with the following error:
>>
>> :  && mkdir -p doc  && ./pre-inst-env /usr/bin/perl ./doc/help2man
>> --output=doc/aclocal-1.16.1 aclocal-1.16
>> :  && mkdir -p doc  && ./pre-inst-env /usr/bin/perl ./doc/help2man
>> --output=doc/automake-1.16.1 automake-1.16
>> help2man: can't get `--help' info from aclocal-1.16
>> Try `--no-discard-stderr' if option outputs to stderr
>> help2man: can't get `--help' info from automake-1.16
>> Try `--no-discard-stderr' if option outputs to stderr
>>
>> The patch below fixes build. Tested with MAKE_JOBS=2 on amd64
>> and i386.
>
> Hmm, no issues with MAKE_JOBS=4 and without your patch on amd64.

Furhter testing showed up that it fails without a patch every time
with MAKE_JOBS=2, about 1 time from 20 with MAKE_JOBS=4 and 1 time
of 2 with MAKE_JOBS=6.

>>
>> OK to commit?
>>
>> Index: patches/patch-Makefile_in
>> ===
>> RCS file: patches/patch-Makefile_in
>> diff -N patches/patch-Makefile_in
>> --- /dev/null1 Jan 1970 00:00:00 -
>> +++ patches/patch-Makefile_in12 Jan 2019 20:02:58 -
>> @@ -0,0 +1,20 @@
>> +$OpenBSD$
>> +
>> +Fix parallel build.Taken from FreeBSD.
>> +https://svnweb.freebsd.org/ports/head/devel/automake/files/patch-Makefile.in?revision=474011=markup
>
> I think the link is uesless because no more additional informations.

OK

>> +
>> +Index: Makefile.in
>> +--- Makefile.in.orig
>>  Makefile.in
>> +@@ -3690,9 +3690,9 @@ doc/aclocal.1 doc/automake.1:
>> +  && echo ".so man1/$$f-$(APIVERSION).1" > $@
>> +
>> + doc/aclocal-$(APIVERSION).1: $(aclocal_script) lib/Automake/Config.pm
>> +-   $(update_mans) aclocal-$(APIVERSION)
>> ++   $(update_mans) $(aclocal_script)
>> + doc/automake-$(APIVERSION).1: $(automake_script) lib/Automake/Config.pm
>> +-   $(update_mans) automake-$(APIVERSION)
>> ++   $(update_mans) $(automake_script)
>> +
>> + # We depend on configure.ac so that we regenerate the tarball
>> + # whenever the Automake version changes.
>>
>>
>

Re: powerpc bulk build report

[Charlene Wendling]

On Sat, 12 Jan 2019 22:28:38 +0100
Christian Weisgerber wrote:

> Charlene Wendling:
> 
> > > http://build-failures.rhaalovely.net//powerpc/2019-01-01/x11/bbpager.log
> > 
> > It fails because some headers are missing.  With those patches, it 
> > builds [1] and runs [2] fine on macppc, and doesn't break amd64
> > build.
> > 
> > Comments/feedback are welcome! 
> 
> Personally, I try to follow the existing style of the source, i.e.,
> if C headers are included in their C form like
> 
> #include 
> 
> then I'll add  and not .
> 
> -- 
> Christian "naddy" Weisgerber
> na...@mips.inka.de
> 

Hi, 

Here it is. I'm unhappy as well with Baseresource.cxx, but it would 
make the patch intent's less clear. 

Charlène. 

Index: Makefile
===
RCS file: /cvs/ports/x11/bbpager/Makefile,v
retrieving revision 1.27
diff -u -p -u -p -r1.27 Makefile
--- Makefile24 Oct 2018 14:28:12 -  1.27
+++ Makefile13 Jan 2019 13:39:23 -
@@ -3,7 +3,7 @@
 COMMENT =  pager for the Blackbox window manager
 
 DISTNAME = bbpager-0.4.7
-REVISION = 6
+REVISION = 7
 
 CATEGORIES =   x11
 MASTER_SITES = ${MASTER_SITE_SOURCEFORGE:=bbtools/}
Index: patches/patch-src_Baseresource_cxx
===
RCS file: patches/patch-src_Baseresource_cxx
diff -N patches/patch-src_Baseresource_cxx
--- /dev/null   1 Jan 1970 00:00:00 -
+++ patches/patch-src_Baseresource_cxx  13 Jan 2019 13:39:23 -
@@ -0,0 +1,13 @@
+$OpenBSD$
+Fix missing header for ports-gcc
+Index: src/Baseresource.cxx
+--- src/Baseresource.cxx.orig
 src/Baseresource.cxx
+@@ -22,6 +22,7 @@
+ #include 
+ #include "Baseresource.h"
+ #include 
++#include 
+ #include "Menu.hh"
+ 
+ BaseResource::BaseResource(bt::Application &_app, unsigned int _screen, const 
std::string _rc_filename, const std::string ) : 
Index: patches/patch-src_main_cxx
===
RCS file: patches/patch-src_main_cxx
diff -N patches/patch-src_main_cxx
--- /dev/null   1 Jan 1970 00:00:00 -
+++ patches/patch-src_main_cxx  13 Jan 2019 13:39:23 -
@@ -0,0 +1,13 @@
+$OpenBSD$
+Fix missing header for ports-gcc
+Index: src/main.cxx
+--- src/main.cxx.orig
 src/main.cxx
+@@ -24,6 +24,7 @@
+ #include "config.h"
+ 
+ #include 
++#include 
+ #include 
+ #include 
+ 
Index: patches/patch-src_resource_cxx
===
RCS file: patches/patch-src_resource_cxx
diff -N patches/patch-src_resource_cxx
--- /dev/null   1 Jan 1970 00:00:00 -
+++ patches/patch-src_resource_cxx  13 Jan 2019 13:39:23 -
@@ -0,0 +1,13 @@
+$OpenBSD$
+Fix missing header for ports-gcc
+Index: src/resource.cxx
+--- src/resource.cxx.orig
 src/resource.cxx
+@@ -19,6 +19,7 @@
+ // (See the included file COPYING / GPL-2.0)
+ //
+ 
++#include 
+ #include "resource.h"
+ #include "blackboxstyle.h"
+ 

Re: UPDATE: editors/kile (Replacement for editors/kile-kde4)

[Raphael Graf]

On Sun, January 13, 2019 1:06 pm, Rafael Sadowski wrote:
> On Fri Jan 04, 2019 at 09:24:54PM +0100, Rafael Sadowski wrote:
>> Information for inst:kile-2.9.92
>>
>> Comment:
>> user-friendly TeX/LaTex editor for KDE
>>
>> Description:
>> Kile is a user-friendly TeX/LaTeX editor for KDE. The main features are:
>>
>>   * Auto-completion of (La)TeX commands.
>>   * Templates and wizards make starting a new document very little work.
>>   * Easy insertion of many standard tags and symbols and the option to
>> define (an arbitrary number of) user defined tags.
>>   * Easy finding chapter or sections.
>>   * Collecting documents that belong together into a project.
>>   * Easy insertion of citations and references when using projects.
>>   * Flexible and smart build system to compile your LaTeX documents.
>>   * QuickPreview, preview a selected part of your document.
>>   * Easy access to various help sources.
>>   * Advanced editing commands.
>>
>> Maintainer: Rafael Sadowski 
>>
>> WWW: https://kile.sourceforge.io
>>
>> OK to replace / update KDE4 kile by KDE5? Feedback welcome.
>>
>> RS
>
> ping
>
>

It looks good and works for me on amd64.

There is a typo in the COMMENT:

--- Makefile.orig   Sun Jan 13 14:03:08 2019
+++ MakefileSun Jan 13 14:32:07 2019
@@ -1,6 +1,6 @@
 # $OpenBSD: Makefile,v 1.16 2018/07/01 18:33:35 espie Exp $

-COMMENT =  user-friendly TeX/LaTex editor for KDE
+COMMENT =  user-friendly TeX/LaTeX editor for KDE

 DISTNAME = kile-2.9.92

[macppc] games/hyperrogue: fix build, but colors are off

[Charlene Wendling]

> http://build-failures.rhaalovely.net//powerpc/2019-01-01/games/hyperrogue.log

__multc3 is defined in compiler-rt, but is not built when building llvm.
So we need ports-gcc to build that port on macppc. I'm joining a diff
for that.

Runtime is fine, but colors are off: the ice is red instead of
blueish [1]. Anyone know where to start to look at? 

Charlène. 

[1] https://transfer.sh/AWemo/c21e4a8081b2e9b4.jpg


Index: Makefile
===
RCS file: /cvs/ports/games/hyperrogue/Makefile,v
retrieving revision 1.8
diff -u -p -u -p -r1.8 Makefile
--- Makefile21 Nov 2018 01:58:38 -  1.8
+++ Makefile13 Jan 2019 16:22:41 -
@@ -17,7 +17,7 @@ WANTLIB += ${COMPILER_LIBCXX} GL GLEW SD
 WANTLIB += c m png
 
 # C++11
-COMPILER = base-clang ports-clang ports-gcc
+COMPILER = base-clang ports-gcc
 
 BUILD_DEPENDS =${MODGNU_AUTOCONF_DEPENDS} \
${MODGNU_AUTOMAKE_DEPENDS}

Re: burp: 2 questions

[Olivier Cherrier]

On Sun, Dec 23, 2018 at 10:54:04PM +, o...@symacx.com wrote:
> 
> _ burp, in server mode, is starting, logging such a following line:
> Dec 23 22:09:18 burp burp[48269]: server 0.0.0.0:4971 (max 5)
>   and exits just after.  Adding the '-F' option makes it continuing to
>   run but it is not the normal mode.

Responding to myself.
It comes from the fact burp server is exiting (except if run with '-F'
option) when no 'listen' directive is specified for IPv6:

$ pgrep -l burp
14320 burp
$
$ grep '^listen' /etc/burp/burp-server.conf   
listen = 0.0.0.0:4971
listen = :::5971
$
$ vi  /etc/burp/burp-server.conf  
$
$ grep '^listen' /etc/burp/burp-server.conf   
listen = 0.0.0.0:4971
$
$
$ rcctl restart burp
burp(ok)
burp(ok)
$
$ pgrep -l burp
$
$ vi  /etc/burp/burp-server.conf  
$
$ grep '^listen' /etc/burp/burp-server.conf 
listen = 0.0.0.0:4971
listen = :::5971
$ 
$ rcctl restart burp
burp(ok)
$ 
$ 
$ pgrep -l burp
76694 burp
$ 


-- 
Olivier Cherrier
Phone: +352691754777
mailto:o...@symacx.com

Re: burp: 2 questions

[Olivier Cherrier]

On Sun, Dec 23, 2018 at 11:49:32PM +, o...@symacx.com wrote:
> Date: Sun, 23 Dec 2018 23:49:32 +
> From: Olivier Cherrier 
> To: OpenBSD ports 
> Subject: Re: burp: 2 questions
> User-Agent: Mutt/1.10.1 (2018-07-13)
> 
> 
> Here is an update to 2.2.14.

to 2.2.16 now …

-- 
Olivier Cherrier
Phone: +352691754777
mailto:o...@symacx.com
Index: sysutils/burp//2.2/Makefile
===
RCS file: /cvs/ports/sysutils/burp/2.2/Makefile,v
retrieving revision 1.1.1.1
diff -u -r1.1.1.1 Makefile
--- sysutils/burp//2.2/Makefile 1 Dec 2018 09:13:08 -   1.1.1.1
+++ sysutils/burp//2.2/Makefile 13 Jan 2019 16:40:57 -
@@ -1,6 +1,6 @@
 # $OpenBSD: Makefile,v 1.1.1.1 2018/12/01 09:13:08 florian Exp $
 
-VERSION =  2.2.12
+VERSION =  2.2.16
 
 post-extract:
sed -i 's,"/usr/sbin/%s","${TRUEPREFIX}/sbin/%s",' \
Index: sysutils/burp//2.2/distinfo
===
RCS file: /cvs/ports/sysutils/burp/2.2/distinfo,v
retrieving revision 1.1.1.1
diff -u -r1.1.1.1 distinfo
--- sysutils/burp//2.2/distinfo 1 Dec 2018 09:13:08 -   1.1.1.1
+++ sysutils/burp//2.2/distinfo 13 Jan 2019 16:40:57 -
@@ -1,2 +1,2 @@
-SHA256 (burp-2.2.12.tar.bz2) = g5baoJOgrKOhfiXlumZL6Nu0S8z0+O07xy2jAsBjOG0=
-SIZE (burp-2.2.12.tar.bz2) = 810424
+SHA256 (burp-2.2.16.tar.bz2) = 25IcF8yUrGu1OQzlZVV1KQNDzDxh+I+2UT+BXAJ3RE4=
+SIZE (burp-2.2.16.tar.bz2) = 814844
Index: sysutils/burp//2.2/pkg/PLIST
===
RCS file: /cvs/ports/sysutils/burp/2.2/pkg/PLIST,v
retrieving revision 1.1.1.1
diff -u -r1.1.1.1 PLIST
--- sysutils/burp//2.2/pkg/PLIST1 Dec 2018 09:13:08 -   1.1.1.1
+++ sysutils/burp//2.2/pkg/PLIST13 Jan 2019 16:40:57 -
@@ -72,8 +72,8 @@
 @owner _burp
 @group _burp
 @sample ${SYSCONFDIR}/clientconfdir/incexc/example
+@sample ${LOCALSTATEDIR}/spool/burp/
 @owner
 @group
 share/examples/burp/clientconfdir/testclient
-@sample ${LOCALSTATEDIR}/spool/burp/
 @sample ${SYSCONFDIR}/CA-client/

Re: UPDATE: audio/mscore to 3.0.0

[Brian Callahan]

On 1/11/19 6:23 PM, Sebastian Reitenbach wrote:

Hi Brian,
  
Am Freitag, Januar 11, 2019 23:14 CET, Brian Callahan  schrieb:
  

Hi Sebastian --

On 1/10/19 8:26 AM, Sebastian Reitenbach wrote:

Find attached and inline an update of Musescore to 3.0.0.
See the release notes for changes: 
https://musescore.org/en/handbook/developers-handbook/release-notes/release-notes-musescore-3
Also you may find this mentioning remarks/bugs/issues with the Musescore 3 
version:
https://musescore.org/en/handbook/developers-handbook/references/musescore-roadmap/remarks-musescore3-bugs-issues
Most notably thereof the last point under the TODOs, mentioning the
dependency to QtWebEngine for the MuseScore connect window.
QtWebEngine is not yet ported to OpenBSD, so I disabled that
for the time being.

Also in the ports Makefile, I introduced the MAJ_MIN variable, thats then used
in the PLIST to (hopefully) prevent further huge diffs in the future once they
bump major or minor version again.

Other than that, It happily imported and converted my few scores from the older 
version, played
them etc.


tested on amd64.


any comments, reports, or even OKs welcome.

Thanks for this; it was on my TODO list, somewhere on the list of a
million other things :)
One thought: could we take those BUILD_*+=ON/OFF patches out and just
put them in CONFIGURE_ARGS like we do everything else?


indeed, so how about this then (still building for me):


I'm good with this. Thanks.

~Brian




Index: Makefile
===
RCS file: /cvs/ports/audio/mscore/Makefile,v
retrieving revision 1.36
diff -u -r1.36 Makefile
--- Makefile2 Sep 2018 23:16:28 -   1.36
+++ Makefile11 Jan 2019 23:18:38 -
@@ -2,7 +2,7 @@
  
  BROKEN-hppa =	ICE on moc_qtsingleapplication.cxx
  
-V =		2.3.2

+V =3.0.0
  COMMENT = WYSIWYG music notation software (MuseScore)
  DISTNAME =MuseScore-${V}
  PKGNAME = musescore-${V}
@@ -15,9 +15,9 @@
  # GPLv2 only
  PERMIT_PACKAGE_CDROM =Yes
  
-WANTLIB += Qt5Concurrent Qt5Core Qt5Designer Qt5Gui Qt5Help Qt5Network

+WANTLIB += Qt5Concurrent Qt5Core Qt5Gui Qt5Help Qt5Network
  WANTLIB += Qt5OpenGL Qt5PrintSupport Qt5Qml Qt5Quick Qt5QuickWidgets
-WANTLIB += Qt5Sql Qt5Svg Qt5Test Qt5WebKit Qt5WebKitWidgets Qt5Widgets
+WANTLIB += Qt5Sql Qt5Svg Qt5Test Qt5Widgets
  WANTLIB += Qt5Xml Qt5XmlPatterns c freetype m ogg portaudio pthread
  WANTLIB += sndfile vorbis vorbisfile z
  WANTLIB += ${COMPILER_LIBCXX}
@@ -35,7 +35,6 @@
x11/qt5/qtdeclarative \
x11/qt5/qtsvg \
x11/qt5/qttools \
-   x11/qt5/qtwebkit \
x11/qt5/qtxmlpatterns
  
  RUN_DEPENDS =	devel/desktop-file-utils \

@@ -48,9 +47,13 @@
-DBUILD_JACK=OFF \
-DBUILD_PORTMIDI=OFF \
-DBUILD_PULSEAUDIO=OFF \
+   -DBUILD_PCH=OFF \
+   -DBUILD_WEBENGINE=OFF \
-DUSE_SYSTEM_FREETYPE=ON
  
  NO_TEST =	Yes

+MAJ_MIN =  ${V:C/^([0-9]+\.[0-9]+).*/\1/}
+SUBST_VARS +=  MAJ_MIN
  
  # Bad zip file...

  WRKDIST = ${WRKDIR}
Index: distinfo
===
RCS file: /cvs/ports/audio/mscore/distinfo,v
retrieving revision 1.8
diff -u -r1.8 distinfo
--- distinfo2 Sep 2018 23:16:28 -   1.8
+++ distinfo11 Jan 2019 23:18:38 -
@@ -1,2 +1,2 @@
-SHA256 (MuseScore-2.3.2.zip) = W/RKmC6Y8xYIyFsO0JxQ1+TwGi+sxbXm8HLnF0wr2vg=
-SIZE (MuseScore-2.3.2.zip) = 78629900
+SHA256 (MuseScore-3.0.0.zip) = inZKH4kRUCy5iHyEzeSo3aGTp2Yze45HfokYXL83IsY=
+SIZE (MuseScore-3.0.0.zip) = 116183004
Index: patches/patch-CMakeLists_txt
===
RCS file: /cvs/ports/audio/mscore/patches/patch-CMakeLists_txt,v
retrieving revision 1.5
diff -u -r1.5 patch-CMakeLists_txt
--- patches/patch-CMakeLists_txt25 Apr 2018 19:11:03 -  1.5
+++ patches/patch-CMakeLists_txt11 Jan 2019 23:18:38 -
@@ -2,89 +2,109 @@
  Index: CMakeLists.txt
  --- CMakeLists.txt.orig
  +++ CMakeLists.txt
-@@ -170,7 +170,7 @@ if ("${CMAKE_CXX_COMPILER_ID}" MATCHES "GNU")
- endif ("${CMAKE_CXX_COMPILER_ID}" MATCHES "GNU")
+@@ -208,7 +208,7 @@ endif (${CMAKE_CXX_COMPILER_ID} MATCHES "GNU")
   
- set(CMAKE_CXX_FLAGS_DEBUG   "-g")

--set(CMAKE_CXX_FLAGS_RELEASE "-O2 -DNDEBUG -DQT_NO_DEBUG")
-+set(CMAKE_CXX_FLAGS_RELEASE "-DNDEBUG -DQT_NO_DEBUG")
+ if (NOT MSVC)
+set(CMAKE_CXX_FLAGS_DEBUG   "-g")
+-   set(CMAKE_CXX_FLAGS_RELEASE "-O2 -DNDEBUG -DQT_NO_DEBUG")
++   set(CMAKE_CXX_FLAGS_RELEASE "-DNDEBUG -DQT_NO_DEBUG")
+ endif (NOT MSVC)
   
   if (APPLE)

-set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11 -fPIC -stdlib=libc++ 
-Wno-inconsistent-missing-override")
-@@ -328,13 +328,13 @@ endif (BUILD_ALSA)
+@@ -384,13 +384,13 @@ endif (BUILD_ALSA)
   ## MIDI
   ##
   
--if (APPLE OR MINGW)

-+if (APPLE OR MINGW OR UNIX)
+-if (APPLE OR MINGW OR MSVC)

Re: Ruby 2.6.0

[Jeremie Courreges-Anglas]

On Sat, Jan 12 2019, Jeremy Evans  wrote:
> On 01/12 07:50, Jeremy Evans wrote:
>> Thank you very much for the testing and analysis.  I can confirm the JIT
>> compilation problems on amd64 because it uses the cached cc path (before
>> it would even get to the __guard_local issues on gcc).
>> 
>> Below is a new Makefile that will use dist patches pulled from Ruby's
>> SVN to handle the random.c and addr2line.c issues, as well as an
>> additional significant regression.  I tested that it also builds with
>> CC=/usr/bin/gcc.
>> 
>> I will keep working on the JIT issues to see if I can fix them and get
>> something that works with both base clang and gcc.
>
> I think was able to fix both JIT issues.  Here's an updated Makefile as
> well as a new patch to mjit_worker.c.  I've tested this with both base
> clang and base gcc on amd64, same results on both.
>
> I would appreciate testing to see if this fixes the issues identified,
> and if so, OKs to import and hookup.  Assuming the mjit_worker.c patch
> fixes the JIT issues on gcc, I'll try to get it upstreamed.

I'd gladly test with your fixes, but an inline Makefile plus an inline
patch make testing harder than needed.  For new ports it's much more
convenient to propose an updated, standalone tarball.

-- 
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE

Re: mongodb and mincore(2)

[Jeremie Courreges-Anglas]

On Sat, Jan 12 2019, Antoine Jacoutot  wrote:
> On Sat, Jan 12, 2019 at 02:01:13PM +0100, Jeremie Courreges-Anglas wrote:
>> 
>> So mongodb didn't show up when guenther@ looked for mincore(2) users in
>> ports, because the search was done over i386 .o, but mongodb is only for
>> amd64.
>> 
>> Here's a diff to unbreak it.  There's a "feature check" function
>> blockCheckSupported() which now returns false, if code ends up querying
>> the presence of pages in memory with blockInMemory() or pagesInMemory()
>> the answer will also be false.  I think those defaults are consistent
>> with the choices made in 
>> src/mongo/db/storage/mmap_v1/record_access_tracker.cpp
>> See "if (!_blockSupported)".
>> 
>> --8<--
>> bool RecordAccessTracker::checkAccessedAndMark(const void* record) {
>> const size_t page = reinterpret_cast(record) >> 12;
>> const size_t region = page >> 6;
>> const size_t offset = page & 0x3f;
>> 
>> // This is like the "L1 cache". If we're a miss then we fall through and 
>> check the
>> // "L2 cache". If we're still a miss, then we defer to a system-specific 
>> system
>> // call (or give up and return false if deferring to the system call is 
>> not enabled).
>> if (PointerTable::seen(PointerTable::getData(), 
>> reinterpret_cast(record))) {
>> return true;
>> }
>> 
>> // We were a miss in the PointerTable. See if we can find 'record' in 
>> the Rolling table.
>> if (_rollingTable[bigHash(region)].access(region, offset, false)) {
>> return true;
>> }
>> 
>> if (!_blockSupported) {
>> // This means we don't fall back to a system call. Instead we assume 
>> things aren't
>> // in memory. This could mean that we yield too much, but this is 
>> much better
>> // than the alternative of not yielding through a page fault.
>> return false;
>> }
>> 
>> return ProcessInfo::blockInMemory(const_cast(record));
>> }
>> -->8--
>> 
>> mongodb will now print a warning on startup:
>> --8<--
>>  2019-01-12T13:56:00.082+0100 I CONTROL  [initandlisten] ** NOTE: your 
>> operating system version does not support the method that MongoDB
>>  2019-01-12T13:56:00.082+0100 I CONTROL  [initandlisten] **   uses to 
>> detect impending page faults.
>>  2019-01-12T13:56:00.082+0100 I CONTROL  [initandlisten] **   This may 
>> result in slower performance for certain use cases
>> -->8--
>> 
>> On a new install I had to tweak the rcscript so that the default log
>> file can be opened and written to.  I can commit that part separately.
>
> No, that should be created by the package itseld (@sample).

Indeed, thanks.  And guess what, that's already what the package does! 8)
The 0750 perms on the directory prevent tab completion as my usual
account, I guess this put me on the wrong track.

So here's a diff that just handles the mincore(2) removal.  ok?


Index: Makefile
===
RCS file: /cvs/ports/databases/mongodb/Makefile,v
retrieving revision 1.33
diff -u -p -r1.33 Makefile
--- Makefile13 Dec 2018 19:53:23 -  1.33
+++ Makefile13 Jan 2019 14:30:21 -
@@ -12,7 +12,7 @@ COMMENT = scalable, high-performance doc
 DISTNAME = mongodb-src-r3.2.13
 PKGNAME =  ${DISTNAME:S/src-r//}
 CATEGORIES =   databases
-REVISION = 3
+REVISION = 4
 
 HOMEPAGE = https://www.mongodb.com/
 
Index: patches/patch-src_mongo_util_processinfo_openbsd_cpp
===
RCS file: 
/cvs/ports/databases/mongodb/patches/patch-src_mongo_util_processinfo_openbsd_cpp,v
retrieving revision 1.1
diff -u -p -r1.1 patch-src_mongo_util_processinfo_openbsd_cpp
--- patches/patch-src_mongo_util_processinfo_openbsd_cpp8 Jun 2017 
16:16:43 -   1.1
+++ patches/patch-src_mongo_util_processinfo_openbsd_cpp13 Jan 2019 
14:30:21 -
@@ -1,5 +1,8 @@
 $OpenBSD: patch-src_mongo_util_processinfo_openbsd_cpp,v 1.1 2017/06/08 
16:16:43 sthen Exp $
 
+- avoid use after free
+- mincore(2) has been removed
+
 Index: src/mongo/util/processinfo_openbsd.cpp
 --- src/mongo/util/processinfo_openbsd.cpp.orig
 +++ src/mongo/util/processinfo_openbsd.cpp
@@ -27,3 +30,35 @@ Index: src/mongo/util/processinfo_openbs
  }
  
  double ProcessInfo::getSystemMemoryPressurePercentage() {
+@@ -182,28 +184,14 @@ bool ProcessInfo::supported() {
+ }
+ 
+ bool ProcessInfo::blockCheckSupported() {
+-return true;
++return false;
+ }
+ 
+ bool ProcessInfo::blockInMemory(const void* start) {
+-char x = 0;
+-if (mincore((void*)alignToStartOfPage(start), getPageSize(), )) {
+-log() << "mincore failed: " << errnoWithDescription() << endl;
+-return 1;
+-}
+-return x & 0x1;
++return false;
+ }
+ 
+ bool ProcessInfo::pagesInMemory(const void* start, size_t numPages, 
vector* out) {
+-out->resize(numPages);
+-// int mincore(const void *addr, size_t len, char *vec);
+-if 

Re: fix parallel build of automake-1.16

[Klemens Nanni]

OK

Re: UPDATE: editors/kile (Replacement for editors/kile-kde4)

[Klemens Nanni]

Looks good besides the following:

$ make port-lib-depends-check
kile-2.9.92(editors/kile):
Bogus WANTLIB: poppler.47 (/usr/local/lib/libkdeinit5_kile.so) (NOT 
REACHABLE)
Extra:  phonon4qt5.2

Starting kile for the first time runs the setup, which detects a missing
`convert' binary for converting PNG files.  Maybe you can add a note to
README or so to inform users about installing ImageMagick for this to
work properly?

Settings > Configure Kile... > Kile > Help > Location of Tex documentation
points to /usr/share/texmf/doc by default.  Can this be adjusted to
wherever our textlive packages install theirs?

I also see this on the console; is it normal that samba is involved
here?  Literal "\n" do not look sane to me, either.

QFSFileEngine::open: No file name specified
kf5.kio.core: We got some errors while running testparm "Load smb 
config files from /etc/samba/smb.conf\nLoaded services file OK.\nERROR: lock 
directory /var/run/samba does not exist\n\nWARNING: state directory /var/samba 
should have permissions 0755 for browsing to work\n\nWARNING: cache directory 
/var/cache/samba should have permissions 0755 for browsing to work\n\nERROR: 
pid directory /var/run/samba does not exist\n\n"
kf5.kio.core: We got some errors while running 'net usershare info'
kf5.kio.core: "mkdir failed on directory /var/run/samba: Permission 
denied\nFailed to init messaging context\n"

Re: [macppc] games/hyperrogue: fix build, but colors are off

[George Koehler]

On Sun, 13 Jan 2019 17:34:04 +0100
Charlene Wendling  wrote:

> Runtime is fine, but colors are off: the ice is red instead of
> blueish [1]. Anyone know where to start to look at? 

There's an endian problem in polygons.cpp:part().  I tried to fix it
with the patch below, but it didn't work.  My patch turned the ice
walls into invisible black tiles.

My PowerBook G4 with OpenBSD/macppc has 1G of RAM and uses more than
700M of swap to compile hyper.cpp, which includes most of the other
.cpp files.  This takes about an hour, so I can't easily test changes
to the code.

=begin patches/patch-polygons_cpp
$OpenBSD$

Fix color on big-endian machines.
clang or gcc defines __BIG_ENDIAN__.

Index: polygons.cpp
--- polygons.cpp.orig
+++ polygons.cpp
@@ -762,7 +762,12 @@ void fixMercator(bool tinf) {
   }
   
 unsigned char& part(color_t& col, int i) {
-  unsigned char* c = (unsigned char*)  return c[i];
+  unsigned char* c = (unsigned char*) 
+#ifdef __BIG_ENDIAN__
+  return c[sizeof(col) - i];
+#else
+  return c[i];
+#endif
   }
 
 bool in_twopoint = false;
=end

-- 
George Koehler 

Re: please help fixing php-fpm on armv7 - egdb result

[s_graf]

I gather the warnings about wrong libraries did not provide any clues.
Running the debugger will be a challenge for me, but I will give it a try.  
What needs to be rebuilt with debug enabled?  The more detailed the 
instructions the better.
It is the child processes that are failing.

-Original Message-
From: owner-po...@openbsd.org  On Behalf Of Stuart 
Henderson
Sent: January 12, 2019 11:10 AM
To: Karel Gardas 
Cc: s_g...@telus.net; ports@openbsd.org; a...@openbsd.org; 'Jeremie 
Courreges-Anglas' 
Subject: Re: please help fixing php-fpm on armv7 - egdb result

On 2019/01/12 16:41, Karel Gardas wrote:
> On Fri, 11 Jan 2019 17:02:25 -0800
>  wrote:
> 
> > Thank you Jeremie for your suggestion.  I built  the gdb package and ran 
> > egdb with the result below.  I hope this provides some clues.
> > 
> > op1bsdsnap1228# egdb /usr/local/sbin/php-fpm-7.2 /root/php-fpm-7.2.core
> > GNU gdb (GDB) 7.12.1
> > Copyright (C) 2017 Free Software Foundation, Inc.
> > License GPLv3+: GNU GPL version 3 or later 
> > 
> > This is free software: you are free to change and redistribute it.
> > There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
> > and "show warranty" for details.
> > This GDB was configured as "arm-unknown-openbsd6.4".
> > Type "show configuration" for configuration details.
> > For bug reporting instructions, please see:
> > .
> > Find the GDB manual and other documentation resources online at:
> > .
> > For help, type "help".
> > Type "apropos word" to search for commands related to "word"...
> > Reading symbols from /usr/local/sbin/php-fpm-7.2...(no debugging symbols 
> > found)...done.
> > [New process 446218]
> > 
> > warning: .dynamic section for "/usr/lib/libcrypto.so.45.1" is not at the 
> > expected address (wrong library or version mismatch?)
> > 
> > warning: .dynamic section for "/usr/lib/libc.so.93.0" is not at the 
> > expected address (wrong library or version mismatch?)
> > Core was generated by `php-fpm-7.2'.
> > Program terminated with signal SIGSEGV, Segmentation fault.
> > #0  0xdfdfdfde in ?? ()
> > (gdb)
> 
> Here ^ you should type "bt" and hit Enter and this way you will see 
> stacktrace of the crash. Also if you build php-fpm-7.2 with debugging 
> enabled, it will help too probably since you will/should see line numbers etc.

That is not how a healthy stack looks. It's highly unlikely you will get
anything useful from a backtrace of this coredump.

I think it's likely to need one or a combination of:

- enable as much debug logging as possible
- add extra debugging logs/printfs
- running from the start under a debugger and stepping through to figure
out codepaths leading to the crash

to try to figure out what's going on.

I'd take a look if I had hardware though I don't know if I would be able
to figure it out (I haven't got anywhere with the php-7.3 crashfest on
OpenBSD/amd64 either)..

Re: [MAINT UPDATE] gitsh 0.13

[Mike Burns]

On Fri Jan 11, 2019 at 10:09:15PM +, Mike Burns wrote:
> Update by port maintainer for devel/gitsh.

Now with REVISION removed (thanks Rafael Sadowski):

---

Index: Makefile
===
RCS file: /cvs/ports/devel/gitsh/Makefile,v
retrieving revision 1.9
diff -u -p -r1.9 Makefile
--- Makefile26 Oct 2018 16:16:24 -  1.9
+++ Makefile13 Jan 2019 19:00:55 -
@@ -1,11 +1,10 @@
 # $OpenBSD: Makefile,v 1.9 2018/10/26 16:16:24 jeremy Exp $
 
-VERSION =  0.12
+VERSION =  0.13
 
 COMMENT =  interactive shell for git
 
 DISTNAME = gitsh-${VERSION}
-REVISION = 0
 
 CATEGORIES =   devel
 
Index: distinfo
===
RCS file: /cvs/ports/devel/gitsh/distinfo,v
retrieving revision 1.5
diff -u -p -r1.5 distinfo
--- distinfo2 Feb 2018 21:08:09 -   1.5
+++ distinfo13 Jan 2019 19:00:55 -
@@ -1,2 +1,2 @@
-SHA256 (gitsh-0.12.tar.gz) = c9glH2b0Ow4sHrzIBH8w1AXSRfU/o1yp7ZbP4L6tlRA=
-SIZE (gitsh-0.12.tar.gz) = 1152133
+SHA256 (gitsh-0.13.tar.gz) = bDjbU4d/LESEAg8GgaRPEaKc5NHh2ssGK65JwvLDoqA=
+SIZE (gitsh-0.13.tar.gz) = 1180908
Index: pkg/PLIST
===
RCS file: /cvs/ports/devel/gitsh/pkg/PLIST,v
retrieving revision 1.6
diff -u -p -r1.6 PLIST
--- pkg/PLIST   6 Feb 2018 10:15:05 -   1.6
+++ pkg/PLIST   13 Jan 2019 19:00:55 -
@@ -1,6 +1,7 @@
 @comment $OpenBSD: PLIST,v 1.6 2018/02/06 10:15:05 sthen Exp $
 @bin bin/gitsh
 @man man/man1/gitsh.1
+@man man/man5/gitsh_completions.5
 share/gitsh/
 share/gitsh/ruby/
 share/gitsh/ruby/gitsh.rb
@@ -23,12 +24,11 @@ share/gitsh/ruby/lib/gitsh/commands/inte
 share/gitsh/ruby/lib/gitsh/commands/noop.rb
 share/gitsh/ruby/lib/gitsh/commands/shell_command.rb
 share/gitsh/ruby/lib/gitsh/commands/tree.rb
-share/gitsh/ruby/lib/gitsh/completer.rb
-share/gitsh/ruby/lib/gitsh/completion_escaper.rb
 share/gitsh/ruby/lib/gitsh/environment.rb
 share/gitsh/ruby/lib/gitsh/error.rb
 share/gitsh/ruby/lib/gitsh/exit_statuses.rb
 share/gitsh/ruby/lib/gitsh/file_runner.rb
+share/gitsh/ruby/lib/gitsh/git_command_list.rb
 share/gitsh/ruby/lib/gitsh/git_repository/
 share/gitsh/ruby/lib/gitsh/git_repository.rb
 share/gitsh/ruby/lib/gitsh/git_repository/status.rb
@@ -51,6 +51,39 @@ share/gitsh/ruby/lib/gitsh/prompt_color.
 share/gitsh/ruby/lib/gitsh/prompter.rb
 share/gitsh/ruby/lib/gitsh/quote_detector.rb
 share/gitsh/ruby/lib/gitsh/shell_command_runner.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/
+share/gitsh/ruby/lib/gitsh/tab_completion/automaton.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/automaton_factory.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/command_completer.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/context.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/dsl/
+share/gitsh/ruby/lib/gitsh/tab_completion/dsl.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/dsl/choice_factory.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/dsl/concatenation_factory.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/dsl/lexer.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/dsl/maybe_operation_factory.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/dsl/null_factory.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/dsl/option_transition_factory.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/dsl/parser.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/dsl/plus_operation_factory.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/dsl/rule_factory.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/dsl/rule_set_factory.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/dsl/star_operation_factory.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/dsl/text_transition_factory.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/dsl/variable_transition_factory.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/escaper.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/facade.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/matchers/
+share/gitsh/ruby/lib/gitsh/tab_completion/matchers/anything_matcher.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/matchers/base_matcher.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/matchers/command_matcher.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/matchers/path_matcher.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/matchers/remote_matcher.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/matchers/revision_matcher.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/matchers/text_matcher.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/matchers/unknown_option_matcher.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/variable_completer.rb
+share/gitsh/ruby/lib/gitsh/tab_completion/visualization.rb
 share/gitsh/ruby/lib/gitsh/terminal.rb
 share/gitsh/ruby/lib/gitsh/version.rb
 share/gitsh/vendor/
@@ -135,7 +168,7 @@ share/gitsh/vendor/gems/ffi-1.9.17/ext/f
 share/gitsh/vendor/gems/ffi-1.9.17/ext/ffi_c/libffi/configure.ac
 share/gitsh/vendor/gems/ffi-1.9.17/ext/ffi_c/libffi/configure.host
 

Re: UPDATE: LLVM

[Jeremie Courreges-Anglas]

On Sun, Jan 13 2019, Brad Smith  wrote:
> Sync in bits from base.

Thanks, tests ongoing.

While here: I plan to work on an update to llvm-7.0.1 soonish (like,
in the next couple of days).  If you or other folks already have a diff,
please let me know. :)

-- 
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE

UPDATE: LLVM

[Brad Smith]

Sync in bits from base.


Index: Makefile
===
RCS file: /home/cvs/ports/devel/llvm/Makefile,v
retrieving revision 1.204
diff -u -p -u -p -r1.204 Makefile
--- Makefile5 Jan 2019 11:38:49 -   1.204
+++ Makefile12 Jan 2019 00:30:58 -
@@ -20,9 +20,9 @@ PKGSPEC-main =llvm-=${LLVM_V}
 PKGNAME-main = llvm-${LLVM_V}
 PKGNAME-python =   py-llvm-${LLVM_V}
 PKGNAME-lldb = lldb-${LLVM_V}
-REVISION-main =22
-REVISION-python = 4
-REVISION-lldb =1
+REVISION-main =23
+REVISION-python = 5
+REVISION-lldb =2
 CATEGORIES =   devel
 DISTFILES =llvm-${LLVM_V}.src${EXTRACT_SUFX} \
cfe-${LLVM_V}.src${EXTRACT_SUFX} \
@@ -87,6 +87,15 @@ CONFIGURE_ARGS = -DLLVM_ENABLE_FFI:Bool=
-DLLVM_BUILD_LLVM_DYLIB:Bool=True \
-DGO_EXECUTABLE=GO_EXECUTABLE-NOTFOUND \
-DBacktrace_LIBRARY=''
+
+# Disable some protections in the compiler to regain performance
+.if ${MACHINE_ARCH} == "aarch64" || ${MACHINE_ARCH} == "amd64" || \
+${MACHINE_ARCH} == "i386"
+CXXFLAGS +=-fno-ret-protector
+.endif
+.if ${MACHINE_ARCH} == "amd64" || ${MACHINE_ARCH} == "i386"
+CXXFLAGS +=-mno-retpoline
+.endif
 
 # Workaround relocation overflow
 .if ${MACHINE_ARCH} == "powerpc"
Index: patches/patch-lib_Target_X86_X86Subtarget_cpp
===
RCS file: patches/patch-lib_Target_X86_X86Subtarget_cpp
diff -N patches/patch-lib_Target_X86_X86Subtarget_cpp
--- /dev/null   1 Jan 1970 00:00:00 -
+++ patches/patch-lib_Target_X86_X86Subtarget_cpp   13 Jan 2019 19:37:30 
-
@@ -0,0 +1,22 @@
+$OpenBSD$
+
+Turn on -mretpoline by default in clang on amd64.
+
+Index: lib/Target/X86/X86Subtarget.cpp
+--- lib/Target/X86/X86Subtarget.cpp.orig
 lib/Target/X86/X86Subtarget.cpp
+@@ -207,6 +207,14 @@ void X86Subtarget::initSubtargetFeatures(StringRef CPU
+   FullFS = "+sahf";
+   }
+ 
++  // OpenBSD/amd64 defaults to -mretpoline.
++  if (isTargetOpenBSD() && In64BitMode) {
++if (!FullFS.empty())
++  FullFS = "+retpoline," + FullFS;
++else
++  FullFS = "+retpoline";
++  }
++
+   // Parse features string and set the CPU.
+   ParseSubtargetFeatures(CPUName, FullFS);
+ 
Index: patches/patch-lib_Target_X86_X86Subtarget_h
===
RCS file: patches/patch-lib_Target_X86_X86Subtarget_h
diff -N patches/patch-lib_Target_X86_X86Subtarget_h
--- /dev/null   1 Jan 1970 00:00:00 -
+++ patches/patch-lib_Target_X86_X86Subtarget_h 13 Jan 2019 19:37:40 -
@@ -0,0 +1,15 @@
+$OpenBSD$
+
+Turn on -mretpoline by default in clang on amd64.
+
+Index: lib/Target/X86/X86Subtarget.h
+--- lib/Target/X86/X86Subtarget.h.orig
 lib/Target/X86/X86Subtarget.h
+@@ -603,6 +603,7 @@ class X86Subtarget final : public X86GenSubtargetInfo 
+ 
+   bool isTargetDarwin() const { return TargetTriple.isOSDarwin(); }
+   bool isTargetFreeBSD() const { return TargetTriple.isOSFreeBSD(); }
++  bool isTargetOpenBSD() const { return TargetTriple.isOSOpenBSD(); }
+   bool isTargetDragonFly() const { return TargetTriple.isOSDragonFly(); }
+   bool isTargetSolaris() const { return TargetTriple.isOSSolaris(); }
+   bool isTargetPS4() const { return TargetTriple.isPS4CPU(); }
Index: patches/patch-tools_clang_include_clang_Analysis_Analyses_FormatString_h
===
RCS file: 
patches/patch-tools_clang_include_clang_Analysis_Analyses_FormatString_h
diff -N patches/patch-tools_clang_include_clang_Analysis_Analyses_FormatString_h
--- /dev/null   1 Jan 1970 00:00:00 -
+++ patches/patch-tools_clang_include_clang_Analysis_Analyses_FormatString_h
12 Jan 2019 22:57:25 -
@@ -0,0 +1,22 @@
+$OpenBSD$
+
+The %b printf extension in the kernel is not fixed to a int type. On sparc64
+there are various %llb formats. Adjust the code to handle the length specifiers
+and type check like it is used by the regular case.
+
+Index: tools/clang/include/clang/Analysis/Analyses/FormatString.h
+--- tools/clang/include/clang/Analysis/Analyses/FormatString.h.orig
 tools/clang/include/clang/Analysis/Analyses/FormatString.h
+@@ -227,8 +227,10 @@ class ConversionSpecifier { (public)
+ 
+   bool isIntArg() const { return (kind >= IntArgBeg && kind <= IntArgEnd) ||
+ kind == FreeBSDrArg || kind == FreeBSDyArg; }
+-  bool isUIntArg() const { return kind >= UIntArgBeg && kind <= UIntArgEnd; }
+-  bool isAnyIntArg() const { return kind >= IntArgBeg && kind <= UIntArgEnd; }
++  bool isUIntArg() const { return (kind >= UIntArgBeg && kind <= UIntArgEnd) 
||
++kind == FreeBSDbArg; }
++  bool isAnyIntArg() const { return (kind >= IntArgBeg && kind <= UIntArgEnd) 
||
++kind == FreeBSDbArg; }
+   bool isDoubleArg() const {
+ return kind >= DoubleArgBeg && kind <= DoubleArgEnd;
+   }
Index: 

Re: UPDATE: math/octave

[Klemens Nanni]

On Thu, Jan 10, 2019 at 09:42:48PM +0100, Steven Mestdagh wrote:
> maintenance update, seems to work fine on amd64.
> test suite shows 2 failures less than with the previous version.
Keeps working fine on amd64, thanks.

> comments / ok?

$ portcheck
trailing whitespace in pkg/DESCR
missing RDEP on x11/gtk+3,-guic
math/octave

$ make port-lib-depends-check
octave-4.4.1(math/octave):
Missing: btf.2 from suitesparse-4.4.6p0 
(/usr/local/lib/octave/4.4.1/oct/x86_64-unknown-openbsd6.4/__ode15__.oct)
Missing: klu.2 from suitesparse-4.4.6p0 
(/usr/local/lib/octave/4.4.1/oct/x86_64-unknown-openbsd6.4/__ode15__.oct)
Missing: png16.17 from png-1.6.35 
(/usr/local/lib/octave/4.4.1/liboctinterp.so.10.0)
Missing: xcb-dri3.0 
(/usr/local/libexec/octave/4.4.1/exec/x86_64-unknown-openbsd6.4/octave-gui) 
(system lib)
Missing: xcb-present.0 
(/usr/local/libexec/octave/4.4.1/exec/x86_64-unknown-openbsd6.4/octave-gui) 
(system lib)
Missing: xcb-sync.1 
(/usr/local/libexec/octave/4.4.1/exec/x86_64-unknown-openbsd6.4/octave-gui) 
(system lib)
Missing: xcb-xfixes.1 
(/usr/local/libexec/octave/4.4.1/exec/x86_64-unknown-openbsd6.4/octave-gui) 
(system lib)
Missing: xshmfence.0 
(/usr/local/libexec/octave/4.4.1/exec/x86_64-unknown-openbsd6.4/octave-gui) 
(system lib)
Extra:  png.17
WANTLIB += btf klu png16 xcb-dri3 xcb-present xcb-sync xcb-xfixes
WANTLIB += xshmfence
*** Error 1 in target 'port-lib-depends-check' (ignored)

Please sort BDEP; I'd say you can just merge both lines together as well.

Re: UPDATE: LLVM

[Brad Smith]

On 1/13/2019 3:32 PM, Jeremie Courreges-Anglas wrote:


On Sun, Jan 13 2019, Brad Smith  wrote:

Sync in bits from base.

Thanks, tests ongoing.

While here: I plan to work on an update to llvm-7.0.1 soonish (like,
in the next couple of days).  If you or other folks already have a diff,
please let me know. :)


I have had a diff in my tree for awhile but I would have to update / 
re-merge it. Last
time I was playing around the compiler was crashing and I think it was 
something
from base that was causing the crashing, but at the time I didn't spend 
anytime looking
into it. It was more so to see how far I could get merging patches and 
getting it to

build as a first step.

Re: [new] gzdoom-3.6.0

[Solene Rapenne]

Solene Rapenne  wrote:
> Solene Rapenne  wrote:
> > Solene Rapenne  wrote:
> > > timo.my...@bittivirhe.fi (Timo Myyrä) wrote:
> > > > Jeremie Courreges-Anglas  writes:
> > > > 
> > > > > On Wed, Dec 19 2018, Solene Rapenne  wrote:
> > > > >
> > > > >> Frederic Cambus  wrote:
> > > > >>> On Sat, Dec 08, 2018 at 01:52:26PM +0200, Timo Myyrä wrote:
> > > > >>> > 
> > > > >>> > > Also, it seems that the two patches are unnecessary? And that 
> > > > >>> > > we can
> > > > >>> > > also remove the following line from the port Makefile?
> > > > >>> > > 
> > > > >>> > > CONFIGURE_ARGS +=   -DFLUIDSYNTHLIB1="libfluidsynth.so"
> > > > >>> > > 
> > > > >>> > > Lastly, I wonder if it wouldn't be better to link against 
> > > > >>> > > fluidsynth,
> > > > >>> > > add audio/generaluser-gs-soundfont in RUN_DEPENDS, and finally 
> > > > >>> > > patch
> > > > >>> > > src/gameconfigfile.cpp so it automatically finds the SoundFont 
> > > > >>> > > banks.
> > > > >>> > > This would result in a fully functional port playing music out 
> > > > >>> > > of the
> > > > >>> > > box, without requiring any manual fiddling. Any thoughts on 
> > > > >>> > > this?
> > > > >>> > > 
> > > > >>> > 
> > > > >>> > Yeah, the patches are only there to keep fluidsynth as optional 
> > > > >>> > dependency. 
> > > > >>> 
> > > > >>> When testing, it worked without patches both when linking against
> > > > >>> fluidsynth and when keeping it as an optional dependency.
> > > > >
> > > > > By default, ie without any patch, "libfluidsynth.so.1" is dlopened.
> > > > > This works because that's the major version currently shipped by our
> > > > > fluidsynth port:
> > > > >
> > > > >   SHARED_LIBS +=  fluidsynth   1.0  # 6.2
> > > > >
> > > > > Timo patched the port so that the dlopen'd name is "libfluidsynth.so"
> > > > > with no version; this allows supporting fluidsynth as an optional 
> > > > > sound
> > > > > backend, without having to sync gzdoom with the major bumps of
> > > > > libfluidsynth.  I think this is the right way to handle it.
> > > > >
> > > > >>> > The music works without fluidsynth, fluidsynth sounds a bit 
> > > > >>> > better.
> > > > >>> > If we add fluidsynth as mandatory dependency, how about timidity?
> > > > >>> > Thats why I added only those that are strictly needed to run the
> > > > >>> > game as dependency. The game could probably be patched so that it
> > > > >>> > only shows the available sound / music backends in the menus.
> > > > >>> 
> > > > >>> I see, I was indeed able to get music playing without fluidsynth by
> > > > >>> selecting the OPL synth emulation, it wasn't enabled by default.
> > > > >>> 
> > > > >>> I don't think patching the game to only show available backends 
> > > > >>> would
> > > > >>> be worth it, but if it's not too much work, having OPL synth 
> > > > >>> emulation
> > > > >>> be the default would be nice.
> > > > >>
> > > > >> updated tarball.
> > > > >>
> > > > >> - added stuff to pkg/README from prboom README
> > > > >> - removed BUILD_DEPENDS
> > > > >> - removed CONFIGURE_ARGS +=  -DFLUIDSYNTHLIB1="libfluidsynth.so"
> > > > >
> > > > > If you remove -DFLUIDSYNTHLIB1="libfluidsynth.so" then the value from
> > > > > patch-src_CMakeLists_txt is used:
> > > > >
> > > > >  ... -DDYN_FLUIDSYNTH -DFLUIDSYNTHLIB1=\"libfluidsynth.so.0\"
> > > > >
> > > > > so fluidsynth support *won't* work.
> > > > 
> > > > Update to gzdoom-3.7.0, upstream added jit support which requires 
> > > > execinfo as
> > > > build depends.
> > > > 
> > > > timo
> > > 
> 
> I finally found the origin of the issue when loading brutal doom (and some
> others mods I hope). When loading assets and making animations, it looks for
> duplicates.
> 
> I don't fully understand the logic here, I read that in case of duplicate,
> it free the old one and replace it with the current object?
> 
> It's a dirty hack but commenting the free instruction permit to play mods
> (having duplicates in their animations I think?) with no noticeable memory
> usage change. I added a puts("duplicate") to see if this happens often, it
> was like 7 duplicates for brutal doom. As it's only happening at load time,
> I would say it's pretty safe to not free this.
> 
> What do you think?
> 
> Index: src/textures/animations.cpp
> --- src/textures/animations.cpp.orig
> +++ src/textures/animations.cpp
> @@ -73,7 +73,7 @@ FAnimDef *FTextureManager::AddAnim (FAnimDef *anim)
>   if (mAnimations[i]->BasePic == anim->BasePic)
>   {
>   // Found one!
> - free (mAnimations[i]);
> + //free (mAnimations[i]);
>   mAnimations[i] = anim;
>   return anim;
>   }

Last version in tarball, including my patch. I've been able to play and did not
encounter any issue. From my code reading, it will wastes a few MB of memory if
a mod make heavy use of animated textures and have duplicates.



gzdoom.tgz
Description: 

CVS: cvs.openbsd.org: ports

[Antoine Jacoutot]

CVSROOT:/cvs
Module name:ports
Changes by: ajacou...@cvs.openbsd.org   2019/01/13 01:14:43

Modified files:
sysutils/awscli: Makefile distinfo 

Log message:
Update to awscli-1.16.88.

CVS: cvs.openbsd.org: ports

[Antoine Jacoutot]

CVSROOT:/cvs
Module name:ports
Changes by: ajacou...@cvs.openbsd.org   2019/01/13 01:14:25

Modified files:
net/py-botocore: Makefile distinfo 

Log message:
Update to py-botocore-1.12.78.

CVS: cvs.openbsd.org: ports

[Antoine Jacoutot]

CVSROOT:/cvs
Module name:ports
Changes by: ajacou...@cvs.openbsd.org   2019/01/13 01:14:12

Modified files:
net/py-boto3   : Makefile distinfo 

Log message:
Update to py-boto3-1.9.78.

CVS: cvs.openbsd.org: ports

[Daniel Jakots]

CVSROOT:/cvs
Module name:ports
Changes by: d...@cvs.openbsd.org2019/01/13 02:54:56

Modified files:
devel/py-greenlet: Makefile distinfo 

Log message:
Update to py-greenlet-0.4.15

>From Pamela Mosiejczuk

CVS: cvs.openbsd.org: ports

[Antoine Jacoutot]

CVSROOT:/cvs
Module name:ports
Changes by: ajacou...@cvs.openbsd.org   2019/01/13 05:26:29

Modified files:
devel/meson/patches: patch-mesonbuild_environment_py 

Log message:
Merged upstream.

CVS: cvs.openbsd.org: ports

[Rafael Sadowski]

CVSROOT:/cvs
Module name:ports
Changes by: rsadow...@cvs.openbsd.org   2019/01/13 05:32:15

Modified files:
x11/qt5/qtbase : Makefile 
Added files:
x11/qt5/qtbase/patches: patch-mkspecs_common_gcc-base_conf 

Log message:
zap -O3 in QMAKE_CFLAGS_OPTIMIZE and respect CXXFLAGS

"go for it" sthen@

CVS: cvs.openbsd.org: ports

[Rafael Sadowski]

CVSROOT:/cvs
Module name:ports
Changes by: rsadow...@cvs.openbsd.org   2019/01/13 05:48:10

Modified files:
x11/i3 : Makefile distinfo 
x11/i3/patches : patch-Makefile_in patch-configure 
 patch-i3-nagbar_main_c 
 patch-man_i3-config-wizard_man 
 patch-man_i3-nagbar_man patch-man_i3_man 
 patch-src_bindings_c patch-src_manage_c 

Log message:
Update i3 to 4.16

Maintainer timeout

feedback and ok tb, tested from Josh Grosse

CVS: cvs.openbsd.org: ports

[Antoine Jacoutot]

CVSROOT:/cvs
Module name:ports
Changes by: ajacou...@cvs.openbsd.org   2019/01/13 02:43:05

Modified files:
x11/gtk+3  : Makefile 
Added files:
x11/gtk+3/patches: patch-gdk_x11_gdkwindow-x11_c 

Log message:
x11: Fix deprecation macro use which could lead to a crash (upstream)

CVS: cvs.openbsd.org: ports

[Antoine Jacoutot]

CVSROOT:/cvs
Module name:ports
Changes by: ajacou...@cvs.openbsd.org   2019/01/13 01:18:36

Modified files:
sysutils/nomad/patches: 

patch-vendor_github_com_shirou_gopsutil_cpu_cpu_openbsd_go 

Log message:
Merged upstream.

CVS: cvs.openbsd.org: ports

[Antoine Jacoutot]

CVSROOT:/cvs
Module name:ports
Changes by: ajacou...@cvs.openbsd.org   2019/01/13 03:20:18

Modified files:
devel/meson: Makefile 
Added files:
devel/meson/patches: patch-mesonbuild_environment_py 

Log message:
Put 'macppc' into the 'ppc' cpu family.

CVS: cvs.openbsd.org: ports

[Antoine Jacoutot]

CVSROOT:/cvs
Module name:ports
Changes by: ajacou...@cvs.openbsd.org   2019/01/13 01:06:10

Modified files:
www/py-bokeh   : Makefile distinfo 

Log message:
Update to py-bokeh-1.0.4.

CVS: cvs.openbsd.org: ports

[Antoine Jacoutot]

CVSROOT:/cvs
Module name:ports
Changes by: ajacou...@cvs.openbsd.org   2019/01/13 02:21:17

Modified files:
x11/gnome/builder: Makefile distinfo 

Log message:
Update to gnome-builder-3.30.3.

CVS: cvs.openbsd.org: ports

[Pascal Stumpf]

CVSROOT:/cvs
Module name:ports
Changes by: pas...@cvs.openbsd.org  2019/01/13 04:59:18

Modified files:
lang/gcc/8 : Makefile 

Log message:
Fix TEST_DEPENDS on arm and arm64.  From Brad.

CVS: cvs.openbsd.org: ports

[Charlene Wendling]

CVSROOT:/cvs
Module name:ports
Changes by: c...@cvs.openbsd.org2019/01/13 07:16:40

Modified files:
devel/p5-Test-LeakTrace: Makefile distinfo 
devel/p5-Test-LeakTrace/pkg: DESCR 

Log message:
Update p5-Test-LeakTrace to 0.16
Changelog:
https://metacpan.org/source/LEEJO/Test-LeakTrace-0.16/Changes
OK bcallah@ afresh1@

CVS: cvs.openbsd.org: ports

[Christian Weisgerber]

CVSROOT:/cvs
Module name:ports
Changes by: na...@cvs.openbsd.org   2019/01/13 08:34:55

Modified files:
archivers/gtar : Makefile distinfo 
archivers/gtar/patches: patch-configure 
Added files:
archivers/gtar/patches: patch-src_extract_c 
patch-tests_dirrem01_at 
patch-tests_dirrem02_at 
Removed files:
archivers/gtar/patches: patch-gnu_error_c patch-rmt_rmt_c 
patch-tests_difflink_at 

Log message:
Update to 1.31:
* Support for zstd compression (if zstd is installed).
* The -K option interacts properly with member names given on the command line.
* Fix handling of file shrinkage during read access when --sparse is used.

CVS: cvs.openbsd.org: ports

[Kirill Bychkov]

CVSROOT:/cvs
Module name:ports
Changes by: ki...@cvs.openbsd.org   2019/01/13 09:40:20

Added files:
devel/automake/1.16/patches: patch-Makefile_in 

Log message:
fix parallel build. from FreeBSD.
OK rsadowski@, kn@

CVS: cvs.openbsd.org: ports

[Pascal Stumpf]

CVSROOT:/cvs
Module name:ports
Changes by: pas...@cvs.openbsd.org  2019/01/13 08:05:33

Added files:
inputmethods/uim/patches: 
  
patch-sigscheme_libgcroots_include_private_gcconfig_h 

Log message:
Add aarch64 support to the internal copy of boehm-gc; from upstream and our
devel/boehm-gc port.  Unlocks a bunch of ports in a bulk.

ok phessler@

CVS: cvs.openbsd.org: ports

[Rafael Sadowski]

CVSROOT:/cvs
Module name:ports
Changes by: rsadow...@cvs.openbsd.org   2019/01/13 07:00:56

Modified files:
devel/gitsh: Makefile distinfo 
devel/gitsh/pkg: PLIST 

Log message:
Update gitsh to 0.13

Maintainer update with REVISION tweak from me.

CVS: cvs.openbsd.org: ports

[Brian Callahan]

CVSROOT:/cvs
Module name:ports
Changes by: bcal...@cvs.openbsd.org 2019/01/13 07:00:50

Modified files:
sysutils/bfs   : Makefile distinfo 
Added files:
sysutils/bfs/patches: patch-util_h 

Log message:
Bugfix update to bfs-1.3.2
Changelog: https://github.com/tavianator/bfs/compare/1.3.1...1.3.2
While here, upstream assumes that a sys/acl.h header exists which can break
the build so turn that assumption off.
Reported by landry@ from the latest powerpc bulk build, patch tested by
cwen@

CVS: cvs.openbsd.org: ports

[Charlene Wendling]

CVSROOT:/cvs
Module name:ports
Changes by: c...@cvs.openbsd.org2019/01/13 07:28:11

Modified files:
textproc/p5-PPI: Makefile distinfo 
textproc/p5-PPI/pkg: PLIST 

Log message:
Update p5-PPI to 1.236
Changelog:
https://metacpan.org/changes/release/MITHALDU/PPI-1.236
OK bcallah@ afresh1@

CVS: cvs.openbsd.org: ports

[Charlene Wendling]

CVSROOT:/cvs
Module name:ports
Changes by: c...@cvs.openbsd.org2019/01/13 07:52:14

Modified files:
devel  : Makefile 

Log message:
+p5-Test-Compile

CVS: cvs.openbsd.org: ports

[Pascal Stumpf]

CVSROOT:/cvs
Module name:ports
Changes by: pas...@cvs.openbsd.org  2019/01/13 08:08:01

Modified files:
devel/ode  : Makefile 
devel/ode/patches: patch-ode_demo_Makefile_am 
Added files:
devel/ode/patches: patch-ode_pc_in 

Log message:
Use CXX for linking instead of hardcoding -lstdc++.  Fixes build on aarch64.

ok kirby@ (maintainer)

CVS: cvs.openbsd.org: ports

[Charlene Wendling]

CVSROOT:/cvs
Module name:ports
Changes by: c...@cvs.openbsd.org2019/01/13 08:19:52

Modified files:
x11/bbpager: Makefile 
Added files:
x11/bbpager/patches: patch-src_Baseresource_cxx 
 patch-src_main_cxx patch-src_resource_cxx 

Log message:
Add missing headers for ports-gcc
"OK if it builds for you" landry@, tweaks and OK naddy@, OK bcallah@

CVS: cvs.openbsd.org: ports

[Pascal Stumpf]

CVSROOT:/cvs
Module name:ports
Changes by: pas...@cvs.openbsd.org  2019/01/13 08:19:20

Modified files:
net/nyx: Makefile distinfo 

Log message:
Update to nyx 2.1.0.

CVS: cvs.openbsd.org: ports

[Charlene Wendling]

CVSROOT:/cvs
Module name:ports
Changes by: c...@cvs.openbsd.org2019/01/13 07:23:12

Modified files:
devel/p5-Test-SubCalls: Makefile distinfo 

Log message:
Update p5-Test-SubCalls to 1.10
Changelog:
https://metacpan.org/source/ETHER/Test-SubCalls-1.10/Changes
OK bcallah@ afresh1@

CVS: cvs.openbsd.org: ports

[Charlene Wendling]

CVSROOT:/cvs
Module name:ports
Changes by: c...@cvs.openbsd.org2019/01/13 07:33:42

Modified files:
textproc/p5-PPIx-Regexp: Makefile distinfo 
textproc/p5-PPIx-Regexp/pkg: PLIST 

Log message:
Update p5-PPIx-Regexp to 0.063
Changelog:
https://metacpan.org/source/WYANT/PPIx-Regexp-0.063/Changes
OK bcallah@ afresh1@

CVS: cvs.openbsd.org: ports

[Charlene Wendling]

CVSROOT:/cvs
Module name:ports
Changes by: c...@cvs.openbsd.org2019/01/13 07:50:04

Log message:
Import devel/p5-Test-Compile-1.3.0 and taking MAINTAINER.
OK afresh1@ giovanni@ bcallah@

This module lets you check the whether your Perl modules and scripts
compile properly, and reports its results in standard Test::Simple
fashion.

Status:

Vendor Tag: cwen
Release Tags:   cwen_20190113

N ports/devel/p5-Test-Compile/Makefile
N ports/devel/p5-Test-Compile/distinfo
N ports/devel/p5-Test-Compile/pkg/DESCR
N ports/devel/p5-Test-Compile/pkg/PLIST

No conflicts created by this import

CVS: cvs.openbsd.org: ports

[Charlene Wendling]

CVSROOT:/cvs
Module name:ports
Changes by: c...@cvs.openbsd.org2019/01/13 07:56:01

Modified files:
devel/p5-FileHandle-Unget: Makefile distinfo 

Log message:
Update p5-FileHandle-Unget to 0.1634
Changelog:
https://metacpan.org/source/DCOPPIT/FileHandle-Unget-0.1634/CHANGES
OK giovanni@ (MAINTAINER) afresh1@ bcallah@

CVS: cvs.openbsd.org: ports

[Jeremie Courreges-Anglas]

CVSROOT:/cvs
Module name:ports
Changes by: j...@cvs.openbsd.org2019/01/13 16:18:42

Modified files:
databases/mongodb: Makefile 
databases/mongodb/patches: 
   patch-src_mongo_util_processinfo_openbsd_cpp 

Log message:
Unbreak after mincore(2) removal

Follow upstream and assume that we should yield if there's a chance that
the block isn't mapped.  Hopefully the assumption doesn't have
a negative impact on runtime (besides slightly slower operations).

Not waiting for maintainer, the assumption is that a non-optimal package
is better than no package at all.

CVS: cvs.openbsd.org: ports

[Steven Mestdagh]

CVSROOT:/cvs
Module name:ports
Changes by: ste...@cvs.openbsd.org  2019/01/13 15:43:59

Modified files:
math/octave: Makefile distinfo 
math/octave/patches: patch-configure 
 patch-scripts_pkg_private_configure_make_m 
math/octave/pkg: DESCR PLIST 

Log message:
maintenance update to 4.4.1

test and feedback on port consistency from kn@
ok rsadowski@

CVS: cvs.openbsd.org: ports

[Tim van der Molen]

CVSROOT:/cvs
Module name:ports
Changes by: t...@cvs.openbsd.org2019/01/13 13:06:37

Modified files:
audio/siren: Makefile distinfo 

Log message:
Update to 0.9; OK ratchov (maintainer)

CVS: cvs.openbsd.org: ports

[Rafael Sadowski]

CVSROOT:/cvs
Module name:ports
Changes by: rsadow...@cvs.openbsd.org   2019/01/13 12:10:26

Modified files:
editors: Makefile 
meta/kde4  : Makefile 

Log message:
hook kile, unhook kile-kde4, adjust kde4-meta

CVS: cvs.openbsd.org: ports

[Rafael Sadowski]

CVSROOT:/cvs
Module name:ports
Changes by: rsadow...@cvs.openbsd.org   2019/01/13 11:59:41

Added files:
editors/kile   : Makefile distinfo 
editors/kile/pkg: DESCR PLIST 

Log message:
re-add kile-2.9.92 (kile-kde4 replacement)

ok kn@