Hi!
> We have ported LTP open source (Version: ltp-full-20110228) on Android
> 2.2 Froyo. It will validate the reliability, robustness, and stability
> of Android. We want to submit the same to open source community. PFA
> patch for the same. Please let me know if I have to do anything else
> regarding submission of this patch.
First of all, you should use lastest LTP in order to aviod collisions.
The best fit for development would be a git checkout and please use
unified diff format (diff -u) from now on.
diff -crB ltp-full-20110228//Android.mk ltp//Android.mk
*** ltp-full-20110228//Android.mk 2011-04-26 03:28:58.000000000 -0400
--- ltp//Android.mk 2011-06-10 07:11:04.000000000 -0400
***************
*** 10,16 ****
export LTP_SCRIPTS_DIR := $(LTP_ROOT_DIR)/scripts
export LTP_TESTCASES_DIR := $(LTP_ROOT_DIR)/testcases
!
subdirs := \
external/ltp/lib/Android.mk \
--- 10,16 ----
export LTP_SCRIPTS_DIR := $(LTP_ROOT_DIR)/scripts
export LTP_TESTCASES_DIR := $(LTP_ROOT_DIR)/testcases
! LOCAL_CFLAGS := -D_ANDROID
subdirs := \
external/ltp/lib/Android.mk \
Hmm, that seems to be incomplete, we don't have Android.mk in our
sources.
diff -crB ltp-full-20110228//include/ltp_signal.h ltp//include/ltp_signal.h
*** ltp-full-20110228//include/ltp_signal.h 2011-06-09 05:00:49.000000000
-0400
--- ltp//include/ltp_signal.h 2011-08-02 07:11:21.000000000 -0400
***************
*** 23,28 ****
--- 23,30 ----
*
* Author: Liu Bo <[email protected]>
* Author: Garrett Cooper <[email protected]>
+ * 08/01/11 - Updated: Anupam Srivastava, [email protected]
+ * - Config.h is not supported by Android
*
*/
Please don't write dates and comments here. We use git to track this
informations. If you have changed a file just add simple line with your
name there.
Also the diff shows only change in the commend, is it incomplete too?
diff -crB ltp-full-20110228//include/tlibio.h ltp//include/tlibio.h
*** ltp-full-20110228//include/tlibio.h 2011-06-09 05:01:54.000000000 -0400
--- ltp//include/tlibio.h 2011-08-02 07:12:37.000000000 -0400
***************
*** 28,33 ****
--- 28,35 ----
* For further information regarding this notice, see:
*
* http://oss.sgi.com/projects/GenInfo/NoticeExplan/
+ * 08/01/11 - Updated: Anupam Srivastava, [email protected]
+ * - aio.h is not supported by Android
*/
#define LIO_IO_SYNC 00001 /* read/write */
The same problem as well.
... snip ...
diff -crB ltp-full-20110228//testcases/kernel/connectors/Android.mk
ltp//testcases/kernel/connectors/Android.mk
*** ltp-full-20110228//testcases/kernel/connectors/Android.mk 2011-06-29
00:58:03.000000000 -0400
--- ltp//testcases/kernel/connectors/Android.mk 2011-06-29 01:37:34.000000000
-0400
***************
*** 1,19 ****
! ifneq ($(TARGET_SIMULATOR),true)
! LOCAL_PATH:= $(call my-dir)
!
! include $(CLEAR_VARS)
!
! subdirs := \
! external/ltp/testcases/kernel/pec/Android.mk
!
! LOCAL_C_INCLUDES += $(LOCAL_PATH)
! LOCAL_C_INCLUDES += include
!
This file has windows line ends, remove them.
The rest of the patch, if I understand it correctly mostly removes
subdirectories, which is simply wrong. There should be something similar
to ifeq(UCLINUX,1) to skip these tests.
You should also split the patch in several smaller so that each of them
changes one thing at a time.
--
Cyril Hrubis
[email protected]
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
