Re: [PATCH] Makefile.am: don't confuse CFLAGS and CFLAGS_FOR_BUILD

2016-12-07 Thread Dmitry V. Levin
Hi Thomas, On Wed, Dec 07, 2016 at 09:32:20PM +0100, Thomas De Schampheleire wrote: > On Wed, Dec 7, 2016 at 4:18 PM, Dmitry V. Levin wrote: > > On Wed, Dec 07, 2016 at 02:58:35PM +0300, Dmitry V. Levin wrote: > >> On Tue, Dec 06, 2016 at 09:06:08PM +0100, Thomas De Schampheleire wrote: > >> > On

Re: [PATCH] Makefile.am: don't confuse CFLAGS and CFLAGS_FOR_BUILD

2016-12-07 Thread Thomas De Schampheleire
Hi Dmitry, On Wed, Dec 7, 2016 at 4:18 PM, Dmitry V. Levin wrote: > On Wed, Dec 07, 2016 at 02:58:35PM +0300, Dmitry V. Levin wrote: >> On Tue, Dec 06, 2016 at 09:06:08PM +0100, Thomas De Schampheleire wrote: >> > On Wed, Nov 16, 2016 at 8:27 PM, Dmitry V. Levin wrote: >> [...] >> > > The correct

Re: tests: add prctl-name.test, prctl-pdeathsig.test and prctl-tsc.test

2016-12-07 Thread Eugene Syromyatnikov
On Wed, Dec 7, 2016 at 3:29 PM, JingPiao Chen wrote: >>Your patch has been applied in master, >>https://github.com/strace/strace/commit/d945e74377fe069de0fe1aa2823442cf8dbdbb52 >>. Thank you for your contribution. > > What can I contribute more? This is my first times contribute open source > soft

Re: tests: add prctl-name.test, prctl-pdeathsig.test and prctl-tsc.test

2016-12-07 Thread JingPiao Chen
>Your patch has been applied in master, >https://github.com/strace/strace/commit/d945e74377fe069de0fe1aa2823442cf8dbdbb52 >. Thank you for your contribution. What can I contribute more? This is my first times contribute open source software, lack of experience, I cann't find the way deep learnin

Re: [PATCH] Makefile.am: don't confuse CFLAGS and CFLAGS_FOR_BUILD

2016-12-07 Thread Dmitry V. Levin
On Wed, Dec 07, 2016 at 02:58:35PM +0300, Dmitry V. Levin wrote: > On Tue, Dec 06, 2016 at 09:06:08PM +0100, Thomas De Schampheleire wrote: > > On Wed, Nov 16, 2016 at 8:27 PM, Dmitry V. Levin wrote: > [...] > > > The correct fix is going to be more complex, e.g. > > > - move all gl_WARN_ADD/WARN_C

Re: tests: add prctl-name.test, prctl-pdeathsig.test and prctl-tsc.test

2016-12-07 Thread Eugene Syromyatnikov
On Thu, Nov 17, 2016 at 1:17 PM, JingPiao Chen wrote: > ... Your patch has been applied in master, https://github.com/strace/strace/commit/d945e74377fe069de0fe1aa2823442cf8dbdbb52 . Thank you for your contribution. -- Eugene Syromyatnikov mailto:evg...@gmail.com xmpp:esyr@jabber.{ru|org}

Re: readv.test is fail on Ubuntu 16.04 x86_64

2016-12-07 Thread Dmitry V. Levin
Hi, On Sun, Dec 04, 2016 at 07:59:48PM +0800, JingPiao Chen wrote: > Environment: Ubuntu 16.04 x86_64 > Version: dcc9b9cdb2cd6de7c5b300b8d1f5dfb3f0b31ab2 > > readv.test is fail on Ubuntu 16.04 x86_64 > > test-suite.log/readv.log: > > 251 FAIL: readv > 252 === > 253 > 254 ../strace: inv

Re: [PATCH] Makefile.am: don't confuse CFLAGS and CFLAGS_FOR_BUILD

2016-12-07 Thread Dmitry V. Levin
Hi Thomas, On Tue, Dec 06, 2016 at 09:06:08PM +0100, Thomas De Schampheleire wrote: > On Wed, Nov 16, 2016 at 8:27 PM, Dmitry V. Levin wrote: [...] > > The correct fix is going to be more complex, e.g. > > - move all gl_WARN_ADD/WARN_CFLAGS related code from configure.ac to a > > separate m4 mac