Alon Bar-Lev has uploaded a new change for review.

Change subject: packaging: build: enforce at least 10240 file descriptors for 
locale builds
......................................................................

packaging: build: enforce at least 10240 file descriptors for locale builds

gwt creates a lot of files without closing them, building locales
overflow the default 1024 limit.

Change-Id: I6e624afffc2eefb381bee25e3792bf7be60cf0b3
Signed-off-by: Alon Bar-Lev <[email protected]>
---
M Makefile
M README.developer
2 files changed, 25 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/15/25115/1

diff --git a/Makefile b/Makefile
index 19a6960..222722f 100644
--- a/Makefile
+++ b/Makefile
@@ -314,6 +314,15 @@
 
 
 validations:   generated-files
+       if [ "$(BUILD_LOCALES)" != 0 ]; then \
+               require="10240"; \
+               current="$$(ulimit -n)"; \
+               if [ "$${current}" -lt "$${require}" ]; then \
+                       echo "Building locales requires more than $${require} 
available file descriptors, currently $${require}" >&2; \
+                       echo "Refer to README.developer for farther 
instructions" >&2; \
+                       false; \
+               fi; \
+       fi
        if [ "$(BUILD_VALIDATION)" != 0 ]; then \
                build/shell-check.sh && \
                build/python-check.sh && \
diff --git a/README.developer b/README.developer
index ade1e4d..9027e00 100644
--- a/README.developer
+++ b/README.developer
@@ -32,6 +32,22 @@
 >=Jboss 7.1.1 is required, download and extract it if not installed
 using distribution package management.
 
+SYSTEM SETTINGS
+
+Build locales requires at least 10240 file descriptors, create the
+following file, replace <user> with user that is used for building,
+and logout/login:
+
+/etc/security/limits.d/10-nofile.conf
+---
+<user> hard nofile 10240
+#<user> soft nofile 10240  # optional, to apply automatically
+---
+
+If soft limit was not set, before building, apply new limit using:
+
+ $ ulimit -n 10240
+
 POSTGRESQL ACCESSIBILITY
 
 Configure postgresql to accept user and password:


-- 
To view, visit http://gerrit.ovirt.org/25115
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I6e624afffc2eefb381bee25e3792bf7be60cf0b3
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to