On Wed, Jul 8, 2009 at 7:36 AM, Michal Simek<[email protected]> wrote: > > Of course not because I cross compile ltp for Microblaze - correct > /etc/passwd - group is on > target system. Maybe worth to remove calling IDcheck.sh from make > install and call it only before > running the test. >
Michal,
Please give this patch a try, and instead when you run make
install, do SKIP_IDCHECK=1.
HTH,
-Garrett
This patch does the same as the previous released patch in this
thread, which is:
IDcheck.sh:
1. Fixes the DESTDIR != "" | "/" behavior.
2. Spew out less awk errors if files don't exist by instead
short-circuiting the logic to detect whether or not the file exists in
the fe function subroutine.
In addition to the above, this adds SKIP_IDCHECK behavior, by request
of Michal, so it's no longer required for make install, and can be
disabled by entering specifying the variable SKIP_IDCHECK=1 when
calling make install, e.g...
make \
[make-options-and-variables] \
SKIP_IDCHECK=1 \
install
Warning! This patch has Makefile changes and as such the files patched
contents should be committed directly, NOT the inline text shown below
(for reviewing purposes only).
Signed-off-by: Garrett Cooper <[email protected]>
Index: IDcheck.sh
===================================================================
RCS file: /cvsroot/ltp/ltp/IDcheck.sh,v
retrieving revision 1.19
diff -u -r1.19 IDcheck.sh
--- IDcheck.sh 7 Jul 2009 14:30:27 -0000 1.19
+++ IDcheck.sh 9 Jul 2009 05:21:34 -0000
@@ -22,7 +22,6 @@
#
# Prompt user if ids/groups should be created
-clear
echo "Checking for required user/group ids"
echo ""
@@ -41,12 +40,11 @@
# find entry.
fe() {
- ID=$1; shift
- FILE=$1; shift
- awk "/^$ID:/ { FOUND=1 } END { if (\$FOUND == 1) { exit 0; }
exit 1; }" \
- "$FILE"
- ec=$?
- echo "$ID => $ec"
+ ID=$1; shift
+ FILE=$1; shift
+ [ -e "$FILE" ] || return $?
+ awk "/^$ID:/ { FOUND=1 } END { if (\$FOUND == 1) { exit 1; } exit 0; }" \
+ "$FILE"
}
prompt_for_create() {
@@ -70,13 +68,12 @@
EUID=$(id -u)
fi
-if [ -e "$passwd" -a ! -r "$passwd" ] ; then
- echo "/etc/passwd not readable by uid $EUID"
+for i in "$passwd" "$group"; do
+ if [ -e "$i" -a ! -r "$i" ] ; then
+ echo "$i not readable by uid $EUID"
exit 1
-elif [ -e "$group" -a ! -r "$group" ] ; then
- echo "$group not readable by uid $EUID"
- exit 1
-fi
+ fi
+done
fe bin "$passwd"; NO_BIN_ID=$?
fe daemon "$passwd"; NO_DAEMON_ID=$?
@@ -110,10 +107,10 @@
#debug_vals
if [ $CREATE_ENTRIES -ne 0 ] ; then
- if ! touch "$group" ; then
- echo "Couldn't touch $group"
- exit 1
- fi
+ if ! touch "$group" "$passwd" 2>/dev/null; then
+ echo "Failed to touch $group or $passwd"
+ exit 1
+ fi
fi
make_user_group() {
Index: Makefile
===================================================================
RCS file: /cvsroot/ltp/ltp/Makefile,v
retrieving revision 1.39
diff -u -r1.39 Makefile
--- Makefile 25 Mar 2009 06:17:48 -0000 1.39
+++ Makefile 9 Jul 2009 05:27:34 -0000
@@ -14,6 +14,10 @@
export CFLAGS += -Wall $(CROSS_CFLAGS)
export CC AR RANLIB CPPFLAGS LDFLAGS HAS_NUMA
+# SKIP_IDCHECK != 1, defaults to previous behavior, which is to execute
+# IDcheck.sh at the end of `make install'.
+SKIP_IDCHECK ?= 0
+
-include config.mk
VPATH += include m4
@@ -36,8 +40,9 @@
@$(MAKE) -C m4 install
@$(MAKE) -C doc/man1 install
@$(MAKE) -C doc/man3 install
-
+ifneq ($(filter $(SKIP_IDCHECK)),1)
@./IDcheck.sh
+endif
libltp.a: config.h
@$(MAKE) -C lib $@
IDcheck_awk_fix+skip_check.diff
Description: Binary data
------------------------------------------------------------------------------ Enter the BlackBerry Developer Challenge This is your chance to win up to $100,000 in prizes! For a limited time, vendors submitting new applications to BlackBerry App World(TM) will have the opportunity to enter the BlackBerry Developer Challenge. See full prize details at: http://p.sf.net/sfu/Challenge
_______________________________________________ Ltp-list mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ltp-list
