Re: svn commit: r309462 - head/tests/sys/kern

2016-12-03 Thread Ngie Cooper

> On Dec 3, 2016, at 12:02, Conrad Meyer  wrote:
> 
>> On Fri, Dec 2, 2016 at 6:09 PM, Ngie Cooper  wrote:
>> Author: ngie
>> Date: Sat Dec  3 02:09:23 2016
>> New Revision: 309462
>> URL: https://svnweb.freebsd.org/changeset/base/309462
>> 
>> Log:
>>  Fix test coredump_phnum_test:coredump_phnum require.config specification
>> 
>>  The requirement is set via `atf_set "require.config" 
>> "allow_sysctl_side_effects"',
>>  not `atf_set "require.config" "allow_sysctl_side_effects"'
>> 
>>  X-MFC with: r308177
>>  Pointyhat to: cmeyer
> 
> Ngie,
> 
> This line was added verbatim at your request:
> https://reviews.freebsd.org/D7264?id=18609#inline-43800

Ok, I'll take that part of the pointhat.
-Ngie
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r309462 - head/tests/sys/kern

2016-12-03 Thread Conrad Meyer
On Fri, Dec 2, 2016 at 6:09 PM, Ngie Cooper  wrote:
> Author: ngie
> Date: Sat Dec  3 02:09:23 2016
> New Revision: 309462
> URL: https://svnweb.freebsd.org/changeset/base/309462
>
> Log:
>   Fix test coredump_phnum_test:coredump_phnum require.config specification
>
>   The requirement is set via `atf_set "require.config" 
> "allow_sysctl_side_effects"',
>   not `atf_set "require.config" "allow_sysctl_side_effects"'
>
>   X-MFC with: r308177
>   Pointyhat to: cmeyer

Ngie,

This line was added verbatim at your request:
https://reviews.freebsd.org/D7264?id=18609#inline-43800

Conrad
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r309462 - head/tests/sys/kern

2016-12-02 Thread Ngie Cooper
Author: ngie
Date: Sat Dec  3 02:09:23 2016
New Revision: 309462
URL: https://svnweb.freebsd.org/changeset/base/309462

Log:
  Fix test coredump_phnum_test:coredump_phnum require.config specification
  
  The requirement is set via `atf_set "require.config" 
"allow_sysctl_side_effects"',
  not `atf_set "require.config" "allow_sysctl_side_effects"'
  
  X-MFC with: r308177
  Pointyhat to: cmeyer

Modified:
  head/tests/sys/kern/coredump_phnum_test.sh

Modified: head/tests/sys/kern/coredump_phnum_test.sh
==
--- head/tests/sys/kern/coredump_phnum_test.sh  Sat Dec  3 01:55:38 2016
(r309461)
+++ head/tests/sys/kern/coredump_phnum_test.sh  Sat Dec  3 02:09:23 2016
(r309462)
@@ -32,8 +32,8 @@
 atf_test_case coredump_phnum cleanup
 coredump_phnum_head()
 {
-   atf_set "allow_sysctl_side_effects" "1"
atf_set "descr" "More than 65534 segments"
+   atf_set "require.config" "allow_sysctl_side_effects"
atf_set "require.progs" "readelf procstat"
atf_set "require.user" "root"
 }
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"