Build failed in Jenkins: osv-build #1137

2016-11-03 Thread avi
See 

Changes:

[Nadav Har'El] Update apps/ to latest version

--
[...truncated 1227 lines...]
Adding /tests/misc-mmap-big-file.so...
Adding /tests/tst-mmap.so...
Adding /tests/tst-huge.so...
Adding /tests/tst-elf-permissions.so...
Adding /tests/misc-mutex.so...
Adding /tests/misc-sockets.so...
Adding /tests/tst-condvar.so...
Adding /tests/tst-queue-mpsc.so...
Adding /tests/tst-af-local.so...
Adding /tests/tst-pipe.so...
Adding /tests/tst-yield.so...
Adding /tests/misc-ctxsw.so...
Adding /tests/tst-readdir.so...
Adding /tests/tst-read.so...
Adding /tests/tst-symlink.so...
Adding /tests/tst-openat.so...
Adding /tests/tst-eventfd.so...
Adding /tests/tst-remove.so...
Adding /tests/misc-wake.so...
Adding /tests/tst-epoll.so...
Adding /tests/misc-lfring.so...
Adding /tests/misc-fsx.so...
Adding /tests/tst-sleep.so...
Adding /tests/tst-resolve.so...
Adding /tests/tst-except.so...
Adding /tests/misc-tcp-sendonly.so...
Adding /tests/tst-tcp-nbwrite.so...
Adding /tests/misc-tcp-hash-srv.so...
Adding /tests/misc-loadbalance.so...
Adding /tests/misc-scheduler.so...
Adding /tests/tst-console.so...
Adding /tests/tst-app.so...
Adding /tests/misc-setpriority.so...
Adding /tests/misc-timeslice.so...
Adding /tests/misc-tls.so...
Adding /tests/misc-gtod.so...
Adding /tests/tst-dns-resolver.so...
Adding /tests/tst-fs-link.so...
Adding /tests/tst-kill.so...
Adding /tests/tst-truncate.so...
Adding /tests/misc-panic.so...
Adding /tests/tst-utimes.so...
Adding /tests/tst-utimensat.so...
Adding /tests/tst-futimesat.so...
Adding /tests/misc-tcp.so...
Adding /tests/tst-strerror_r.so...
Adding /tests/misc-random.so...
Adding /tests/misc-urandom.so...
Adding /tests/tst-commands.so...
Adding /tests/tst-threadcomplete.so...
Adding /tests/tst-timerfd.so...
Adding /tests/tst-nway-merger.so...
Adding /tests/tst-memmove.so...
Adding /tests/tst-pthread-clock.so...
Adding /tests/misc-procfs.so...
Adding /tests/tst-chdir.so...
Adding /tests/tst-chmod.so...
Adding /tests/tst-hello.so...
Adding /tests/misc-concurrent-io.so...
Adding /tests/tst-concurrent-init.so...
Adding /tests/tst-ring-spsc-wraparound.so...
Adding /tests/tst-shm.so...
Adding /tests/tst-align.so...
Adding /tests/tst-cxxlocale.so...
Adding /tests/misc-tcp-close-without-reading.so...
Adding /tests/tst-sigwait.so...
Adding /tests/tst-sampler.so...
Adding /tests/misc-malloc.so...
Adding /tests/misc-memcpy.so...
Adding /tests/misc-free-perf.so...
Adding /tests/tst-fallocate.so...
Adding /tests/misc-printf.so...
Adding /tests/tst-hostname.so...
Adding /tests/tst-sendfile.so...
Adding /tests/misc-lock-perf.so...
Adding /tests/tst-uio.so...
Adding /tests/tst-printf.so...
Adding /tests/tst-pthread-affinity.so...
Adding /tests/tst-pthread-tsd.so...
Adding /tests/tst-thread-local.so...
Adding /tests/tst-zfs-mount.so...
Adding /tests/tst-regex.so...
Adding /tests/tst-tcp-siocoutq.so...
Adding /tests/libtls.so...
Adding /tests/tst-tls.so...
Adding /tests/tst-select-timeout.so...
Adding /tests/tst-faccessat.so...
Adding /tests/tst-fstatat.so...
Adding /tests/misc-reboot.so...
Adding /tests/tst-fcntl.so...
Adding /tests/payload-namespace.so...
Adding /tests/tst-namespace.so...
Adding /tests/tst-without-namespace.so...
Adding /tests/payload-env.so...
Adding /tests/payload-merge-env.so...
Adding /tests/misc-execve.so...
Adding /tests/misc-execve-payload.so...
Adding /tests/misc-mutex2.so...
Adding /tests/tst-pthread-setcancelstate.so...
Adding /tests/tst-syscall.so...
Adding /tests/tst-pin.so...
Adding /tests/tst-run.so...
Adding /tests/tst-ifaddrs.so...
Adding /tests/testrunner.so...
Adding /tests/tst-rename.so...
Adding /tests/tst-vfs.so...
Adding /tests/tst-libc-locking.so...
Adding /tests/misc-fs-stress.so...
Adding /tests/misc-bdev-write.so...
Adding /tests/misc-bdev-wlatency.so...
Adding /tests/misc-bdev-rw.so...
Adding /tests/tst-promise.so...
Adding /tests/tst-dlfcn.so...
Adding /tests/tst-stat.so...
Adding /tests/tst-wait-for.so...
Adding /tests/tst-bsd-tcp1.so...
Adding /tests/tst-bsd-tcp1-zsnd.so...
Adding /tests/tst-bsd-tcp1-zrcv.so...
Adding /tests/tst-bsd-tcp1-zsndrcv.so...
Adding /tests/tst-async.so...
Adding /tests/tst-rcu-list.so...
Adding /tests/tst-tcp-listen.so...
Adding /tests/tst-poll.so...
Adding /tests/tst-bitset-iter.so...
Adding /tests/tst-timer-set.so...
Adding /tests/tst-clock.so...
Adding /tests/tst-rcu-hashtable.so...
Adding /tests/tst-unordered-ring-mpsc.so...
Adding /tests/tst-seek.so...
Adding /tests/tst-solaris-taskq.so...
Adding /tests/misc-zfs-io.so...
+ cd 
+ exec ./scripts/test.py
  TEST java-perms  OK  (1.253 s)
  TEST java_isolated   OK  (1.470 s)
  TEST java_non_isolated   OK  (2.262 s)
  TEST tcp_close_without_reading   OK  (1.115 s)
  TEST tracing_smoke_test  OK  (16.365 s)
  TEST tst-af-local.so   

Re: [Go runtime support v4 08/11] app: Use GoMain as go entry point

2016-11-03 Thread Nadav Har'El
On Thu, Oct 27, 2016 at 3:06 PM, BenoƮt Canet <
benoit.canet.cont...@gmail.com> wrote:

> Go c-shared library main() get ignored by the
> Go runtime at init. main() cannot be //export since the
> c-go .c file already have a main. Fact is that if main()
> is exported in a Go "C" module the C compiler will complain
> that two main are presents.
>
> So this patch declare GoMain as the official OSv go main()
> entry point.

Since the go library init already took argc and
> argv as parameters GoMain does not need to bother with them
> and hence is specific to Go.:
>

I wouldn't say that this is "specific to go" Sorry for the
resistance... It's just an entry point function to run without arguments
(assuming you get the arguments from somewhere else, or don't need them at
all). You're in fact reusing the concept of "entry_point" which already
existed in the code. So couldn't we just call this function entry_point(),
or osv_entry_point() or something, instead of GoMain? We can have a comment
that this is useful for Go - but it might be used in the future in other
circumstances.

Also, since what you end up doing is setting _entry_point, I wonder if we
really need this patch to choose a fixed name for the entry point, given
that I think the user can choose the entry point when linking the shared
object: with the linker's "-e" option (entry point), we can define which
function (in your case GoMain) is the entry point.  Perhaps something like

  go build -buildmode=c-shared  -ldflags="-extldflags '-e GoMain'"
hello.go

would work? (I didn't test).

-- 
You received this message because you are subscribed to the Google Groups "OSv 
Development" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to osv-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[COMMIT osv master] Update apps/ to latest version

2016-11-03 Thread Commit Bot

From: Nadav Har'El 
Committer: Nadav Har'El 
Branch: master

Update apps/ to latest version

Signed-off-by: Nadav Har'El 

---
diff --git a/apps b/apps
--- a/apps
+++ b/apps
@@ -1 +1 @@
-Subproject commit d09130aace1671aaac79f76182681eeed0a8df9a
+Subproject commit f8731d0253a7537a15c12321151ef8c6168ad902

--
You received this message because you are subscribed to the Google Groups "OSv 
Development" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to osv-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[COMMIT osv-apps master] Renamed openjdk9 app folder to a name without dot so that capstan image can be built

2016-11-03 Thread Commit Bot

From: Waldemar Kozaczuk 
Committer: Nadav Har'El 
Branch: master

Renamed openjdk9 app folder to a name without dot so that capstan image can  
be built


Fixed wrong folder name in module.py

Signed-off-by: Waldemar Kozaczuk 
Message-Id: <1478175918-3254-1-git-send-email-jwkozac...@gmail.com>

---
diff --git a/openjdk9-ea-java-base/Makefile b/openjdk9-ea-java-base/Makefile
--- a/openjdk9-ea-java-base/Makefile
+++ b/openjdk9-ea-java-base/Makefile
null
diff --git a/openjdk9-ea-java-base/latest.sh  
b/openjdk9-ea-java-base/latest.sh

--- a/openjdk9-ea-java-base/latest.sh
+++ b/openjdk9-ea-java-base/latest.sh
null
diff --git a/openjdk9-ea-java-base/module.py  
b/openjdk9-ea-java-base/module.py

--- a/openjdk9-ea-java-base/module.py
+++ b/openjdk9-ea-java-base/module.py
@@ -19,8 +19,8 @@
 api.require('josvsym')

 usr_files = FileMap()
-usr_files.add('${OSV_BASE}/apps/openjdk9-ea/install').to('/').allow_symlink()
-usr_files.add('${OSV_BASE}/apps/openjdk9-ea/install/usr/lib/jvm/java') \
+usr_files.add('${OSV_BASE}/apps/openjdk9-ea-java-base/install').to('/').allow_symlink()
+usr_files.add('${OSV_BASE}/apps/openjdk9-ea-java-base/install/usr/lib/jvm/java')  
\

 .to('/usr/lib/jvm/java') \
 .allow_symlink()
 usr_files.link('/usr/lib/jvm/jre').to('/usr/lib/jvm/java/jre')
diff --git a/openjdk9-ea-java-base/usr.manifest  
b/openjdk9-ea-java-base/usr.manifest

--- a/openjdk9-ea-java-base/usr.manifest
+++ b/openjdk9-ea-java-base/usr.manifest
null

--
You received this message because you are subscribed to the Google Groups "OSv 
Development" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to osv-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.